#91 Reorder actions available only for persisted object. Reorder undo reflector fix.

plugins/org.jkiss.dbeaver.core/src/org/jkiss/dbeaver/ui/controls/itemlist/ItemListControl.java

@@ -103,7 +103,7 @@ public class ItemListControl extends NodeListControl
             contributionManager.add(ActionUtils.makeCommandContribution(workbenchSite, IWorkbenchCommandConstants.FILE_REFRESH));
         }
 
-        if (rootNode instanceof DBNDatabaseNode) {
+        if (rootNode instanceof DBNDatabaseNode && rootNode.isPersisted()) {
             boolean hasReorder = false;
             List<Class<?>> childrenTypes = ((DBNDatabaseNode) rootNode).getChildrenTypes(null);
             for (Class<?> chilType : childrenTypes) {

plugins/org.jkiss.dbeaver.model/src/org/jkiss/dbeaver/model/impl/sql/edit/SQLObjectEditor.java

@@ -143,8 +143,10 @@ public abstract class SQLObjectEditor<OBJECT_TYPE extends DBSObject, CONTAINER_T
 
     protected void addObjectReorderActions(List<DBEPersistAction> actions, ObjectReorderCommand command)
     {
-        // Not supported by implementation
-        throw new IllegalStateException("Object reorder is not supported in " + getClass().getSimpleName()); //$NON-NLS-1$
+        if (command.getObject().isPersisted()) {
+            // Not supported by implementation
+            throw new IllegalStateException("Object reorder is not supported in " + getClass().getSimpleName()); //$NON-NLS-1$
+        }
     }
 
     protected abstract void addObjectDeleteActions(List<DBEPersistAction> actions, ObjectDeleteCommand command);
@@ -473,7 +475,11 @@ public abstract class SQLObjectEditor<OBJECT_TYPE extends DBSObject, CONTAINER_T
         public void undoCommand(ObjectReorderCommand command)
         {
             ((DBPOrderedObject)command.getObject()).setOrdinalPosition(command.oldPosition);
-            DBUtils.fireObjectUpdate(command.getObject());
+            final DBSObject parentObject = command.getObject().getParentObject();
+            if (parentObject != null) {
+                // We need to update order in navigator model
+                DBUtils.fireObjectUpdate(parentObject, DBPEvent.REORDER);
+            }
         }
 
     }