2012-06-30 Santiago Munín <burning1@gmail.com>

* src/pandroid_event_viewer/pandorafms/Main.java: Added a check in order to prevent profile overwritings.
	* src/pandroid_event_viewer/pandorafms/Core.java: Added method which finds a string in a list (case insensitive).


git-svn-id: https://svn.code.sf.net/p/pandora/code/trunk@6726 c3f86ba8-e40f-0410-aaad-9ba5e7f4b01f

extras/pandroid_event_viewer/ChangeLog

@@ -1,3 +1,7 @@
+2012-06-30 Santiago Munín <burning1@gmail.com>
+	* src/pandroid_event_viewer/pandorafms/Main.java: Added a check in order to prevent profile overwritings.
+	* src/pandroid_event_viewer/pandorafms/Core.java: Added method which finds a string in a list (case insensitive). 
+
 2012-06-29 Santiago Munín <burning1@gmail.com>
 	* src/pandroid_event_viewer/pandorafms/Main.java: Added a little hack to make it compatible with v4.0.2
 	* res/layout/main.xml: Added a little hack to make it compatible with v4.0.2

extras/pandroid_event_viewer/src/pandroid_event_viewer/pandorafms/Core.java

@@ -31,6 +31,7 @@ import java.security.SecureRandom;
 import java.util.ArrayList;
 import java.util.Calendar;
 import java.util.HashMap;
+import java.util.Iterator;
 import java.util.List;
 import java.util.Map;
 
@@ -586,6 +587,23 @@ public class Core {
 						Toast.LENGTH_SHORT).show();
 			}
 		}
+	}
 
+	/**
+	 * Checks if a string is inside the list (case insensitive).
+	 * 
+	 * @param list
+	 *            List of strings.
+	 * @param string
+	 *            Given value.
+	 * @return If the string is inside the list.
+	 */
+	public static boolean containsIgnoreCase(List<String> list, String string) {
+		Iterator<String> it = list.iterator();
+		while (it.hasNext()) {
+			if (it.next().equalsIgnoreCase(string))
+				return true;
+		}
+		return false;
 	}
 }

extras/pandroid_event_viewer/src/pandroid_event_viewer/pandorafms/Main.java

@@ -211,14 +211,13 @@ public class Main extends Activity {
 					public void onClick(DialogInterface dialog, int which) {
 						switch (which) {
 						case DialogInterface.BUTTON_POSITIVE:
-							/*
+							if (Core.containsIgnoreCase(profiles, profileName
-							 * if (profiles.contains(profileName.getText()
+									.getText().toString())) {
-							 * .toString())) { Toast.makeText(context,
+								Toast.makeText(context,
-							 * R.string.profile_already_exists,
+										R.string.profile_already_exists,
-							 * Toast.LENGTH_SHORT).show(); break; }
+										Toast.LENGTH_SHORT).show();
-							 */
+								break;
-							// TODO Ask for a confirmation before rewriting the
+							}
-							// profile
 							if (profileName.getText().toString().contains("|")) {
 								Toast.makeText(
 										context,