Add method to BaseVoiceCommandManager to check for uniqueness

Author: Julian Kast

base/src/main/java/com/smartdevicelink/managers/screen/menu/BaseVoiceCommandManager.java

@@ -51,6 +51,7 @@
 
 import java.util.ArrayList;
 import java.util.HashMap;
+import java.util.HashSet;
 import java.util.List;
 
 abstract class BaseVoiceCommandManager extends BaseSubManager {
@@ -136,6 +137,11 @@ public void setVoiceCommands(List<VoiceCommand> voiceCommands) {
             return;
         }
 
+        if (!isVoiceCommandsUnique(voiceCommands)) {
+            DebugTool.logError(TAG, "Not all voice command strings are unique across all voice commands. Voice commands will not be set.");
+            return;
+        }
+
         updateIdsOnVoiceCommands(voiceCommands);
         this.voiceCommands = new ArrayList<>(voiceCommands);
 
@@ -225,4 +231,21 @@ public void onNotified(RPCNotification notification) {
         };
         internalInterface.addOnRPCNotificationListener(FunctionID.ON_COMMAND, commandListener);
     }
+
+    /**
+     * Boolean method that checks to see if all VoiceCommands in a given list are unique
+     *
+     * @param voiceCommands - list of VoiceCommands
+     * @return - true if VoiceCommands are unique, false if not
+     */
+    private boolean isVoiceCommandsUnique(List<VoiceCommand> voiceCommands) {
+        HashSet<String> voiceCommandHashSet = new HashSet<>();
+        int voiceCommandCount = 0;
+
+        for (VoiceCommand voiceCommand : voiceCommands) {
+            voiceCommandHashSet.addAll(voiceCommand.getVoiceCommands());
+            voiceCommandCount += voiceCommand.getVoiceCommands().size();
+        }
+        return (voiceCommandHashSet.size() == voiceCommandCount);
+    }
 }