EcomDev_UrlRewrite icon indicating copy to clipboard operation
EcomDev_UrlRewrite copied to clipboard

Transaction rollbacks

Open phizab opened this issue 12 years ago • 0 comments

Is there a reason why there are no rollbacks in the module? At every transaction is only a commit, but as I know we need a rollback in case of an error. The commit should be wrapped with a try .. catch statement as follows:

$this->_getIndexAdapter()->beginTransaction();

[..]

try {
    $this->_getIndexAdapter()->commit();
} catch (Exception $e) {
    $this->_getIndexAdapter()->rollback();
}

Or am I wrong?

phizab avatar Nov 08 '13 09:11 phizab