[jbosstools-issues] [JBoss JIRA] (JBIDE-9569) Remove VpeVisualDomBuilder.getNodeForUpdate rudiment method

[Denis Maliarevich (Commented) (JIRA)]

    [ https://issues.jboss.org/browse/JBIDE-9569?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=12638733#comment-12638733 ] 

Denis Maliarevich commented on JBIDE-9569:
------------------------------------------

There is similar situation for <style> tag in updateElement(Node sourceNode) method:
{code}
/*
* special processing of "style" element
* 
* for unification of updating nodes - or redevelop updating
* mechanism (for example : transfer this function to template )
* or redevelop template of "style" element
*/
if (HTML.TAG_STYLE.equalsIgnoreCase(sourceNode.getNodeName())) {
	// refresh style node
	VpeStyleUtil.refreshStyleElement(this, elementMapping);
	return;
}
{code}
                
> Remove VpeVisualDomBuilder.getNodeForUpdate rudiment method
> -----------------------------------------------------------
>
>                 Key: JBIDE-9569
>                 URL: https://issues.jboss.org/browse/JBIDE-9569
>             Project: Tools (JBoss Tools)
>          Issue Type: Bug
>          Components: Visual Page Editor core
>    Affects Versions: 3.3.0.M2
>            Reporter: Maxim Areshkau
>            Assignee: Denis Maliarevich
>            Priority: Optional
>              Labels: refactoring
>             Fix For: 3.3.0.Beta1
>
>         Attachments: method-code-sch.png
>
>
> For updating tables was added method VpeVisualDomBuilder.getNodeForUpdate which checks if element table-cell and if such returns parent. Such functionality implements template, so template for tr and td need to be updated and this rudiment method should be deleted.

--
This message is automatically generated by JIRA.
If you think it was sent incorrectly, please contact your JIRA administrators: https://issues.jboss.org/secure/ContactAdministrators!default.jspa
For more information on JIRA, see: http://www.atlassian.com/software/jira