Development

Changeset 12369

You must first sign up to be able to contribute.

Changeset 12369

Show
Ignore:
Timestamp:
10/26/08 01:29:24 (6 years ago)
Author:
Kris.Wallsmith
Message:

[1.1, 1.2] fixed unnecessarily complex widget name generation logic

Files:

Legend:

Unmodified
Added
Removed
Modified
Copied
Moved
  • branches/1.1/lib/widget/sfWidgetFormSchema.class.php

    r11961 r12369  
    454454    $format = $this->getNameFormat(); 
    455455 
    456     if ('[%s]' == substr($format, -4)) 
    457     { 
    458       if (preg_match('/^(.+?)\[(.+)\]$/', $name, $match)) 
    459       { 
    460         $name = sprintf('%s[%s][%s]', substr($format, 0, -4), $match[1], $match[2]); 
    461       } 
    462       else 
    463       { 
    464         $name = sprintf('%s[%s]', substr($format, 0, -4), $name); 
    465       } 
     456    if ('[%s]' == substr($format, -4) && preg_match('/^(.+?)\[(.+)\]$/', $name, $match)) 
     457    { 
     458      $name = sprintf('%s[%s][%s]', substr($format, 0, -4), $match[1], $match[2]); 
    466459    } 
    467460    else if (false !== $format) 
  • branches/1.2/lib/widget/sfWidgetFormSchema.class.php

    r11992 r12369  
    548548    $format = $this->getNameFormat(); 
    549549 
    550     if ('[%s]' == substr($format, -4)) 
    551     { 
    552       if (preg_match('/^(.+?)\[(.+)\]$/', $name, $match)) 
    553       { 
    554         $name = sprintf('%s[%s][%s]', substr($format, 0, -4), $match[1], $match[2]); 
    555       } 
    556       else 
    557       { 
    558         $name = sprintf('%s[%s]', substr($format, 0, -4), $name); 
    559       } 
     550    if ('[%s]' == substr($format, -4) && preg_match('/^(.+?)\[(.+)\]$/', $name, $match)) 
     551    { 
     552      $name = sprintf('%s[%s][%s]', substr($format, 0, -4), $match[1], $match[2]); 
    560553    } 
    561554    else if (false !== $format)