[client] make image details editable also if user has admin rights or if he does not have any custom permissions, the default permissions allows it

and some code cleanup

1 files changed, 27 insertions, 19 deletions

diff --git a/dozentenmodul/src/main/java/org/openslx/dozmod/gui/window/ImageDetailsWindow.java b/dozentenmodul/src/main/java/org/openslx/dozmod/gui/window/ImageDetailsWindow.java
index 15adc4f3..be2ff02c 100644
--- a/dozentenmodul/src/main/java/org/openslx/dozmod/gui/window/ImageDetailsWindow.java
+++ b/dozentenmodul/src/main/java/org/openslx/dozmod/gui/window/ImageDetailsWindow.java
@@ -46,10 +46,17 @@ public class ImageDetailsWindow extends ImageDetailsWindowLayout {
 
 	public ImageDetailsWindow(Frame modalParent) {
 		super(modalParent);
+		setFocusable(true);
 
-		popupMenuItems[0] = new JMenuItem("Neue Veranstaltung");
-		popupMenuItems[1] = new JMenuItem("Löschen");
-		
+		// ESC closes this window
+		addKeyListener(new KeyAdapter() {
+			@Override
+			public void keyPressed(KeyEvent e) {
+				if (e.getKeyCode() == KeyEvent.VK_ESCAPE) {
+					dispose();
+				}
+			}
+		});
 		// Close button closes window
 		btnClose.addActionListener(new ActionListener() {
 			@Override
@@ -57,15 +64,17 @@ public class ImageDetailsWindow extends ImageDetailsWindowLayout {
 				dispose();
 			}
 		});
-		// ESC closes this window
-		addKeyListener(new KeyAdapter() {
+
+		// nothing to save at first since nothing changed
+		btnSaveChanges.setEnabled(false);
+		btnSaveChanges.addActionListener(new ActionListener() {
 			@Override
-			public void keyPressed(KeyEvent e) {
-				if (e.getKeyCode() == KeyEvent.VK_ESCAPE) {
-					dispose();
-				}
+			public void actionPerformed(ActionEvent e) {
+				LOGGER.debug(("Speichern clicked"));
+				// TODO do save
 			}
 		});
+		// Setup popup menu for the right panel
 		final PopupMenu pop = new PopupMenu(new ActionListener() {
 			public void actionPerformed(ActionEvent e) {
 				if (e.getSource().equals(popupMenuItems[0])) {
@@ -79,30 +88,29 @@ public class ImageDetailsWindow extends ImageDetailsWindowLayout {
 
 			}
 		});
-		// add our menu items to the popup menu
+		// our menu items
+		popupMenuItems[0] = new JMenuItem("Neue Veranstaltung");
+		popupMenuItems[1] = new JMenuItem("Löschen");
+		// add them to the popup menu
 		pop.addMenuItem(popupMenuItems[0]);
 		pop.addSeparator();
 		pop.addMenuItem(popupMenuItems[1]);
 
+		// mouse adapter to register clicks and react on the right panel
 		final MouseAdapter ma = new MouseAdapter() {
 			@Override
 			public void mousePressed(MouseEvent e) {
 				processClick(e);
 			}
-
 			@Override
 			public void mouseReleased(MouseEvent e) {
 				processClick(e);
 			}
-
 			private void processClick(MouseEvent e) {
 				int r = versionTable.rowAtPoint(e.getPoint());
-				LOGGER.debug(e.getSource().getClass() + " @ " + r);
 				if (r >= 0 && r < versionTable.getRowCount()) {
+					// highlight the row and popup the menu
 					versionTable.setRowSelectionInterval(r, r);
-					ImageVersionDetails isr = versionTable.getSelectedItem();
-					ImagePermissions userPermissions = image.getUserPermissions();
-					// open popup
 					if (e.isPopupTrigger()) {
 						pop.show(e.getComponent(), e.getX(), e.getY());
 					}
@@ -111,12 +119,13 @@ public class ImageDetailsWindow extends ImageDetailsWindowLayout {
 				}
 			}
 		};
-
+		// register mouse adapter for the right panel
 		versionTableScrollPane.addMouseListener(ma);
 		versionTable.addMouseListener(ma);
-		setFocusable(true);
+
 	}
 
+
 	/**
 	 * @param imageBaseId the id of the image to be displayed
 	 */
@@ -220,7 +229,6 @@ public class ImageDetailsWindow extends ImageDetailsWindowLayout {
 		btnIsTemplate.setEnabled(editable);
 		cboOperatingSystem.setEnabled(editable);
 		cboShareMode.setEnabled(editable);
-		btnSaveChanges.setEnabled(editable);
 	}
 
 	/**