Don't allow Cut, Move to Trash, Delete options for the home folder. This solves issue 76.

[pithos] / src / gr / ebs / gss / client / FilePropertiesDialog.java

diff --git a/src/gr/ebs/gss/client/FilePropertiesDialog.java b/src/gr/ebs/gss/client/FilePropertiesDialog.java
index 4b09700..1a47930 100644 (file)
--- a/src/gr/ebs/gss/client/FilePropertiesDialog.java
+++ b/src/gr/ebs/gss/client/FilePropertiesDialog.java
@@ -29,8 +29,12 @@ import java.util.List;
 import java.util.Set;\r
 \r
 import com.google.gwt.core.client.GWT;\r
+import com.google.gwt.event.dom.client.ChangeEvent;\r
+import com.google.gwt.event.dom.client.ChangeHandler;\r
 import com.google.gwt.event.dom.client.ClickEvent;\r
 import com.google.gwt.event.dom.client.ClickHandler;\r
+import com.google.gwt.event.dom.client.KeyDownEvent;\r
+import com.google.gwt.event.dom.client.KeyDownHandler;\r
 import com.google.gwt.i18n.client.DateTimeFormat;\r
 import com.google.gwt.json.client.JSONArray;\r
 import com.google.gwt.json.client.JSONBoolean;\r
@@ -111,8 +115,9 @@ public class FilePropertiesDialog extends AbstractPropertiesDialog {
                setText("File properties");\r
 \r
                file = (FileResource) GSS.get().getCurrentSelection();\r
-               permList = new PermissionsList(images, file.getPermissions(), file.getOwner());\r
                userFullName = _userFullName;\r
+               permList = new PermissionsList(images, file.getPermissions(), file.getOwner());\r
+\r
 \r
                // Outer contains inner and buttons.\r
                final VerticalPanel outer = new VerticalPanel();\r
@@ -131,11 +136,16 @@ public class FilePropertiesDialog extends AbstractPropertiesDialog {
                final HorizontalPanel vPanel2 = new HorizontalPanel();\r
 \r
                versioned.setValue(file.isVersioned());\r
+               versioned.getElement().setId("filePropertiesDialog.chechBox.versioned");\r
                inner.add(generalPanel, "General");\r
                inner.add(permPanel, "Sharing");\r
                inner.add(verPanel, "Versions");\r
                inner.selectTab(0);\r
 \r
+               final Label fileNameNote = new Label("Please note that slashes ('/') are not allowed in file names.", true);\r
+               fileNameNote.setVisible(false);\r
+               fileNameNote.setStylePrimaryName("gss-readForAllNote");\r
+\r
                final FlexTable generalTable = new FlexTable();\r
                generalTable.setText(0, 0, "Name");\r
                generalTable.setText(1, 0, "Folder");\r
@@ -144,11 +154,25 @@ public class FilePropertiesDialog extends AbstractPropertiesDialog {
                generalTable.setText(4, 0, "Tags");\r
                name.setWidth("100%");\r
                name.setText(file.getName());\r
+               name.getElement().setId("filePropertiesDialog.textBox.name");\r
                generalTable.setWidget(0, 1, name);\r
+               name.addChangeHandler(new ChangeHandler() {\r
+\r
+                       @Override\r
+                       public void onChange(ChangeEvent event) {\r
+                               if(name.getText().contains("/"))\r
+                                       fileNameNote.setVisible(true);\r
+                               else\r
+                                       fileNameNote.setVisible(false);\r
+\r
+                       }\r
+               });\r
+\r
                if(file.getFolderName() != null)\r
                        generalTable.setText(1, 1, file.getFolderName());\r
                else\r
                        generalTable.setText(1, 1, "-");\r
+               generalTable.setWidget(0, 2, fileNameNote);\r
                generalTable.setText(2, 1,userFullName);\r
 \r
                final DateTimeFormat formatter = DateTimeFormat.getFormat("d/M/yyyy h:mm a");\r
@@ -164,6 +188,7 @@ public class FilePropertiesDialog extends AbstractPropertiesDialog {
                        tagsBuffer.delete(tagsBuffer.length() - 2, tagsBuffer.length() - 1);\r
                initialTagText = tagsBuffer.toString();\r
                tags.setWidth("100%");\r
+               tags.getElement().setId("filePropertiesDialog.textBox.tags");\r
                tags.setText(initialTagText);\r
                generalTable.setWidget(4, 1, tags);\r
                generalTable.getFlexCellFormatter().setStyleName(0, 0, "props-labels");\r
@@ -183,10 +208,16 @@ public class FilePropertiesDialog extends AbstractPropertiesDialog {
                final Button ok = new Button("OK", new ClickHandler() {\r
                        @Override\r
                        public void onClick(ClickEvent event) {\r
-                               accept();\r
-                               closeDialog();\r
+                               if(name.getText().contains("/"))\r
+                                       fileNameNote.setVisible(true);\r
+                               else{\r
+                                       fileNameNote.setVisible(true);\r
+                                       accept();\r
+                                       closeDialog();\r
+                               }               \r
                        }\r
                });\r
+               ok.getElement().setId("filePropertiesDialog.button.ok");                \r
                buttons.add(ok);\r
                buttons.setCellHorizontalAlignment(ok, HasHorizontalAlignment.ALIGN_CENTER);\r
                // Create the 'Cancel' button, along with a listener that hides the\r
@@ -197,6 +228,7 @@ public class FilePropertiesDialog extends AbstractPropertiesDialog {
                                closeDialog();\r
                        }\r
                });\r
+               cancel.getElement().setId("filePropertiesDialog.button.cancel");\r
                buttons.add(cancel);\r
                buttons.setCellHorizontalAlignment(cancel, HasHorizontalAlignment.ALIGN_CENTER);\r
                buttons.setSpacing(8);\r
@@ -227,6 +259,7 @@ public class FilePropertiesDialog extends AbstractPropertiesDialog {
                                dlg.center();\r
                        }\r
                });\r
+               add.getElement().setId("filePropertiesDialog.button.addGroup");\r
                permButtons.add(add);\r
                permButtons.setCellHorizontalAlignment(add, HasHorizontalAlignment.ALIGN_CENTER);\r
 \r
@@ -237,6 +270,7 @@ public class FilePropertiesDialog extends AbstractPropertiesDialog {
                                dlg.center();\r
                        }\r
                });\r
+               add.getElement().setId("filePropertiesDialog.button.addUser");\r
                permButtons.add(addUser);\r
                permButtons.setCellHorizontalAlignment(addUser, HasHorizontalAlignment.ALIGN_CENTER);\r
 \r
@@ -251,6 +285,7 @@ public class FilePropertiesDialog extends AbstractPropertiesDialog {
                readForAllNote.setStylePrimaryName("gss-readForAllNote");\r
 \r
                readForAll = new CheckBox();\r
+               readForAll.getElement().setId("filePropertiesDialog.checkBox.public");\r
                readForAll.setValue(file.isReadForAll());\r
                readForAll.addClickHandler(new ClickHandler() {\r
                        @Override\r
@@ -284,6 +319,7 @@ public class FilePropertiesDialog extends AbstractPropertiesDialog {
 \r
                });\r
                path.setText(file.getUri());\r
+               path.getElement().setId("filePropertiesDialog.textBox.link");\r
                path.setTitle("Use this link for sharing the file via e-mail, IM, etc. (crtl-C/cmd-C to copy to system clipboard)");\r
                path.setWidth("100%");\r
                path.setReadOnly(true);\r
@@ -497,10 +533,5 @@ public class FilePropertiesDialog extends AbstractPropertiesDialog {
                };\r
                DeferredCommand.addCommand(cf);\r
        }\r
-       private boolean hasOwner(String _userName) {\r
-               if (GSS.get().findUserFullName(_userName) == null)\r
-                       return false;\r
-               return true;\r
-       }\r
 \r
 }\r