Last modified: 2012-05-27 23:54:14 UTC
Created attachment 10656 [details] Add a check in ApiQuerySemanticWatchlist.php to bailout or continue Error message Catchable fatal error: Argument 1 passed to SMWDIWikiPage::newFromTitle() must be an instance of Title, null given, called in ...\extensions\SemanticWatchlist\includes\SWL_ChangeSet.php on line 80 and defined in ...\extensions\SemanticMediaWiki\includes\dataitems\SMW_DI_WikiPage.php on line 137 Problem analysis A watched page has been deleted and Title::newFromID( $set->edit_page_id ) fails because the page does not exists any more. Solution Check before SWLChangeSet::newFromDBResult( $set ) is called if Title::newFromID( $set->edit_page_id ) results in an object or is empty.
Awesome, thanks for the fix. Gerrit change #9108 Looking at that code I now realize it really is not structured for efficiency... doing way to much queries while it could batch :)