resourceloader: Use AND instead of OR for upsert conds in saveFileDependencies()
authorReedy <reedy@wikimedia.org>
Thu, 23 May 2019 23:16:42 +0000 (00:16 +0100)
committerReedy <reedy@wikimedia.org>
Fri, 24 May 2019 16:41:44 +0000 (17:41 +0100)
commite765067d8a5f96f72dbca96f14ff728a75806966
treef0387c0a9c8375e32b687e15b9fa0adb49d74469
parent922ed49802867593e5b2807598181ab5edfef048
resourceloader: Use AND instead of OR for upsert conds in saveFileDependencies()

Follows-up e7b57d881a, which changed it from replace() to upsert()
but lost one of the wrapping arrays in doing so.

Previously updated many more rows than expected on Postgresql, when it
should only be updating individual rows, not all rows that match either
criteria.

SQL query before:
 WHERE ((md_module = 'jquery.makeCollapsible.styles') OR (md_skin = 'vector|en-gb'))

SQL query after:
 WHERE ((md_module = 'jquery.makeCollapsible.styles' AND md_skin = 'vector|en-gb'))

Not a problem on MySQL as upsert() is implemented differently there.

Bug: T222385
Change-Id: If8a458bf4543b297b3a06f31e09c0e77666bf7e6
RELEASE-NOTES-1.31
includes/resourceloader/ResourceLoaderModule.php