create逻辑错误

syber

原代码如下：

1. public function create($row)
   
2. {
   
3. if(!is_array($row))return FALSE;
   
4. $row = $this->__prepera_format($row);
   
5. if(empty($row))return FALSE;
   
6. foreach($row as $key => $value){
   
7. $cols[] = $key;
   
8. $vals[] = $this->escape($value);
   
9. }
   
10. $col = join(',', $cols);
    
11. $val = join(',', $vals);
    
12. 
    
13. $sql = "INSERT INTO {$this->tbl_name} ({$col}) VALUES ({$val})";
    
14. if( FALSE != $this->_db->exec($sql) ){ // 获取当前新增的ID
    
15. if( $newinserid = $this->_db->newinsertid() ){
    
16. return $newinserid;
    
17. }else{
    
18. return array_pop( $this->find($row, "{$this->pk} DESC",$this->pk) );
    
19. }
    
20. }
    
21. return FALSE;
    
22. }
    

复制代码

如果表没有设置pk的话（允许表没有pk），这里用create就会出错，具体是：
return array_pop( $this->find($row, "{$this->pk} DESC",$this->pk) );
因为没有设置pk，所以，这里的sql会构造出select * from xxx order by desc，造成语法错误。

修复：

1. 
   
2. public function create($row)
   
3. {
   
4.   if(!is_array($row))return FALSE;
   
5.   $row = $this->__prepera_format($row);
   
6.   if(empty($row))return FALSE;
   
7.   foreach($row as $key => $value){
   
8.    $cols[] = $key;
   
9.    $vals[] = $this->escape($value);
   
10.   }
    
11.   $col = join(',', $cols);
    
12.   $val = join(',', $vals);
    
13.   $sql = "INSERT INTO {$this->tbl_name} ({$col}) VALUES ({$val})";
    
14.   if( FALSE != $this->_db->exec($sql) ){ // 获取当前新增的ID
    
15.    if( $newinserid = $this->_db->newinsertid() ){
    
16.     return $newinserid;
    
17.    }else{
    
18.     if ($this->pk)
    
19.      return array_pop( $this->find($row, "{$this->pk} DESC",$this->pk) );
    
20.    }
    
21.   }
    
22.   return FALSE;
    
23. }
    

复制代码

加上一句：if ($this->pk)即可。

该贴已经同步到 syber的微博

jake

原则上是不允许不设置pk值的。即使该表没有pk，也可以设定一个。

当然，这个原则可以考虑改变一下。