Page MenuHomePhabricator

Require rows passed to "loadAllFromArray()" have unique keys
ClosedPublic

Authored by epriestley on Jul 9 2020, 5:56 PM.
Tags
None
Referenced Files
Unknown Object (File)
Thu, Apr 18, 1:32 PM
Unknown Object (File)
Mar 5 2024, 8:52 AM
Unknown Object (File)
Mar 3 2024, 5:42 PM
Unknown Object (File)
Feb 21 2024, 12:42 PM
Unknown Object (File)
Feb 19 2024, 7:21 AM
Unknown Object (File)
Feb 13 2024, 12:23 PM
Unknown Object (File)
Jan 11 2024, 11:57 AM
Unknown Object (File)
Jan 5 2024, 1:20 PM
Subscribers
None

Details

Summary

See PHI1809, which identified a bug in Project search where queries with a large number of slugs could paginate improperly.

This change detects problems in this category: cases where multiple rows with the same ID are passed to "loadAllFromArray()". It's likely that all cases it detects are cases where a GROUP BY is missing.

Since this might have some false positives or detect some things which aren't fundamentally problematic, I'm planning to hold it until the next release.

Test Plan
  • Reverted D21399, then created a project with multiple slugs and queried for one of them via "project.search". Hit this new exeception.
  • Browsed around a bit, didn't immediately catch any collateral damage.

Diff Detail

Repository
rP Phabricator
Lint
Lint Not Applicable
Unit
Tests Not Applicable