Change Details

@0xAFFF notes: If I try to rename a file I get this SQL error: ``` ERROR: FOR UPDATE is not allowed with aggregate functions STATEMENT: SELECT /* LocalFileMoveBatch::verifyDBUpdates */ COUNT(*) FROM "oldimage" WHERE oi_name = 'Ssh-schlüssel-zu-gitlab-hinzufügen.webm' LIMIT 1 FOR UPDATE ``` The used software versions: | Softare | Version | | MediaWiki | 1.28.0 | | PHP | 7.0.13 (apache2handler) | | PostgreSQL | 9.6.1 | | TimedMediaHandler | 0.5.0 (`64461e9`) 21:18, 25. Okt. 2016 | | MwEmbedSupport | 0.3.0 (`2e24ff4`) 00:03, 25. Okt. 2016 | @TheDJ notes This part of the code was recently changed by @aaron in ad95e0182356224dce27f53f7bbea24ffbcb22a7, so I think it's likely to be a regression in 1.28 This was originally reported at {T157424}

@0xAFFF notes: If I try to rename a file I get this SQL error: ``` ERROR: FOR UPDATE is not allowed with aggregate functions STATEMENT: SELECT /* LocalFileMoveBatch::verifyDBUpdates */ COUNT(*) FROM "oldimage" WHERE oi_name = 'Ssh-schlüssel-zu-gitlab-hinzufügen.webm' LIMIT 1 FOR UPDATE ``` The used software versions: | Softare | Version | | MediaWiki | 1.28.0 | | PHP | 7.0.13 (apache2handler) | | PostgreSQL | 9.6.1 | | TimedMediaHandler | 0.5.0 (64461e9) 21:18, 25. Okt. 2016 | | MwEmbedSupport | 0.3.0 (2e24ff4) 00:03, 25. Okt. 2016 | @TheDJ notes This part of the code was recently changed by @aaron in ad95e0182356224dce27f53f7bbea24ffbcb22a7, so I think it's likely to be a regression in 1.28 This was originally reported at {T157424}

@0xAFFF notes: If I try to rename a file I get this SQL error: ``` ERROR: FOR UPDATE is not allowed with aggregate functions STATEMENT: SELECT /* LocalFileMoveBatch::verifyDBUpdates */ COUNT(*) FROM "oldimage" WHERE oi_name = 'Ssh-schlüssel-zu-gitlab-hinzufügen.webm' LIMIT 1 FOR UPDATE ``` The used software versions: | Softare | Version | | MediaWiki | 1.28.0 | | PHP | 7.0.13 (apache2handler) | | PostgreSQL | 9.6.1 | | TimedMediaHandler | 0.5.0 (`0 (64461e9`) 21:18, 25. Okt. 2016 | | MwEmbedSupport | 0.3.0 (`0 (2e24ff4`) 00:03, 25. Okt. 2016 | @TheDJ notes This part of the code was recently changed by @aaron in ad95e0182356224dce27f53f7bbea24ffbcb22a7, so I think it's likely to be a regression in 1.28 This was originally reported at {T157424}