jenkins-bot has submitted this change and it was merged.
Change subject: Revert "Revert "Remove dual-type support for Languages and
Categories""
......................................................................
Revert "Revert "Remove dual-type support for Languages and Categories""
The Parsoid team says the cache is clean of old language /
category types, and would like us to use the new types
for newly generates languages/categories.
This reverts commit d8dcafe25bf3c0ce1fb07b43bf3e8c97e1ba574d.
Change-Id: I0cffd5d299624230f3805f9468b2497f392c12c0
---
M modules/ve-mw/dm/metaitems/ve.dm.MWCategoryMetaItem.js
M modules/ve-mw/dm/metaitems/ve.dm.MWLanguageMetaItem.js
M modules/ve-mw/test/dm/ve.dm.mwExample.js
3 files changed, 20 insertions(+), 34 deletions(-)
Approvals:
Catrope: Looks good to me, approved
jenkins-bot: Verified
diff --git a/modules/ve-mw/dm/metaitems/ve.dm.MWCategoryMetaItem.js
b/modules/ve-mw/dm/metaitems/ve.dm.MWCategoryMetaItem.js
index 5ad9493..648b6cf 100644
--- a/modules/ve-mw/dm/metaitems/ve.dm.MWCategoryMetaItem.js
+++ b/modules/ve-mw/dm/metaitems/ve.dm.MWCategoryMetaItem.js
@@ -30,14 +30,10 @@
ve.dm.MWCategoryMetaItem.static.matchTagNames = [ 'link' ];
-ve.dm.MWCategoryMetaItem.static.matchRdfaTypes = [
- 'mw:WikiLink/Category', // old type, pre-bug 53432
- 'mw:PageProp/Category' // new type
-];
+ve.dm.MWCategoryMetaItem.static.matchRdfaTypes = [ 'mw:PageProp/Category' ];
ve.dm.MWCategoryMetaItem.static.toDataElement = function ( domElements ) {
- var firstDomElement = domElements[0],
- href = firstDomElement.getAttribute( 'href' ),
+ var href = domElements[0].getAttribute( 'href' ),
/*jshint regexp:false */
matches = href.match( /^((?:\.\.?\/)*)(.*?)(?:#(.*))?$/ ),
rawSortkey = matches[3] || '';
@@ -48,8 +44,7 @@
'category': decodeURIComponent( matches[2] ).replace(
/_/g, ' ' ),
'origCategory': matches[2],
'sortkey': decodeURIComponent( rawSortkey ).replace(
/_/g, ' ' ),
- 'origSortkey': rawSortkey,
- 'origRel': firstDomElement.getAttribute( 'rel' )
+ 'origSortkey': rawSortkey
}
};
};
@@ -74,7 +69,7 @@
} else {
category = encodeURIComponent( category );
}
- domElement.setAttribute( 'rel', dataElement.attributes.origRel ||
'mw:WikiLink/Category' );
+ domElement.setAttribute( 'rel', 'mw:PageProp/Category' );
href = hrefPrefix + category;
if ( sortkey !== '' ) {
href += '#' + sortkey;
diff --git a/modules/ve-mw/dm/metaitems/ve.dm.MWLanguageMetaItem.js
b/modules/ve-mw/dm/metaitems/ve.dm.MWLanguageMetaItem.js
index 2c7d074..3fd1190 100644
--- a/modules/ve-mw/dm/metaitems/ve.dm.MWLanguageMetaItem.js
+++ b/modules/ve-mw/dm/metaitems/ve.dm.MWLanguageMetaItem.js
@@ -28,26 +28,21 @@
ve.dm.MWLanguageMetaItem.static.matchTagNames = [ 'link' ];
-ve.dm.MWLanguageMetaItem.static.matchRdfaTypes = [
- 'mw:WikiLink/Language', // old type, pre-bug 53432
- 'mw:PageProp/Language' // new type
-];
+ve.dm.MWLanguageMetaItem.static.matchRdfaTypes = [ 'mw:PageProp/Language' ];
ve.dm.MWLanguageMetaItem.static.toDataElement = function ( domElements ) {
- var firstDomElement = domElements[0],
- href = firstDomElement.getAttribute( 'href' );
+ var href = domElements[0].getAttribute( 'href' );
return {
'type': 'mwLanguage',
'attributes': {
- 'href': href,
- 'origRel': firstDomElement.getAttribute( 'rel' )
+ 'href': href
}
};
};
ve.dm.MWLanguageMetaItem.static.toDomElements = function ( dataElement, doc ) {
var domElement = doc.createElement( 'link' );
- domElement.setAttribute( 'rel', dataElement.attributes.origRel ||
'mw:WikiLink/Language' );
+ domElement.setAttribute( 'rel', 'mw:PageProp/Language' );
domElement.setAttribute( 'href', dataElement.attributes.href );
return [ domElement ];
};
diff --git a/modules/ve-mw/test/dm/ve.dm.mwExample.js
b/modules/ve-mw/test/dm/ve.dm.mwExample.js
index 809a7b1..0bdab61 100644
--- a/modules/ve-mw/test/dm/ve.dm.mwExample.js
+++ b/modules/ve-mw/test/dm/ve.dm.mwExample.js
@@ -24,14 +24,14 @@
'inlineOpenModified': '<span about="#mwt1" typeof="mw:Transclusion"
data-mw="{"id":"mwt1","target":{"wt":"Inline"},"params":{"1":{"wt":"5,678"}}}"
data-parsoid="{"tsr":[18,34],"src":"{{Inline|1,234}}","dsr":[18,34,null,null]}">',
'inlineContent': '$1,234.00',
'inlineClose': '</span>',
- 'mixed': '<link about="#mwt1" rel="mw:WikiLink/Category"
typeof="mw:Transclusion"
data-mw="{"id":"mwt1","target":{"wt":"Inline"},"params":{"1":{"wt":"5,678"}}}"><span
about="#mwt1">Foo</span>',
+ 'mixed': '<link about="#mwt1" rel="mw:PageProp/Category"
typeof="mw:Transclusion"
data-mw="{"id":"mwt1","target":{"wt":"Inline"},"params":{"1":{"wt":"5,678"}}}"><span
about="#mwt1">Foo</span>',
'pairOne': '<p about="#mwt1" typeof="mw:Transclusion"
data-mw="{"params":{"1":{"wt":"foo"}}}"
data-parsoid="1">foo</p>',
'pairTwo': '<p about="#mwt2" typeof="mw:Transclusion"
data-mw="{"params":{"1":{"wt":"foo"}}}"
data-parsoid="2">foo</p>',
'meta':
- '<link rel="mw:WikiLink/Category" href="./Category:Page"
about="#mwt1" typeof="mw:Transclusion" ' +
+ '<link rel="mw:PageProp/Category" href="./Category:Page"
about="#mwt1" typeof="mw:Transclusion" ' +
'data-mw="{"target":{"wt":"Template:Echo","href":"./Template:Echo"},"params":{"1":{"wt":"[[Category:Page]]\\n[[Category:Book]]"}},"i":0}">'
+
'<span about="#mwt1" data-parsoid="{}">\n</span>' +
- '<link rel="mw:WikiLink/Category" href="./Category:Book"
about="#mwt1">'
+ '<link rel="mw:PageProp/Category" href="./Category:Book"
about="#mwt1">'
};
ve.dm.mwExample.MWTransclusion.blockData = {
'type': 'mwTransclusionBlock',
@@ -98,7 +98,7 @@
'htmlAttributes': [
{ 'values': {
'about': '#mwt1',
- 'rel': 'mw:WikiLink/Category',
+ 'rel': 'mw:PageProp/Category',
'typeof': 'mw:Transclusion',
'data-mw':
'{\"id\":\"mwt1\",\"target\":{\"wt\":\"Inline\"},\"params\":{\"1\":{\"wt\":\"5,678\"}}}'
} },
@@ -255,8 +255,7 @@
'category': 'Category:Bar',
'origCategory': 'Category:Bar',
'sortkey': '',
- 'origSortkey': '',
- 'origRel': 'mw:PageProp/Category'
+ 'origSortkey': ''
},
'htmlAttributes': [
{
@@ -313,8 +312,7 @@
'category': 'Category:Foo foo',
'origCategory': 'Category:Foo_foo',
'sortkey': 'Bar baz#quux',
- 'origSortkey': 'Bar baz%23quux',
- 'origRel': 'mw:PageProp/Category'
+ 'origSortkey': 'Bar baz%23quux'
},
'htmlAttributes': [
@@ -388,7 +386,7 @@
'htmlAttributes': [
{
'values': {
- 'rel': 'mw:WikiLink/Category',
+ 'rel': 'mw:PageProp/Category',
'href': './Category:Bar'
},
'computed': {
@@ -441,7 +439,7 @@
'origSortkey': 'Bar baz%23quux'
},
'htmlAttributes': [ { 'values': {
- 'rel': 'mw:WikiLink/Category',
+ 'rel': 'mw:PageProp/Category',
'href': './Category:Foo_foo#Bar baz%23quux'
} } ]
},
@@ -1553,7 +1551,7 @@
},
'whitespace preservation with wrapped comments and language links': {
'body': 'Foo\n' +
- '<link rel="mw:WikiLink/Language"
href="http://de.wikipedia.org/wiki/Foo";>\n' +
+ '<link rel="mw:PageProp/Language"
href="http://de.wikipedia.org/wiki/Foo";>\n' +
'<link rel="mw:PageProp/Language"
href="http://fr.wikipedia.org/wiki/Foo";>',
'data': [
{
@@ -1570,14 +1568,13 @@
{
'type': 'mwLanguage',
'attributes': {
- 'href':
'http://de.wikipedia.org/wiki/Foo',
- 'origRel': 'mw:WikiLink/Language'
+ 'href':
'http://de.wikipedia.org/wiki/Foo'
},
'htmlAttributes': [
{
'values': {
'href':
'http://de.wikipedia.org/wiki/Foo',
- 'rel':
'mw:WikiLink/Language'
+ 'rel':
'mw:PageProp/Language'
},
'computed': {
'href':
'http://de.wikipedia.org/wiki/Foo'
@@ -1590,8 +1587,7 @@
{
'type': 'mwLanguage',
'attributes': {
- 'href':
'http://fr.wikipedia.org/wiki/Foo',
- 'origRel': 'mw:PageProp/Language'
+ 'href':
'http://fr.wikipedia.org/wiki/Foo'
},
'htmlAttributes': [
{
--
To view, visit https://gerrit.wikimedia.org/r/99734
To unsubscribe, visit https://gerrit.wikimedia.org/r/settings
Gerrit-MessageType: merged
Gerrit-Change-Id: I0cffd5d299624230f3805f9468b2497f392c12c0
Gerrit-PatchSet: 1
Gerrit-Project: mediawiki/extensions/VisualEditor
Gerrit-Branch: master
Gerrit-Owner: Catrope <[email protected]>
Gerrit-Reviewer: Catrope <[email protected]>
Gerrit-Reviewer: GWicke <[email protected]>
Gerrit-Reviewer: jenkins-bot
_______________________________________________
MediaWiki-commits mailing list
[email protected]
https://lists.wikimedia.org/mailman/listinfo/mediawiki-commits
