Question

我正在开发一个自定义Magento模块，需要将电子邮件地址列表保存到自定义表中。相关代码如下所示：

foreach($this->getNewSubscriptionsForAPI() as $email){
            $requestModel->setEmail($email);
            $requestModel->setAction('subscribe');
            $requestModel->setPosted(date('Y-m-d H:i:s'));
            $requestModel->save();
        }

getNewSubscriptionsForAPI()方法返回有效数据，$requestModel是有效的Magento模型。

但是，当此循环运行时，数据库适配器会简单地覆盖前一个记录之上的每个记录。即，如果我在phpMyAdmin中反复点击'browse'，我可以看到不同的电子邮件地址被写入数据库但总是在同一行，覆盖了前一个条目。 id字段已正确设置：作为主键，设置为AUTO_INCREMENT，并在资源模型中标记为id字段。

有趣的是，我可以使用上面的循环在核心newsletter/subscriber表中编写连续记录而没有任何问题。

当然，我可以简单地unset()模型并在循环的每个循环中将其从Mage::getModel()中取回，但这（a）似乎非常浪费，并且（b）破坏我的依赖注入设置用于测试（我不希望代码实例化自己的模型，但使用我传递给它的模型）。

有人能指出我正确的方向吗？

Answer 1

当Magento保存新对象时，它会自动为该对象生成一个id。保存对象后，将生成一个id。要使magento再次将您的数据识别为新模型，您只需取消设置模型id即可。我认为unsetData更优雅，因为它也会删除可能为前一个对象创建的任何数据： - ）。

Answer 2

啊，好像我可能回答了我自己的问题。通过在调用$requestModel->unsetData();之后立即调用$requestModel->save()，循环按预期工作。

unsetData是由Varien_Object提供的方法，Mage_Core_Model_Abstract继承自。

我很高兴有人能够提供更优雅的解决方案，我很高兴听到它。

Answer 3

我不知道这是否更优雅，但设法通过此代码解决您的问题。在每个循环中，将添加一个新行和新信息。

foreach($this->getNewSubscriptionsForAPI() as $email){
            $data = array('email' => $email, 'action' => 'subscribe', 
                   'posted' => date('Y-m-d H:i:s');
            $requestModel->setData($data)->save();
        }

考虑'email'，'action'和'posted'是数据库表的字段名称。

Answer 4

这是一个旧线程，但是对于遇到此问题的任何人，尽管unsetData（）建议是正确的，但此方法在循环内调用save（）函数的事实是不好的做法，并且肯定会减慢您的网站速度。

您可以在Model结构的ResourceModel中编写一个函数，以允许在事务内添加多个项目，这将更加安全并成倍地提高性能。

See Here和Here，以获得一些更好的主意。

在Magento中多次调用$ model-＆gt; save（）覆盖数据

4 个答案: