Markus Kroetzsch has submitted this change and it was merged. Change subject: Fixed Bug 42032 ......................................................................
Fixed Bug 42032 The bug is caused by more than one entry referring to special properties in the SMW IDs table. There are multiple reasons why this has happened: * Predefined properties are stored by their keys (e.g., _TYPE) but if they also have a property page, this page will be called like their current label (e.g., "has type"); this was not matched * The refresh script carelessly created refresh jobs even for predefined properties, thus refreshing pages with titles like "TYPE" that never existed. Both problems surfaced due to the new way of producing Special:Properties. The new code requires Store3 to be reset and recreated to fix any existing problems caused by the earlier misbehaviour. A simple refresh is not going to delete the obsolete ID entries. Note for review: SMW_Sql3SmwIds.php applies some inconsistent naming for new symbols. The old ones should be fixed in another commit, but this is not release critical and would require more far-reaching changes since some of the methods that should be renamed are public. Can be done in SMW 1.9. Change set 2: Using property IDs (with property keys instead of titles) in all places where a property is used requires us to also reconstruct wikipage DIs from such IDs now. If a predefined property is used as a property value, then this is necessary. The DIHandler for wikipage has been augmented to do this, and all code that created SMWDIWikipage from SMW ID table contents is now calling the handler instead. Also fixed a lot of documentation. Change-Id: Ifd6267cbd3fca1c3bee1b1927c70b91bd8128356 --- M includes/dataitems/SMW_DI_Property.php M includes/storage/SQLStore/SMW_DIHandler_WikiPage.php M includes/storage/SQLStore/SMW_SQLStore3_Queries.php M includes/storage/SQLStore/SMW_SQLStore3_Readers.php M includes/storage/SQLStore/SMW_SQLStore3_SetupHandlers.php M includes/storage/SQLStore/SMW_SQLStore3_Writers.php M includes/storage/SQLStore/SMW_Sql3SmwIds.php 7 files changed, 398 insertions(+), 156 deletions(-) Approvals: Markus Kroetzsch: Verified; Looks good to me, approved Jeroen De Dauw: Verified; Looks good to me, but someone else must approve -- To view, visit https://gerrit.wikimedia.org/r/34211 To unsubscribe, visit https://gerrit.wikimedia.org/r/settings Gerrit-MessageType: merged Gerrit-Change-Id: Ifd6267cbd3fca1c3bee1b1927c70b91bd8128356 Gerrit-PatchSet: 2 Gerrit-Project: mediawiki/extensions/SemanticMediaWiki Gerrit-Branch: master Gerrit-Owner: Markus Kroetzsch <[email protected]> Gerrit-Reviewer: Daniel Werner <[email protected]> Gerrit-Reviewer: Denny Vrandecic <[email protected]> Gerrit-Reviewer: Foxtrott <[email protected]> Gerrit-Reviewer: Jeroen De Dauw <[email protected]> Gerrit-Reviewer: Kghbln <[email protected]> Gerrit-Reviewer: Markus Kroetzsch <[email protected]> Gerrit-Reviewer: Mwjames <[email protected]> Gerrit-Reviewer: Nischayn22 <[email protected]> Gerrit-Reviewer: Yaron Koren <[email protected]> _______________________________________________ MediaWiki-commits mailing list [email protected] https://lists.wikimedia.org/mailman/listinfo/mediawiki-commits
