Development

Changeset 10950

You must first sign up to be able to contribute.

Changeset 10950

Show
Ignore:
Timestamp:
08/19/08 16:42:07 (11 months ago)
Author:
fabien
Message:

[1.0, 1.1, 1.2] fixed sfBrowser parseArgumentAsArray function discards array indices == 0 (closes #4197)

Files:

Legend:

Unmodified
Added
Removed
Modified
Copied
Moved
  • branches/1.0/lib/util/sfBrowser.class.php

    r10739 r10950  
    446446    { 
    447447      $var = &$vars; 
    448       $tmps = array_filter(preg_split('/(\[ | \[\] | \])/x', $name)); 
     448      $tmps = array_filter(preg_split('/(\[ | \[\] | \])/x', $name), create_function('$s', 'return $s !== "";')); 
    449449      foreach ($tmps as $tmp) 
    450450      { 
  • branches/1.0/test/unit/util/sfBrowserTest.php

    r6129 r10950  
    1111require_once(dirname(__FILE__).'/../../bootstrap/unit.php'); 
    1212 
    13 $t = new lime_test(43, new lime_output_color()); 
     13$t = new lime_test(44, new lime_output_color()); 
    1414 
    1515// ->click() 
     
    197197  setField('text', 'myothervalue')-> 
    198198  setField('article[title]', 'mytitle')-> 
     199  setField('myarray[0]', 'value0')-> 
     200  setField('myarray[1]', 'value1')-> 
     201  setField('myarray[2]', 'value2')-> 
    199202  click('submit') 
    200203; 
     
    202205$t->is($parameters['text'], 'myothervalue', '->setField() overrides default form field values'); 
    203206$t->is($parameters['article']['title'], 'mytitle', '->setField() overrides default form field values'); 
     207$t->is($parameters['myarray'], array('value0', 'value1', 'value2'), '->setField() overrides default form field values'); 
    204208 
    205209list($method, $uri, $parameters) = $b-> 
  • branches/1.1/lib/util/sfBrowser.class.php

    r10739 r10950  
    681681    { 
    682682      $var = &$vars; 
    683       $tmps = array_filter(preg_split('/(\[ | \[\] | \])/x', $name)); 
     683      $tmps = array_filter(preg_split('/(\[ | \[\] | \])/x', $name), create_function('$s', 'return $s !== "";')); 
    684684      foreach ($tmps as $tmp) 
    685685      { 
  • branches/1.1/test/unit/util/sfBrowserTest.php

    r8534 r10950  
    1111require_once(dirname(__FILE__).'/../../bootstrap/unit.php'); 
    1212 
    13 $t = new lime_test(50, new lime_output_color()); 
     13$t = new lime_test(51, new lime_output_color()); 
    1414 
    1515// ->click() 
     
    207207  setField('text', 'myothervalue')-> 
    208208  setField('article[title]', 'mytitle')-> 
     209  setField('myarray[0]', 'value0')-> 
     210  setField('myarray[1]', 'value1')-> 
     211  setField('myarray[2]', 'value2')-> 
    209212  click('submit') 
    210213; 
     
    212215$t->is($parameters['text'], 'myothervalue', '->setField() overrides default form field values'); 
    213216$t->is($parameters['article']['title'], 'mytitle', '->setField() overrides default form field values'); 
     217$t->is($parameters['myarray'], array('value0', 'value1', 'value2'), '->setField() overrides default form field values'); 
    214218 
    215219list($method, $uri, $parameters) = $b-> 
  • branches/1.2/lib/util/sfBrowser.class.php

    r10830 r10950  
    744744    { 
    745745      $var = &$vars; 
    746       $tmps = array_filter(preg_split('/(\[ | \[\] | \])/x', $name)); 
     746      $tmps = array_filter(preg_split('/(\[ | \[\] | \])/x', $name), create_function('$s', 'return $s !== "";')); 
    747747      foreach ($tmps as $tmp) 
    748748      { 
  • branches/1.2/test/unit/util/sfBrowserTest.php

    r8534 r10950  
    1111require_once(dirname(__FILE__).'/../../bootstrap/unit.php'); 
    1212 
    13 $t = new lime_test(50, new lime_output_color()); 
     13$t = new lime_test(51, new lime_output_color()); 
    1414 
    1515// ->click() 
     
    207207  setField('text', 'myothervalue')-> 
    208208  setField('article[title]', 'mytitle')-> 
     209  setField('myarray[0]', 'value0')-> 
     210  setField('myarray[1]', 'value1')-> 
     211  setField('myarray[2]', 'value2')-> 
    209212  click('submit') 
    210213; 
     
    212215$t->is($parameters['text'], 'myothervalue', '->setField() overrides default form field values'); 
    213216$t->is($parameters['article']['title'], 'mytitle', '->setField() overrides default form field values'); 
     217$t->is($parameters['myarray'], array('value0', 'value1', 'value2'), '->setField() overrides default form field values'); 
    214218 
    215219list($method, $uri, $parameters) = $b-> 

The Sensio Labs Network

Since 1998, Sensio Labs has been promoting the Open-Source software movement by providing quality web application development, training, consulting, and supporting several large Open-Source projects.