jenkins-bot has submitted this change and it was merged. ( 
https://gerrit.wikimedia.org/r/351527 )

Change subject: Follow-up 57a06a6: Use the same attribute name on read and on 
write
......................................................................


Follow-up 57a06a6: Use the same attribute name on read and on write

Bug: T164054
Change-Id: I6770f6c91248fcd9ab39322413874e20a1b65552
(cherry picked from commit 486bbd76090919306f5cadbae0bd85eefc8500f1)
---
M modules/ve-mw/dm/metaitems/ve.dm.MWFlaggedMetaItem.js
1 file changed, 8 insertions(+), 8 deletions(-)

Approvals:
  Catrope: Looks good to me, approved
  jenkins-bot: Verified



diff --git a/modules/ve-mw/dm/metaitems/ve.dm.MWFlaggedMetaItem.js 
b/modules/ve-mw/dm/metaitems/ve.dm.MWFlaggedMetaItem.js
index d04011e..794b3e0 100644
--- a/modules/ve-mw/dm/metaitems/ve.dm.MWFlaggedMetaItem.js
+++ b/modules/ve-mw/dm/metaitems/ve.dm.MWFlaggedMetaItem.js
@@ -30,26 +30,26 @@
 ve.dm.MWFlaggedMetaItem.static.matchTagNames = [ 'meta' ];
 
 ve.dm.MWFlaggedMetaItem.static.toDataElement = function ( domElements ) {
-       var type = domElements[ 0 ].getAttribute( 'property' );
+       var property = domElements[ 0 ].getAttribute( 'property' );
 
-       if ( !type || this.matchRdfaTypes.indexOf( type ) === -1 ) {
+       if ( !property || this.matchRdfaTypes.indexOf( property ) === -1 ) {
                // Fallback to first match if somehow unset
-               type = this.matchRdfaTypes[ 0 ];
+               property = this.matchRdfaTypes[ 0 ];
        }
 
-       return { type: this.name, attributes: { type: type } };
+       return { type: this.name, attributes: { property: property } };
 };
 
 ve.dm.MWFlaggedMetaItem.static.toDomElements = function ( dataElement, doc ) {
        var meta = doc.createElement( 'meta' ),
-               type = OO.getProp( dataElement, 'attributes', 'property' );
+               property = OO.getProp( dataElement, 'attributes', 'property' );
 
-       if ( !type || this.matchRdfaTypes.indexOf( type ) === -1 ) {
+       if ( !property || this.matchRdfaTypes.indexOf( property ) === -1 ) {
                // Fallback to first item if somehow unset
-               type = this.matchRdfaTypes[ 0 ];
+               property = this.matchRdfaTypes[ 0 ];
        }
 
-       meta.setAttribute( 'property', type );
+       meta.setAttribute( 'property', property );
 
        return [ meta ];
 };

-- 
To view, visit https://gerrit.wikimedia.org/r/351527
To unsubscribe, visit https://gerrit.wikimedia.org/r/settings

Gerrit-MessageType: merged
Gerrit-Change-Id: I6770f6c91248fcd9ab39322413874e20a1b65552
Gerrit-PatchSet: 1
Gerrit-Project: mediawiki/extensions/VisualEditor
Gerrit-Branch: wmf/1.29.0-wmf.21
Gerrit-Owner: Jforrester <jforres...@wikimedia.org>
Gerrit-Reviewer: Catrope <r...@wikimedia.org>
Gerrit-Reviewer: jenkins-bot <>

_______________________________________________
MediaWiki-commits mailing list
MediaWiki-commits@lists.wikimedia.org
https://lists.wikimedia.org/mailman/listinfo/mediawiki-commits

Reply via email to