jenkins-bot has submitted this change and it was merged. Change subject: fingerprint*view widget sanitation ......................................................................
fingerprint*view widget sanitation Change-Id: Iff5401fb8efd3a4a8c391a945e5bee60a7a92ef8 --- M lib/resources/jquery.wikibase/jquery.wikibase.fingerprintgroupview.js M lib/resources/jquery.wikibase/jquery.wikibase.fingerprintlistview.js M lib/resources/jquery.wikibase/jquery.wikibase.fingerprintview.js 3 files changed, 24 insertions(+), 24 deletions(-) Approvals: Adrian Lang: Looks good to me, approved jenkins-bot: Verified diff --git a/lib/resources/jquery.wikibase/jquery.wikibase.fingerprintgroupview.js b/lib/resources/jquery.wikibase/jquery.wikibase.fingerprintgroupview.js index 55b66db..5333009 100644 --- a/lib/resources/jquery.wikibase/jquery.wikibase.fingerprintgroupview.js +++ b/lib/resources/jquery.wikibase/jquery.wikibase.fingerprintgroupview.js @@ -29,7 +29,7 @@ * - {jQuery.Event} * * @event afterstartediting - * - [jQuery.Event} + * - {jQuery.Event} * * @event stopediting * - {jQuery.Event} @@ -122,31 +122,31 @@ prefix = $.wikibase.fingerprintlistview.prototype.widgetEventPrefix; this.$fingerprintlistview - .fingerprintlistview( { - value: this.options.value, - entityId: this.options.entityId, - api: this.options.api - } ) - .on( prefix + 'change', function( event ) { + .on( prefix + 'change.' + this.widgetName, function( event ) { event.stopPropagation(); self._trigger( 'change' ); } ) - .on( prefix + 'toggleerror.' + self.widgetName, function( event, error ) { + .on( prefix + 'toggleerror.' + this.widgetName, function( event, error ) { event.stopPropagation(); self.setError( error ); } ) .on( [ - prefix + 'create.' + self.widgetName, - prefix + 'afterstartediting.' + self.widgetName, - prefix + 'stopediting.' + self.widgetName, - prefix + 'afterstopediting.' + self.widgetName, - prefix + 'disable.' + self.widgetName + prefix + 'create.' + this.widgetName, + prefix + 'afterstartediting.' + this.widgetName, + prefix + 'stopediting.' + this.widgetName, + prefix + 'afterstopediting.' + this.widgetName, + prefix + 'disable.' + this.widgetName ].join( ' ' ), function( event ) { event.stopPropagation(); } - ); + ) + .fingerprintlistview( { + value: this.options.value, + entityId: this.options.entityId, + api: this.options.api + } ); }, /** @@ -286,7 +286,7 @@ enableRemove: false } ); - $fingerprintgroupview.on( 'keyup', function( event ) { + $fingerprintgroupview.on( 'keyup.edittoolbar', function( event ) { if( fingerprintgroupview.option( 'disabled' ) ) { return; } diff --git a/lib/resources/jquery.wikibase/jquery.wikibase.fingerprintlistview.js b/lib/resources/jquery.wikibase/jquery.wikibase.fingerprintlistview.js index a03a32d..bfac4d6 100644 --- a/lib/resources/jquery.wikibase/jquery.wikibase.fingerprintlistview.js +++ b/lib/resources/jquery.wikibase/jquery.wikibase.fingerprintlistview.js @@ -26,7 +26,7 @@ * - {jQuery.Event} * * @event afterstartediting - * - [jQuery.Event} + * - {jQuery.Event} * * @event stopediting * - {jQuery.Event} @@ -98,21 +98,21 @@ // Fully encapsulate child widgets by suppressing their events: this.element - .on( prefix + 'change', function( event ) { + .on( prefix + 'change.' + this.widgetName, function( event ) { event.stopPropagation(); self._trigger( 'change' ); } ) - .on( prefix + 'toggleerror.' + self.widgetName, function( event, error ) { + .on( prefix + 'toggleerror.' + this.widgetName, function( event, error ) { event.stopPropagation(); self.setError( error ); } ) .on( [ - prefix + 'create.' + self.widgetName, - prefix + 'afterstartediting.' + self.widgetName, - prefix + 'stopediting.' + self.widgetName, - prefix + 'afterstopediting.' + self.widgetName, - prefix + 'disable.' + self.widgetName + prefix + 'create.' + this.widgetName, + prefix + 'afterstartediting.' + this.widgetName, + prefix + 'stopediting.' + this.widgetName, + prefix + 'afterstopediting.' + this.widgetName, + prefix + 'disable.' + this.widgetName ].join( ' ' ), function( event ) { event.stopPropagation(); diff --git a/lib/resources/jquery.wikibase/jquery.wikibase.fingerprintview.js b/lib/resources/jquery.wikibase/jquery.wikibase.fingerprintview.js index f156735..760e06d 100644 --- a/lib/resources/jquery.wikibase/jquery.wikibase.fingerprintview.js +++ b/lib/resources/jquery.wikibase/jquery.wikibase.fingerprintview.js @@ -32,7 +32,7 @@ * - {jQuery.Event} * * @event afterstartediting - * - [jQuery.Event} + * - {jQuery.Event} * * @event stopediting * - {jQuery.Event} -- To view, visit https://gerrit.wikimedia.org/r/159988 To unsubscribe, visit https://gerrit.wikimedia.org/r/settings Gerrit-MessageType: merged Gerrit-Change-Id: Iff5401fb8efd3a4a8c391a945e5bee60a7a92ef8 Gerrit-PatchSet: 2 Gerrit-Project: mediawiki/extensions/Wikibase Gerrit-Branch: master Gerrit-Owner: Henning Snater <henning.sna...@wikimedia.de> Gerrit-Reviewer: Adrian Lang <adrian.l...@wikimedia.de> Gerrit-Reviewer: jenkins-bot <> _______________________________________________ MediaWiki-commits mailing list MediaWiki-commits@lists.wikimedia.org https://lists.wikimedia.org/mailman/listinfo/mediawiki-commits