[jbosstools-issues] [JBoss JIRA] Resolved: (JBIDE-6439) Refactor and partially reimpement VpeDnD class

[Yahor Radtsevich (JIRA)]

     [ https://jira.jboss.org/browse/JBIDE-6439?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ]

Yahor Radtsevich resolved JBIDE-6439.
-------------------------------------

    Resolution: Done


RESOLVED
- InnerDragBuffer is replaced by 'vpe/xpath' flavor
- 'text/unicode' and 'text/html' flavors now contain source of the selected node
- references to VpeDnD are removed from VpeVisualDomBuilder
- some minor changes

> Refactor and partially reimpement VpeDnD class
> ----------------------------------------------
>
>                 Key: JBIDE-6439
>                 URL: https://jira.jboss.org/browse/JBIDE-6439
>             Project: Tools (JBoss Tools)
>          Issue Type: Task
>          Components: Visual Page Editor core
>            Reporter: Yahor Radtsevich
>            Assignee: Yahor Radtsevich
>            Priority: Critical
>             Fix For: 3.2.0.Beta
>
>
> org.jboss.tools.vpe.dnd.VpeDnD class needs to be refactored and reimplemented for some parts.
> It has the following weak points:
> 1. 'sourceInnerDragInfo' variable is not always reflects actual state of the 'VpeDnD' objects. It is used to DnD elements inside VPE only, in other cases, e.g. for external DnD it is ignored. It is error-prone and leads to many if-else structures.
> 2. 'selectionChanged' method has no interface and 'VpeDnD' class is tightly coupled with 'VpeVisualDomBuilder' class.
> 3. InnerDragBuffer class with static fields is used to pass transfer data.

-- 
This message is automatically generated by JIRA.
-
If you think it was sent incorrectly contact one of the administrators: https://jira.jboss.org/secure/Administrators.jspa
-
For more information on JIRA, see: http://www.atlassian.com/software/jira