Page MenuHomePhorge

Avoid extra query if we know no entry exists
ClosedPublic

Authored by mollekopf on Aug 27 2021, 12:48 PM.
Tags
None
Referenced Files
F11585775: D2851.id8206.diff
Thu, Mar 28, 1:03 PM
Unknown Object (File)
Sun, Mar 17, 4:36 PM
Unknown Object (File)
Feb 20 2024, 6:57 AM
Unknown Object (File)
Feb 9 2024, 6:12 PM
Unknown Object (File)
Feb 1 2024, 4:55 PM
Unknown Object (File)
Jan 25 2024, 4:14 AM
Unknown Object (File)
Jan 24 2024, 2:34 PM
Unknown Object (File)
Jan 24 2024, 7:45 AM
Subscribers

Details

Reviewers
machniak
Group Reviewers
Restricted Project
Commits
rK5ae2e7a89350: Avoid extra query if we know no entry exists

Diff Detail

Repository
rK kolab
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Event Timeline

mollekopf created this revision.
mollekopf added a reviewer: Restricted Project.Aug 27 2021, 12:48 PM
machniak subscribed.

Please, also fix this:

FILE: ...home/alec/repos/kolab/src/tests/Feature/Stories/GreylistTest.php
----------------------------------------------------------------------
FOUND 1 ERROR AFFECTING 1 LINE
----------------------------------------------------------------------
 434 | ERROR | [x] The closing brace for the class must go on the
     |       |     next line after the body
src/app/Policy/Greylist/Request.php
190

I usually would write this as if (!$noEntry && ($connect = $this->findConnectsCollection()->first())) { even if the brackets aren't required.

This revision now requires changes to proceed.Aug 27 2021, 12:51 PM
This revision was not accepted when it landed; it landed in state Needs Revision.Aug 27 2021, 1:05 PM
This revision was automatically updated to reflect the committed changes.