Santhosh has uploaded a new change for review.

  https://gerrit.wikimedia.org/r/200126

Change subject: Templates: Remove compount content blocks from several 
transclusions
......................................................................

Templates: Remove compount content blocks from several transclusions

An example: The infobox at enwiki:California
At present, we cannot provide edit support to them.
So remove it from source.

Also see
See 
http://www.mediawiki.org/wiki/Parsoid/MediaWiki_DOM_spec#Transclusion_content

Change-Id: I9a98379bc8bf365cb996a64e48de423d29a0721a
---
M modules/source/ext.cx.source.filter.js
1 file changed, 11 insertions(+), 5 deletions(-)


  git pull 
ssh://gerrit.wikimedia.org:29418/mediawiki/extensions/ContentTranslation 
refs/changes/26/200126/1

diff --git a/modules/source/ext.cx.source.filter.js 
b/modules/source/ext.cx.source.filter.js
index d0bdecc..a0056ff 100644
--- a/modules/source/ext.cx.source.filter.js
+++ b/modules/source/ext.cx.source.filter.js
@@ -81,14 +81,20 @@
 
                        mwData = $template.data( 'mw' );
 
-                       if ( !mwData || mwData.parts.length > 1 ) {
-                               // Either the template is missing mw data or 
having multiple parts.
-                               // At present, we cannot handle them.
+                       if ( !mwData ) {
+                               mw.log( '[CX] Skipping template!' );
+                               return;
+                       }
+
+                       if ( mwData.parts.length > 1 ) {
+                               // This is compound content blocks that include 
output from several transclusions
+                               // At present, we cannot provide edit support 
to them.
+                               // See 
http://www.mediawiki.org/wiki/Parsoid/MediaWiki_DOM_spec#Transclusion_content
                                // An example:
                                // {{Version |o |1.1}}{{efn-ua |Due to an 
incident ...<ref name="releases" />}}
                                // in enwiki:Debian, Timeline table.
-                               mw.log( '[CX] Skipping template!' );
-
+                               mw.log( '[CX] Removing multi part template' );
+                               sourceFilter.removeTemplate( $template );
                                return;
                        }
 

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

Gerrit-MessageType: newchange
Gerrit-Change-Id: I9a98379bc8bf365cb996a64e48de423d29a0721a
Gerrit-PatchSet: 1
Gerrit-Project: mediawiki/extensions/ContentTranslation
Gerrit-Branch: master
Gerrit-Owner: Santhosh <santhosh.thottin...@gmail.com>

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

Reply via email to