/*\r
-Copyright (c) 2003-2010, CKSource - Frederico Knabben. All rights reserved.\r
+Copyright (c) 2003-2012, CKSource - Frederico Knabben. All rights reserved.\r
For licensing, see LICENSE.html or http://ckeditor.com/license\r
*/\r
\r
typeof extendEmptyBlock == 'function' && ( extendEmptyBlock( block ) === false ) ) )\r
return false;\r
\r
- // 1. For IE version >=8, empty blocks are displayed correctly themself in wysiwiyg;\r
- // 2. For the rest, at least table cell and list item need no filler space.\r
- // (#6248)\r
- if ( fromSource && CKEDITOR.env.ie &&\r
- ( document.documentMode > 7\r
- || block.name in CKEDITOR.dtd.tr\r
- || block.name in CKEDITOR.dtd.$listItem ) )\r
- return false;\r
+ // 1. For IE version >=8, empty blocks are displayed correctly themself in wysiwiyg;\r
+ // 2. For the rest, at least table cell and list item need no filler space.\r
+ // (#6248)\r
+ if ( fromSource && CKEDITOR.env.ie &&\r
+ ( document.documentMode > 7\r
+ || block.name in CKEDITOR.dtd.tr\r
+ || block.name in CKEDITOR.dtd.$listItem ) )\r
+ return false;\r
\r
var lastChild = lastNoneSpaceChild( block );\r
\r
\r
var dtd = CKEDITOR.dtd;\r
\r
+ // Define orders of table elements.\r
+ var tableOrder = [ 'caption', 'colgroup', 'col', 'thead', 'tfoot', 'tbody' ];\r
+\r
// Find out the list of block-like tags that can contain <br>.\r
var blockLikeTags = CKEDITOR.tools.extend( {}, dtd.$block, dtd.$listItem, dtd.$tableContent );\r
for ( var i in blockLikeTags )\r
// Attributes saved for changes and protected attributes.\r
[ ( /^data-cke-(saved|pa)-/ ), '' ],\r
\r
- // All "data-cke" attributes are to be ignored.\r
- [ ( /^data-cke.*/ ), '' ],\r
+ // All "data-cke-" attributes are to be ignored.\r
+ [ ( /^data-cke-.*/ ), '' ],\r
\r
[ 'hidefocus', '' ]\r
],\r
return element;\r
},\r
\r
+ // The contents of table should be in correct order (#4809).\r
+ table : function( element )\r
+ {\r
+ var children = element.children;\r
+ children.sort( function ( node1, node2 )\r
+ {\r
+ return node1.type == CKEDITOR.NODE_ELEMENT && node2.type == node1.type ?\r
+ CKEDITOR.tools.indexOf( tableOrder, node1.name ) > CKEDITOR.tools.indexOf( tableOrder, node2.name ) ? 1 : -1 : 0;\r
+ } );\r
+ },\r
+\r
embed : function( element )\r
{\r
var parent = element.parent;\r
delete element.name;\r
},\r
\r
+ // Empty <pre> in IE is reported with filler node ( ).\r
+ pre : function( element ) { CKEDITOR.env.ie && trimFillers( element ); },\r
+\r
html : function( element )\r
{\r
delete element.attributes.contenteditable;\r
// Remove all class names starting with "cke_".\r
return CKEDITOR.tools.ltrim( value.replace( /(?:^|\s+)cke_[^\s]*/g, '' ) ) || false;\r
}\r
- },\r
-\r
- comment : function( contents )\r
- {\r
- // If this is a comment for protected source.\r
- if ( contents.substr( 0, protectedSourceMarker.length ) == protectedSourceMarker )\r
- {\r
- // Remove the extra marker for real comments from it.\r
- if ( contents.substr( protectedSourceMarker.length, 3 ) == '{C}' )\r
- contents = contents.substr( protectedSourceMarker.length + 3 );\r
- else\r
- contents = contents.substr( protectedSourceMarker.length );\r
-\r
- return new CKEDITOR.htmlParser.cdata( decodeURIComponent( contents ) );\r
- }\r
-\r
- return contents;\r
}\r
};\r
\r
if ( CKEDITOR.env.ie )\r
{\r
// IE outputs style attribute in capital letters. We should convert\r
- // them back to lower case.\r
+ // them back to lower case, while not hurting the values (#5930)\r
defaultHtmlFilterRules.attributes.style = function( value, element )\r
{\r
- return value.toLowerCase();\r
+ return value.replace( /(^|;)([^\:]+)/g, function( match )\r
+ {\r
+ return match.toLowerCase();\r
+ });\r
};\r
}\r
\r
function protectReadOnly( element )\r
{\r
- element.attributes.contenteditable = "false";\r
+ var attrs = element.attributes;\r
+\r
+ // We should flag that the element was locked by our code so\r
+ // it'll be editable by the editor functions (#6046).\r
+ if ( attrs.contenteditable != "false" )\r
+ attrs[ 'data-cke-editable' ] = attrs.contenteditable ? 'true' : 1;\r
+\r
+ attrs.contenteditable = "false";\r
}\r
function unprotectReadyOnly( element )\r
{\r
- delete element.attributes.contenteditable;\r
+ var attrs = element.attributes;\r
+ switch( attrs[ 'data-cke-editable' ] )\r
+ {\r
+ case 'true': attrs.contenteditable = 'true'; break;\r
+ case '1': delete attrs.contenteditable; break;\r
+ }\r
}\r
// Disable form elements editing mode provided by some browers. (#5746)\r
for ( i in { input : 1, textarea : 1 } )\r
defaultHtmlFilterRules.elements[ i ] = unprotectReadyOnly;\r
}\r
\r
- var protectAttributeRegex = /<((?:a|area|img|input)\b[\s\S]*?\s)((href|src|name)\s*=\s*(?:(?:"[^"]*")|(?:'[^']*')|(?:[^ "'>]+)))([^>]*)>/gi,\r
- findSavedSrcRegex = /\sdata-cke-saved-src\s*=/;\r
+ var protectElementRegex = /<(a|area|img|input)\b([^>]*)>/gi,\r
+ protectAttributeRegex = /\b(on\w+|href|src|name)\s*=\s*(?:(?:"[^"]*")|(?:'[^']*')|(?:[^ "'>]+))/gi;\r
\r
var protectElementsRegex = /(?:<style(?=[ >])[^>]*>[\s\S]*<\/style>)|(?:<(:?link|meta|base)[^>]*>)/gi,\r
encodedElementsRegex = /<cke:encoded>([^<]*)<\/cke:encoded>/gi;\r
\r
function protectAttributes( html )\r
{\r
- return html.replace( protectAttributeRegex, function( tag, beginning, fullAttr, attrName, end )\r
+ return html.replace( protectElementRegex, function( element, tag, attributes )\r
+ {\r
+ return '<' + tag + attributes.replace( protectAttributeRegex, function( fullAttr, attrName )\r
{\r
- // We should not rewrite the _cke_saved_src attribute (#5218)\r
- if ( attrName == 'src' && findSavedSrcRegex.test( tag ) )\r
- return tag;\r
- else\r
- return '<' + beginning + fullAttr + ' data-cke-saved-' + fullAttr + end + '>';\r
- });\r
+ // Avoid corrupting the inline event attributes (#7243).\r
+ // We should not rewrite the existed protected attributes, e.g. clipboard content from editor. (#5218)\r
+ if ( !( /^on/ ).test( attrName ) && attributes.indexOf( 'data-cke-saved-' + attrName ) == -1 )\r
+ return ' data-cke-saved-' + fullAttr + ' data-cke-' + CKEDITOR.rnd + '-' + fullAttr;\r
+\r
+ return fullAttr;\r
+ }) + '>';\r
+ });\r
}\r
\r
function protectElements( html )\r
});\r
}\r
\r
- function protectSource( data, protectRegexes )\r
+ function unprotectSource( html, editor )\r
+ {\r
+ var store = editor._.dataStore;\r
+\r
+ return html.replace( /<!--\{cke_protected\}([\s\S]+?)-->/g, function( match, data )\r
+ {\r
+ return decodeURIComponent( data );\r
+ }).replace( /\{cke_protected_(\d+)\}/g, function( match, id )\r
+ {\r
+ return store && store[ id ] || '';\r
+ });\r
+ }\r
+\r
+ function protectSource( data, editor )\r
{\r
var protectedHtml = [],\r
+ protectRegexes = editor.config.protectedSource,\r
+ store = editor._.dataStore || ( editor._.dataStore = { id : 1 } ),\r
tempRegex = /<\!--\{cke_temp(comment)?\}(\d*?)-->/g;\r
\r
var regexes =\r
return protectedHtml[ id ];\r
}\r
);\r
- return '<!--{cke_temp}' + ( protectedHtml.push( match ) - 1 ) + '-->';\r
+\r
+ // Avoid protecting over protected, e.g. /\{.*?\}/\r
+ return ( /cke_temp(comment)?/ ).test( match ) ? match\r
+ : '<!--{cke_temp}' + ( protectedHtml.push( match ) - 1 ) + '-->';\r
});\r
}\r
data = data.replace( tempRegex, function( $, isComment, id )\r
'-->';\r
}\r
);\r
- return data;\r
+\r
+ // Different protection pattern is used for those that\r
+ // live in attributes to avoid from being HTML encoded.\r
+ return data.replace( /(['"]).*?\1/g, function ( match )\r
+ {\r
+ return match.replace( /<!--\{cke_protected\}([\s\S]+?)-->/g, function( match, data )\r
+ {\r
+ store[ store.id ] = decodeURIComponent( data );\r
+ return '{cke_protected_'+ ( store.id++ ) + '}';\r
+ });\r
+ });\r
}\r
\r
CKEDITOR.plugins.add( 'htmldataprocessor',\r
// The source data is already HTML, but we need to clean\r
// it up and apply the filter.\r
\r
- data = protectSource( data, this.editor.config.protectedSource );\r
+ data = protectSource( data, this.editor );\r
\r
// Before anything, we must protect the URL attributes as the\r
// browser may changing them when setting the innerHTML later in\r
// Call the browser to help us fixing a possibly invalid HTML\r
// structure.\r
var div = new CKEDITOR.dom.element( 'div' );\r
+\r
// Add fake character to workaround IE comments bug. (#3801)\r
div.setHtml( 'a' + data );\r
data = div.getHtml().substr( 1 );\r
\r
+ // Restore shortly protected attribute names.\r
+ data = data.replace( new RegExp( ' data-cke-' + CKEDITOR.rnd + '-', 'ig' ), ' ' );\r
+\r
// Unprotect "some" of the protected elements at this point.\r
data = unprotectElementNames( data );\r
\r
\r
fragment.writeHtml( writer, this.htmlFilter );\r
\r
- return writer.getHtml( true );\r
+ var data = writer.getHtml( true );\r
+\r
+ // Restore those non-HTML protected source. (#4475,#4880)\r
+ data = unprotectRealComments( data );\r
+ data = unprotectSource( data, this.editor );\r
+\r
+ return data;\r
}\r
};\r
})();\r
Got questions, comments, patches, etc.? Contact Jason Woofenden