From a7f2b456c931a97267dd8b0335c441df1de75706 Mon Sep 17 00:00:00 2001
From: Selvasundaram <selvam@gluster.com>
Date: Tue, 26 Jul 2011 18:46:46 +0530
Subject: Bug 3048 - Added brick does not show proper free & total space. - fix

---
 .../storage/management/gui/views/VolumeSummaryView.java    | 14 ++++++++++++--
 1 file changed, 12 insertions(+), 2 deletions(-)

(limited to 'src')

diff --git a/src/com.gluster.storage.management.gui/src/com/gluster/storage/management/gui/views/VolumeSummaryView.java b/src/com.gluster.storage.management.gui/src/com/gluster/storage/management/gui/views/VolumeSummaryView.java
index 9879f2af..91a4e9b2 100644
--- a/src/com.gluster.storage.management.gui/src/com/gluster/storage/management/gui/views/VolumeSummaryView.java
+++ b/src/com.gluster.storage.management.gui/src/com/gluster/storage/management/gui/views/VolumeSummaryView.java
@@ -78,6 +78,9 @@ public class VolumeSummaryView extends ViewPart {
 	private String nfsMountInfo;
 	private Label nfsLabel;
 	private String nfs;
+	
+	private Label numberOfBricks;
+	private Label totalDiskSpace;
 
 	@Override
 	public void createPartControl(Composite parent) {
@@ -97,6 +100,7 @@ public class VolumeSummaryView extends ViewPart {
 				updateVolumeStatusLabel();
 				populateAccessControlText();
 				changeNFSStatus(volume.isNfsEnabled());
+				updateBrickChanges(volume);
 				toolbarManager.updateToolbar(volume);
 			}
 		};
@@ -420,6 +424,12 @@ public class VolumeSummaryView extends ViewPart {
 		nfsLabel.setVisible(isNFSExported);
 		nfsCheckBox.setSelection(isNFSExported);
 	}
+	
+	
+	private void updateBrickChanges(Volume volume) {
+		numberOfBricks.setText("" + volume.getNumOfBricks());
+		totalDiskSpace.setText("" + NumberUtil.formatNumber((getTotalDiskSpace() / 1024)));
+	}
 
 //	private void createChangeLinkForNASProtocol(Composite section, final Button nfsCheckBox) {
 //		final Hyperlink nasChangeLink = toolkit.createHyperlink(section, "change", SWT.NONE);
@@ -489,7 +499,7 @@ public class VolumeSummaryView extends ViewPart {
 	private void createDiskSpaceField(Composite section) {
 		Label diskSpaceLabel = toolkit.createLabel(section, "Total Disk Space (GB): ", SWT.NONE);
 		diskSpaceLabel.setToolTipText("<b>bold</b>normal");
-		toolkit.createLabel(section, "" + NumberUtil.formatNumber((getTotalDiskSpace() / 1024)), SWT.NONE);
+		totalDiskSpace = toolkit.createLabel(section, "" + NumberUtil.formatNumber((getTotalDiskSpace() / 1024)), SWT.NONE);
 		toolkit.createLabel(section, "", SWT.NONE); // dummy
 	}
 
@@ -520,7 +530,7 @@ public class VolumeSummaryView extends ViewPart {
 
 	private void createNumOfBricksField(Composite section) {
 		toolkit.createLabel(section, "Number of Bricks: ", SWT.NONE);
-		toolkit.createLabel(section, "" + volume.getNumOfBricks(), SWT.NONE);
+		numberOfBricks = toolkit.createLabel(section, "" + volume.getNumOfBricks(), SWT.NONE);
 		toolkit.createLabel(section, "", SWT.NONE); // dummy
 	}
 
-- 
cgit 


From 8ca69acc1f5a2df9f619fc222e985bb2a72c0174 Mon Sep 17 00:00:00 2001
From: Selvasundaram <selvam@gluster.com>
Date: Tue, 26 Jul 2011 20:39:31 +0530
Subject: White space character validation added for change password feature.

---
 .../storage/management/gui/dialogs/ChangePasswordDialog.java   | 10 +++++++++-
 1 file changed, 9 insertions(+), 1 deletion(-)

(limited to 'src')

diff --git a/src/com.gluster.storage.management.gui/src/com/gluster/storage/management/gui/dialogs/ChangePasswordDialog.java b/src/com.gluster.storage.management.gui/src/com/gluster/storage/management/gui/dialogs/ChangePasswordDialog.java
index b1367e00..9fc8fd3c 100644
--- a/src/com.gluster.storage.management.gui/src/com/gluster/storage/management/gui/dialogs/ChangePasswordDialog.java
+++ b/src/com.gluster.storage.management.gui/src/com/gluster/storage/management/gui/dialogs/ChangePasswordDialog.java
@@ -177,7 +177,7 @@ public class ChangePasswordDialog extends Dialog {
 			return;
 		}
 
-		if (confirmPwd.equals(CoreConstants.DEFAULT_PASSWORD)) {
+		if (confirmPwd.equals(CoreConstants.DEFAULT_PASSWORD) || isContainsWhiteSpace(confirmPwd)) {
 			okButton.setEnabled(false);
 			return;
 		}
@@ -199,6 +199,10 @@ public class ChangePasswordDialog extends Dialog {
 		setupDataBinding();
 	}
 
+	public boolean isContainsWhiteSpace(String text) {
+		return text.matches(".*[\\s\\\\].*"); // White space or backslash
+	}
+	
 	public class ConfirmPasswordValidator extends StringRequiredValidator {
 		public ConfirmPasswordValidator(String errorText, ControlDecoration controlDecoration, Control linkedControl) {
 			super(errorText, controlDecoration, linkedControl);
@@ -210,6 +214,10 @@ public class ChangePasswordDialog extends Dialog {
 			IStatus status = super.validate(value);
 			if (status.isOK()) {
 				String errMsg = null;
+				if (errMsg == null && isContainsWhiteSpace( newPassword.getText())) {
+					errMsg = "Password should not contain space or back slash characters";
+				}
+				
 				if (!value.equals(newPassword.getText())) {
 					errMsg = "Passwords mismatched";
 				}
-- 
cgit