diff options
| author | Jeremy Benoist <jeremy.benoist@gmail.com> | 2016-01-15 15:28:22 +0100 |
|---|---|---|
| committer | Jeremy Benoist <jeremy.benoist@gmail.com> | 2016-01-15 15:28:32 +0100 |
| commit | 78833672469f7beb0c4a195aa0a76f7ca4133057 (patch) | |
| tree | 87e295afbfc22e94262f3c498ce758a1101c939e /src/Wallabag/CoreBundle/Controller/EntryController.php | |
| parent | 790573d45899504bdecd2573c8f64018e23b139e (diff) | |
| download | wallabag-78833672469f7beb0c4a195aa0a76f7ca4133057.tar.gz wallabag-78833672469f7beb0c4a195aa0a76f7ca4133057.tar.zst wallabag-78833672469f7beb0c4a195aa0a76f7ca4133057.zip | |
Fix `findOneByUrl` side effect in tests
Fix #1566
Diffstat (limited to 'src/Wallabag/CoreBundle/Controller/EntryController.php')
| -rw-r--r-- | src/Wallabag/CoreBundle/Controller/EntryController.php | 3 |
1 files changed, 1 insertions, 2 deletions
diff --git a/src/Wallabag/CoreBundle/Controller/EntryController.php b/src/Wallabag/CoreBundle/Controller/EntryController.php index 3d22c7bc..1949bdf8 100644 --- a/src/Wallabag/CoreBundle/Controller/EntryController.php +++ b/src/Wallabag/CoreBundle/Controller/EntryController.php | |||
| @@ -49,8 +49,7 @@ class EntryController extends Controller | |||
| 49 | 49 | ||
| 50 | if ($form->isValid()) { | 50 | if ($form->isValid()) { |
| 51 | // check for existing entry, if it exists, redirect to it with a message | 51 | // check for existing entry, if it exists, redirect to it with a message |
| 52 | $existingEntry = $this->get('wallabag_core.entry_repository') | 52 | $existingEntry = $this->get('wallabag_core.entry_repository')->findByUrlAndUserId($entry->getUrl(), $this->getUser()->getId()); |
| 53 | ->existByUrlAndUserId($entry->getUrl(), $this->getUser()->getId()); | ||
| 54 | 53 | ||
| 55 | if (false !== $existingEntry) { | 54 | if (false !== $existingEntry) { |
| 56 | $this->get('session')->getFlashBag()->add( | 55 | $this->get('session')->getFlashBag()->add( |