# Cakephp 3 trans can\`t rollback auto?

**URL:** <https://discourse.cakephp.org/t/cakephp-3-trans-can-t-rollback-auto/439>\
**Category:** Need Help\
**Tags:** release\
**Created:** [June 1, 2016, 10:50am UTC](https://discourse.cakephp.org/t/cakephp-3-trans-can-t-rollback-auto/439 "2016-06-01T10:50:26Z")\
**Posts on this page:** 3\
**Page:** 1

<div class="post-metadata">

**Author:** ![visonforcoding](https://yyz1.discourse-cdn.com/flex029/user_avatar/discourse.cakephp.org/visonforcoding/32/237_2.png) [@visonforcoding](https://discourse.cakephp.org/u/visonforcoding)\
**Post date:** [June 1, 2016, 10:50am UTC](https://discourse.cakephp.org/t/cakephp-3-trans-can-t-rollback-auto/439/1 "2016-06-01T10:50:26Z")

</div>

`$BookTable->connection()->transactional(function()use($book, $BookTable, $order, $OrderTable, &$errorFlag) { $errorFlag[] = $BookTable->save($book); $errorFlag[] = $OrderTable->save($order); });`

the second save if false,but it haven`t rollback.

---

<div class="post-metadata">

**Author:** ![jeremyharris](https://yyz1.discourse-cdn.com/flex029/user_avatar/discourse.cakephp.org/jeremyharris/32/66_2.png) [@jeremyharris](https://discourse.cakephp.org/u/jeremyharris)\
**Post date:** [June 1, 2016, 12:39pm UTC](https://discourse.cakephp.org/t/cakephp-3-trans-can-t-rollback-auto/439/2 "2016-06-01T12:39:28Z")

</div>

Transactions roll back when they raise an exception or return false, as stated in [the docs](http://book.cakephp.org/3.0/en/orm/database-basics.html#using-transactions). In your code, you are not returning anything so it assumes everything went well. You should return something like:

```php
return $BookTable->save($book) && $OrderTable->save($order);

```

---

<div class="post-metadata">

**Author:** ![visonforcoding](https://yyz1.discourse-cdn.com/flex029/user_avatar/discourse.cakephp.org/visonforcoding/32/237_2.png) [@visonforcoding](https://discourse.cakephp.org/u/visonforcoding)\
**Post date:** [June 2, 2016, 1:22am UTC](https://discourse.cakephp.org/t/cakephp-3-trans-can-t-rollback-auto/439/3 "2016-06-02T01:22:54Z")

</div>

yes. i saw it in the docs. thank you !!! I sloved it and your code is better .😁