diff options
author | Automerge <automerge@vaadin.com> | 2012-02-15 02:18:42 +0000 |
---|---|---|
committer | Automerge <automerge@vaadin.com> | 2012-02-15 02:18:42 +0000 |
commit | 5b2e12623424e3603a5983895d122ba333db5b18 (patch) | |
tree | 9c4d652d612fbf5e21708238b817e728a918530c | |
parent | b495b8f7f4d4cf1ea1731b45d1cf6e35bdd86779 (diff) | |
download | vaadin-framework-5b2e12623424e3603a5983895d122ba333db5b18.tar.gz vaadin-framework-5b2e12623424e3603a5983895d122ba333db5b18.zip |
[merge from 6.7] #8373 Handler DragAndDropService changeVariables error by finding the drop target and using that for error reporting
svn changeset:23013/svn branch:6.8
-rw-r--r-- | src/com/vaadin/terminal/gwt/server/AbstractCommunicationManager.java | 12 |
1 files changed, 8 insertions, 4 deletions
diff --git a/src/com/vaadin/terminal/gwt/server/AbstractCommunicationManager.java b/src/com/vaadin/terminal/gwt/server/AbstractCommunicationManager.java index e96f2d2b56..4454287ae9 100644 --- a/src/com/vaadin/terminal/gwt/server/AbstractCommunicationManager.java +++ b/src/com/vaadin/terminal/gwt/server/AbstractCommunicationManager.java @@ -1396,12 +1396,16 @@ public abstract class AbstractCommunicationManager implements } } } catch (Exception e) { + Component errorComponent = null; if (owner instanceof Component) { - handleChangeVariablesError(app, (Component) owner, e, m); - } else { - // TODO DragDropService error handling - throw new RuntimeException(e); + errorComponent = (Component) owner; + } else if (owner instanceof DragAndDropService) { + if (m.get("dhowner") instanceof Component) { + errorComponent = (Component) m.get("dhowner"); + } } + + handleChangeVariablesError(app, errorComponent, e, m); } } else { |