2012-01-31 122 views
3

我试图用CakePHP的SaveAll()方法更新记录,但是它增加了新行而不是更新它。saveAll()插入新行而不是更新

我的模型如下:商店hasOne地图

class Store extends AppModel { 
    var $name = 'Store'; 

    var $hasOne = array(
     'Map' => array( 
      'className' => 'Map', 
      'foreignKey' => 'store_id', 
      'dependent' => true, 
      'exclusive' => true 
     ) 
    ); 
} 

我的编辑形式所具有的存储ID(隐藏字段)告诉CakePHP的只更新特定的记录。

<?php echo $this->Form->input('Store.id'); ?> 
<?php echo $this->Form->input('Store.name', array('label' => 'Store name', 'required')); ?> 
<?php echo $this->Form->input('Map.latlng', array('label' => 'Map location', 'required')); ?> 

我在Store控制器中的编辑方法如下。

if ($this->Store->saveAll($this->data)) { 
    $this->Session->setFlash('Store has been updated.'); 
    $this->redirect(array('controller' => 'store', 'action' => 'admin_index')); 
} 

每当我编辑一个存储,商店名称更新正常,但CakePHP不断在Map表上插入一个新行。

我在这里错过了什么吗?由于

其他信息

我的调试($这个 - >数据)如下

Array 
(
    [Store] => Array 
     (
      [id] => 52 
      [name] => Sena The Accessorizer 
     ) 

    [Map] => Array 
     (
      [latlng] => 3.1580681, 101.7126311 
     ) 

) 
+0

显示'$ this-> data'的调试 – Dunhamzzz 2012-01-31 15:11:18

+0

@Dunhamzzz问题已更新。谢谢 – 2012-01-31 15:22:07

+0

@John - 什么是你的Form->创建? – Dave 2012-01-31 15:42:46

回答

3

正如@Dunhamzzz指出的那样,Map没有ID,因此CakePHP会插入一条新记录。

为了解决这个问题,我创建了一个隐藏字段用于Map.id

<?php echo $this->Form->input('Map.id'); ?> 

这将告诉CakePHP的更新而不是插入一个新的,特定的记录。

2

如果要更新数据,而不是储蓄,你应该总是设置你的ID想之前保存更新,就像这样:

// Set the store ID 
$this->Store->id = $this->data['Store']['id']; 

// Then save using that id 
if ($this->Store->saveAll($this->data)) { 
    $this->Session->setFlash('Store has been updated.'); 
    $this->redirect(array('controller' => 'store', 'action' => 'admin_index')); 
} 
+0

该id已被设置为他的形式(如您可以在他的数据阵列中看到) – Dave 2012-01-31 16:56:07

+0

仍然应该使用Oldskool的方法,因为表单数据可以由最终用户更改 – porjolovsky 2015-01-26 14:00:18

1

进行更新,你应该使用:

$this->Store->updateAll($this->data, array('Store.id' => $this->data['Store']['id'])); 
+1

不,您不应该在这种情况下。他正在更新一条记录。全部更新可防止发生某些蛋糕操作(如更新修改的字段) – 2012-02-01 10:12:08

相关问题