[Wikidata-bugs] [Maniphest] [Commented On] T189842: Undefined index error in CachingResultsBuilder when checking constraints on empty entity

2018-03-26 Thread gerritbot
gerritbot added a comment. Change 420993 abandoned by Lucas Werkmeister (WMDE): Revert "Work around CheckingResultsBuilder bug" Reason: CachingResultsBuilder has been removed and we never copied this workaround to its successor, CachingResultsSource. https://gerrit.wikimedia.org/r/420993TASK

[Wikidata-bugs] [Maniphest] [Commented On] T189842: Undefined index error in CachingResultsBuilder when checking constraints on empty entity

2018-03-26 Thread thiemowmde
thiemowmde added a comment. Guys, please communicate. This back-and-forth does not look like fun.TASK DETAILhttps://phabricator.wikimedia.org/T189842EMAIL PREFERENCEShttps://phabricator.wikimedia.org/settings/panel/emailpreferences/To: Lucas_Werkmeister_WMDE, thiemowmdeCc: thiemowmde, Jonas,

[Wikidata-bugs] [Maniphest] [Commented On] T189842: Undefined index error in CachingResultsBuilder when checking constraints on empty entity

2018-03-21 Thread gerritbot
gerritbot added a comment. Change 420993 had a related patch set uploaded (by Lucas Werkmeister (WMDE); owner: Lucas Werkmeister (WMDE)): [mediawiki/extensions/WikibaseQualityConstraints@master] Revert "Work around CheckingResultsBuilder bug" https://gerrit.wikimedia.org/r/420993TASK

[Wikidata-bugs] [Maniphest] [Commented On] T189842: Undefined index error in CachingResultsBuilder when checking constraints on empty entity

2018-03-21 Thread gerritbot
gerritbot added a comment. Change 420321 merged by jenkins-bot: [mediawiki/extensions/WikibaseQualityConstraints@master] Ensure that result is populated even for empty entities https://gerrit.wikimedia.org/r/420321TASK DETAILhttps://phabricator.wikimedia.org/T189842EMAIL

[Wikidata-bugs] [Maniphest] [Commented On] T189842: Undefined index error in CachingResultsBuilder when checking constraints on empty entity

2018-03-21 Thread gerritbot
gerritbot added a comment. Change 420320 merged by jenkins-bot: [mediawiki/extensions/WikibaseQualityConstraints@master] Add EntityContextCursor https://gerrit.wikimedia.org/r/420320TASK DETAILhttps://phabricator.wikimedia.org/T189842EMAIL

[Wikidata-bugs] [Maniphest] [Commented On] T189842: Undefined index error in CachingResultsBuilder when checking constraints on empty entity

2018-03-21 Thread gerritbot
gerritbot added a comment. Change 420319 merged by jenkins-bot: [mediawiki/extensions/WikibaseQualityConstraints@master] Replace NullResult’s Context with ContextCursor https://gerrit.wikimedia.org/r/420319TASK DETAILhttps://phabricator.wikimedia.org/T189842EMAIL

[Wikidata-bugs] [Maniphest] [Commented On] T189842: Undefined index error in CachingResultsBuilder when checking constraints on empty entity

2018-03-21 Thread gerritbot
gerritbot added a comment. Change 420010 merged by jenkins-bot: [mediawiki/extensions/WikibaseQualityConstraints@master] Work around CheckingResultsBuilder bug https://gerrit.wikimedia.org/r/420010TASK DETAILhttps://phabricator.wikimedia.org/T189842EMAIL

[Wikidata-bugs] [Maniphest] [Commented On] T189842: Undefined index error in CachingResultsBuilder when checking constraints on empty entity

2018-03-19 Thread gerritbot
gerritbot added a comment. Change 420321 had a related patch set uploaded (by Lucas Werkmeister (WMDE); owner: Lucas Werkmeister (WMDE)): [mediawiki/extensions/WikibaseQualityConstraints@master] Ensure that result is populated even for empty entities https://gerrit.wikimedia.org/r/420321TASK

[Wikidata-bugs] [Maniphest] [Commented On] T189842: Undefined index error in CachingResultsBuilder when checking constraints on empty entity

2018-03-19 Thread gerritbot
gerritbot added a comment. Change 420319 had a related patch set uploaded (by Lucas Werkmeister (WMDE); owner: Lucas Werkmeister (WMDE)): [mediawiki/extensions/WikibaseQualityConstraints@master] Replace NullResult’s Context with ContextCursor https://gerrit.wikimedia.org/r/420319TASK

[Wikidata-bugs] [Maniphest] [Commented On] T189842: Undefined index error in CachingResultsBuilder when checking constraints on empty entity

2018-03-19 Thread gerritbot
gerritbot added a comment. Change 420320 had a related patch set uploaded (by Lucas Werkmeister (WMDE); owner: Lucas Werkmeister (WMDE)): [mediawiki/extensions/WikibaseQualityConstraints@master] Add EntityContextCursor https://gerrit.wikimedia.org/r/420320TASK

[Wikidata-bugs] [Maniphest] [Commented On] T189842: Undefined index error in CachingResultsBuilder when checking constraints on empty entity

2018-03-19 Thread Lucas_Werkmeister_WMDE
Lucas_Werkmeister_WMDE added a comment. This is already a violation of the wbcheckconstraints API, I think (consumers should probably be able to expect the entity ID and an empty claims array, at least) Well, not quite… we actually work around this in CheckConstraints.php itself, so the API

[Wikidata-bugs] [Maniphest] [Commented On] T189842: Undefined index error in CachingResultsBuilder when checking constraints on empty entity

2018-03-16 Thread Lucas_Werkmeister_WMDE
Lucas_Werkmeister_WMDE added a comment. I uploaded a workaround to fix the PHP warning at least; do you think this is serious enough to warrant a backport?TASK DETAILhttps://phabricator.wikimedia.org/T189842EMAIL PREFERENCEShttps://phabricator.wikimedia.org/settings/panel/emailpreferences/To:

[Wikidata-bugs] [Maniphest] [Commented On] T189842: Undefined index error in CachingResultsBuilder when checking constraints on empty entity

2018-03-16 Thread gerritbot
gerritbot added a comment. Change 420010 had a related patch set uploaded (by Lucas Werkmeister (WMDE); owner: Lucas Werkmeister (WMDE)): [mediawiki/extensions/WikibaseQualityConstraints@master] Work around CheckingResultsBuilder bug https://gerrit.wikimedia.org/r/420010TASK