- Limit where persistent files can be saved.

CASA/gui/CasaMain.cs

@@ -1750,6 +1750,7 @@ namespace Novell.CASA.GUI
 		public void okbuttonPreferences_clicked(object abj, EventArgs args)
 		{    	
 			Logger.DbgLog("GUI:CasaMain.okbuttonPreferences_clicked() - BEGIN");
+            bool bCloseDialog = true;
         
 			string[] storeID = new string[]{""};
         
@@ -1757,7 +1758,7 @@ namespace Novell.CASA.GUI
 			Common.IS_MOZILLA	  	= checkbuttonMozilla.Active;
 			Common.IS_GNOMEKEYRING 	= checkbuttonGnomeKeyring.Active;
 			Common.IS_KDEWALLET	  	= checkbuttonKdeWallet.Active;
-			dialogPreferences.Destroy();
+			
 			
 			/// FIREFOX
 			if(Common.IS_FIREFOX)
@@ -1841,12 +1842,18 @@ namespace Novell.CASA.GUI
 			string sOldDir = Common.GetUserPersistentDir(config);
 			try
 			{
-				if (MoveMiCASAFiles(sOldDir, sNewDir))
+                string sMoveMessage = MoveMiCASAFiles(sOldDir, sNewDir);
-				{
+                if (sMoveMessage.Equals("Success"))
-					// save off selected directory
+                {
-					Common.SetUserPersistentDir(config, sNewDir);										
+                    // save off selected directory
-					CommonGUI.DisplayMessage(Gtk.MessageType.Info, "Files moved from\r\n" + sOldDir + "\r\nto\r\n" + sNewDir);
+                    Common.SetUserPersistentDir(config, sNewDir);
-				}
+                    CommonGUI.DisplayMessage(Gtk.MessageType.Info, "Files moved from\r\n" + sOldDir + "\r\nto\r\n" + sNewDir);
+                }
+                else
+                {
+                    CommonGUI.DisplayMessage(Gtk.MessageType.Warning, sMoveMessage);
+                    bCloseDialog = false;
+                }
 			}
 			catch (Exception e)
 			{
@@ -1854,6 +1861,11 @@ namespace Novell.CASA.GUI
 			}
 			
 			config.WriteConfig();
+            if (bCloseDialog)
+            {
+                dialogPreferences.Destroy();
+            }
+
 			Logger.DbgLog("GUI:CasaMain.okbuttonPreferences_clicked() - END");          	   	
 		}
     
@@ -2225,8 +2237,13 @@ namespace Novell.CASA.GUI
 			}
 		}
 
-		private bool MoveMiCASAFiles(string sOldDirectory, string sNewDirectory)
+		private string MoveMiCASAFiles(string sOldDirectory, string sNewDirectory)
 		{
+            if (sNewDirectory == null || sNewDirectory.Length == 0)
+            {
+                return "No persistent directory entered";
+            }
+
 			if ((sOldDirectory != null) && (sNewDirectory != null))
 			{
 				if (!sOldDirectory.Equals(sNewDirectory))
@@ -2234,11 +2251,14 @@ namespace Novell.CASA.GUI
 					// instruct our daemon/service to change the location
 					ChangePersistentDir cpd = new ChangePersistentDir(sOldDirectory, sNewDirectory);
 					cpd = (ChangePersistentDir)MiCasaRequestReply.Send(MiCasaRequestReply.VERB_CHANGE_PERSIST_DIR, cpd);
-					if (cpd.GetErrorMessage().Equals("Success"))
+                    if (cpd.GetErrorMessage().Equals("Success"))
-						return true;
+                        return "Success";
+                    else
+                        return cpd.GetErrorMessage();
 				}
 			}
-			return false;
+
+            return null;
 		}
 
 

CASA/gui/Common.cs

@@ -442,7 +442,14 @@ public class Common
     
     internal static string GetUserName()
     {
-    	return Environment.GetEnvironmentVariable("USERNAME");    
+        if (Common.IS_LINUX)
+        {
+            return Environment.GetEnvironmentVariable("USER");
+        }
+        else
+        {
+            return Environment.GetEnvironmentVariable("USERNAME");
+        }
     }