Server Disk view issue fix

3 files changed, 14 insertions, 13 deletions

diff --git a/src/com.gluster.storage.management.gui/src/com/gluster/storage/management/gui/ServerDiskTableLabelProvider.java b/src/com.gluster.storage.management.gui/src/com/gluster/storage/management/gui/ServerDiskTableLabelProvider.java
index 57a7b312..714cdf1b 100644
--- a/src/com.gluster.storage.management.gui/src/com/gluster/storage/management/gui/ServerDiskTableLabelProvider.java
+++ b/src/com.gluster.storage.management.gui/src/com/gluster/storage/management/gui/ServerDiskTableLabelProvider.java
@@ -59,17 +59,18 @@ public class ServerDiskTableLabelProvider extends TableLabelProviderAdapter {
 		return null;
 	}
 
-	private String getDiskSpaceInUse(Disk disk) {
-		if(disk.hasErrors() || disk.isUninitialized()) {
-			return CoreConstants.NA;
+
+	private String getDiskFreeSpace(Disk disk) {
+		if (disk.hasErrors() || disk.isUninitialized()) {
+			return "NA";
 		} else {
-			return NumberUtil.formatNumber(disk.getSpaceInUse());
+			return NumberUtil.formatNumber(disk.getFreeSpace());
 		}
 	}
 	
-	private String getDiskSpace(Disk disk) {
-		if(disk.hasErrors() || disk.isUninitialized()) {
-			return CoreConstants.NA;
+	private String getTotalDiskSpace(Disk disk) {
+		if (disk.hasErrors() || disk.isUninitialized()) {
+			return "NA";
 		} else {
 			return NumberUtil.formatNumber(disk.getSpace());
 		}
@@ -83,8 +84,8 @@ public class ServerDiskTableLabelProvider extends TableLabelProviderAdapter {
 
 		Disk disk = (Disk) element;
 		String columnText = (columnIndex == SERVER_DISK_TABLE_COLUMN_INDICES.DISK.ordinal() ? disk.getName()
-			: columnIndex == SERVER_DISK_TABLE_COLUMN_INDICES.SPACE.ordinal() ? getDiskSpace(disk) 
-			: columnIndex == SERVER_DISK_TABLE_COLUMN_INDICES.SPACE_IN_USE.ordinal() ? getDiskSpaceInUse(disk)
+			: columnIndex == SERVER_DISK_TABLE_COLUMN_INDICES.SPACE.ordinal() ? getDiskFreeSpace(disk) 
+			: columnIndex == SERVER_DISK_TABLE_COLUMN_INDICES.SPACE_IN_USE.ordinal() ? getTotalDiskSpace(disk)
 			: columnIndex == SERVER_DISK_TABLE_COLUMN_INDICES.STATUS.ordinal() ? glusterDataModelManager.getDiskStatus(disk) // disk.getStatusStr()
 			: "Invalid");
 		return ((columnText == null || columnText.trim().equals("")) ? CoreConstants.NA : columnText); 
diff --git a/src/com.gluster.storage.management.gui/src/com/gluster/storage/management/gui/actions/RemoveServerAction.java b/src/com.gluster.storage.management.gui/src/com/gluster/storage/management/gui/actions/RemoveServerAction.java
index 83489c37..514fd0a5 100644
--- a/src/com.gluster.storage.management.gui/src/com/gluster/storage/management/gui/actions/RemoveServerAction.java
+++ b/src/com.gluster.storage.management.gui/src/com/gluster/storage/management/gui/actions/RemoveServerAction.java
@@ -129,7 +129,7 @@ public class RemoveServerAction extends AbstractActionDelegate {
 			if (usedServers.size() == 1) {
 				showErrorDialog(action.getDescription(), "Server [" + usedServers.keySet().iterator().next()
 						+ "] cannot be removed as it is being used by volume(s): " + CoreConstants.NEWLINE
-						+ usedServers.values().iterator().next() + "]");
+						+ usedServers.values().iterator().next() );
 			} else {
 				String serverList = "";
 				for (Entry<GlusterServer, List<String>> entry : usedServers.entrySet()) {
@@ -137,7 +137,7 @@ public class RemoveServerAction extends AbstractActionDelegate {
 				}
 				showErrorDialog(action.getDescription(),
 						"Following servers cannot be removed as they are being used by volume(s): "
-								+ CoreConstants.NEWLINE + serverList + "]");
+								+ CoreConstants.NEWLINE + serverList );
 			}
 			return false;
 		}
diff --git a/src/com.gluster.storage.management.gui/src/com/gluster/storage/management/gui/views/pages/ServerDisksPage.java b/src/com.gluster.storage.management.gui/src/com/gluster/storage/management/gui/views/pages/ServerDisksPage.java
index c2f76a43..68367e8b 100644
--- a/src/com.gluster.storage.management.gui/src/com/gluster/storage/management/gui/views/pages/ServerDisksPage.java
+++ b/src/com.gluster.storage.management.gui/src/com/gluster/storage/management/gui/views/pages/ServerDisksPage.java
@@ -38,8 +38,8 @@ public class ServerDisksPage extends AbstractDisksPage {
 		DISK, SPACE, SPACE_IN_USE, STATUS
 	};
 
-	private static final String[] SERVER_DISK_TABLE_COLUMN_NAMES = new String[] { "Disk", "Space (GB)",
-			"Space in Use (GB)", "Status" };
+	private static final String[] SERVER_DISK_TABLE_COLUMN_NAMES = new String[] { "Disk", "Free Space (GB)",
+			"Total Space (GB)", "Status" };
 
 	@Override
 	protected String[] getColumnNames() {