Development

Changeset 7151

You must first sign up to be able to contribute.

Changeset 7151

Show
Ignore:
Timestamp:
01/22/08 14:46:18 (2 years ago)
Author:
pookey
Message:

allowing for a PK of 0 in the admin/CRUD generator

Files:

Legend:

Unmodified
Added
Removed
Modified
Copied
Moved
  • branches/1.1/lib/generator/sfCrudGenerator.class.php

    r6882 r7151  
    157157    { 
    158158      $fieldName  = sfInflector::underscore($pk->getPhpName()); 
    159       $test_pks[] = sprintf("!\$this->getRequestParameter(%s)", $fieldNameAsArgument ? "\$$fieldName" : "'".$fieldName."'"); 
     159      // 2 checks needed here, as '0' is a vaild PK. 
     160      $test_pks[] = sprintf("\$this->getRequestParameter(%s) === ''", $fieldNameAsArgument ? "\$$fieldName" : "'".$fieldName."'"); 
     161      $test_pks[] = sprintf("\$this->getRequestParameter(%s) === null", $fieldNameAsArgument ? "\$$fieldName" : "'".$fieldName."'"); 
    160162    } 
    161163 

The Sensio Labs Network

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