Opened 12 years ago

Closed 12 years ago

#8857 closed Bug (fixed)

wrong variable "name" used instead of "tagName"

Reported by: Alfonso Martínez de Lizarrondo Owned by: Alfonso Martínez de Lizarrondo
Priority: Normal Milestone: CKEditor 3.6.4
Component: Core : Parser Version:
Keywords: Cc:

Description

In the core/htmlparser/element.js file, this block

	var tagName = attributes[ 'data-cke-real-element-type' ] || name || '';

	// Reveal the real semantic of our internal custom tag name (#6639).
	var internalTag = tagName.match( /^cke:(.*)/ );
  	internalTag && ( tagName = internalTag[ 1 ] );

	var dtd			= CKEDITOR.dtd,
		isBlockLike	= !!( dtd.$nonBodyContent[ tagName ]
				|| dtd.$block[ tagName ]
				|| dtd.$listItem[ tagName ]
				|| dtd.$tableContent[ tagName ]
				|| dtd.$nonEditable[ tagName ]
				|| tagName == 'br' ),
		isEmpty = !!dtd.$empty[ name ];

	this.isEmpty	= isEmpty;
	this.isUnknown	= !dtd[ name ];

I think that the "name" in lines 50 and 53 should be tagName as all the rest of uses, but instead of applying it as a microchange I would prefer if someone could confirm it.

Attachments (1)

8857.patch (557 bytes) - added by Alfonso Martínez de Lizarrondo 12 years ago.
Proposed patch

Download all attachments as: .zip

Change History (4)

Changed 12 years ago by Alfonso Martínez de Lizarrondo

Attachment: 8857.patch added

Proposed patch

comment:1 Changed 12 years ago by Alfonso Martínez de Lizarrondo

Owner: set to Alfonso Martínez de Lizarrondo
Status: newreview

comment:2 Changed 12 years ago by Garry Yao

Milestone: CKEditor 3.6.4
Status: reviewreview_passed

comment:3 Changed 12 years ago by Garry Yao

Resolution: fixed
Status: review_passedclosed

Fixed with [7542].

Note: See TracTickets for help on using tickets.
© 2003 – 2022, CKSource sp. z o.o. sp.k. All rights reserved. | Terms of use | Privacy policy