[{exception_id}] {exception_url} BadMethodCallException from line 78 of /srv/mediawiki/php-1.31.0-wmf.10/extensions/ProofreadPage/includes/page/DatabaseIndexForPageLookup.php: Call to a member function exists() on a non-object (boolean)
Actual code:
$image = $this->repoGroup->findFile( $imageTitle ); if ( $image->exists() .. ) { .. }
Most likely a regression caused by rEPRP5f80293b30df: Introduces IndexForPageLookup (or a56ea07e0). (Both merged without review?).
Previously:
| Subject | Repo | Branch | Lines +/- | |
|---|---|---|---|---|
| Do not fail if no image is built from the Page: page name | mediawiki/extensions/ProofreadPage | master | +1 -1 |
Change 394770 had a related patch set uploaded (by Tpt; owner: Tpt):
[mediawiki/extensions/ProofreadPage@master] Do not fail if no image is built from the Page: page name
Most likely a regression caused by rEPRP5f80293b30df: Introduces IndexForPageLookup (or a56ea07e0).
It's rEPRP57e8b49becad27d3ce3eb3f994c131c206efd62c that merges duplicated code, removing the implementation with the bug fixed but keeping the one with the bug.
Both merged without review?
If you have time to do code reviews for ProofreadPage it would be great :-). I am the only volunteer with merge right on this extension and there is no WMF staff willing to do it.
Change 394770 abandoned by Tpt:
Do not fail if no image is built from the Page: page name
Reason:
Done by change I978ed2f9f3005210f6713a7f3ecde1fe0501a5ff