Ppchelko has uploaded a new change for review. https://gerrit.wikimedia.org/r/315567
Change subject: Don't require performer in properties-change event. ...................................................................... Don't require performer in properties-change event. The page-properties-change event is emitted from LinksUpdateComplete hook, and in that hook the information about the performer might not be available. So, don't require the property. Change-Id: I139f12ae3f592dcf690617f9dd95a4df828c4a70 --- M jsonschema/mediawiki/page/properties-change/1.yaml M test/jsonschema/index.js M test/jsonschema/mediawiki/mediawiki_common_schema.yaml 3 files changed, 6 insertions(+), 32 deletions(-) git pull ssh://gerrit.wikimedia.org:29418/mediawiki/event-schemas refs/changes/67/315567/1 diff --git a/jsonschema/mediawiki/page/properties-change/1.yaml b/jsonschema/mediawiki/page/properties-change/1.yaml index 729231f..5c938c9 100644 --- a/jsonschema/mediawiki/page/properties-change/1.yaml +++ b/jsonschema/mediawiki/page/properties-change/1.yaml @@ -49,35 +49,6 @@ description: The name of the wiki database this event belongs to. type: string - performer: - description: Represents the user that performed this change. - type: object - properties: - user_id: - description: > - The user id that performed this change. This is optional, and - will not be present for anonymous users. - type: integer - user_text: - description: The text representation of the user that performed this change. - type: string - user_groups: - description: A list of the groups this user belongs to. E.g. bot, sysop etc. - type: array - items: - type: string - user_is_bot: - description: > - True if this user is considered to be a bot. This is checked - via the $user->isBot() method, which considers both user_groups - and user permissions. - type: boolean - required: - - user_text - - user_groups - - user_is_bot - - ### page entity fields - all page related events should have these. page_id: description: The page ID of the page which properties has been changed. @@ -128,7 +99,6 @@ required: - meta - database - - performer - page_id - page_title - page_namespace diff --git a/test/jsonschema/index.js b/test/jsonschema/index.js index c3d1e07..4b33a85 100644 --- a/test/jsonschema/index.js +++ b/test/jsonschema/index.js @@ -35,7 +35,12 @@ // Go recursively return Object.keys(example) .filter((key) => key !== 'title' && key !== 'description') - .forEach((key) => assert.isSuperSchema(schema[key], example[key], path + '.' + key)); + .forEach((key) => { + if (example.required.indexOf(key) !== -1 + || schema.required.indexOf(key) !== -1) { + assert.isSuperSchema(schema[key], example[key], path + '.' + key) + } + }); } else if (schema !== example) { throw new assert.AssertionError({ message: `Error at path: ${path}`, diff --git a/test/jsonschema/mediawiki/mediawiki_common_schema.yaml b/test/jsonschema/mediawiki/mediawiki_common_schema.yaml index 220837c..08c0418 100644 --- a/test/jsonschema/mediawiki/mediawiki_common_schema.yaml +++ b/test/jsonschema/mediawiki/mediawiki_common_schema.yaml @@ -27,4 +27,3 @@ required: - database - - performer -- To view, visit https://gerrit.wikimedia.org/r/315567 To unsubscribe, visit https://gerrit.wikimedia.org/r/settings Gerrit-MessageType: newchange Gerrit-Change-Id: I139f12ae3f592dcf690617f9dd95a4df828c4a70 Gerrit-PatchSet: 1 Gerrit-Project: mediawiki/event-schemas Gerrit-Branch: master Gerrit-Owner: Ppchelko <ppche...@wikimedia.org> _______________________________________________ MediaWiki-commits mailing list MediaWiki-commits@lists.wikimedia.org https://lists.wikimedia.org/mailman/listinfo/mediawiki-commits