Move keyboard listener to start of operation

Julian Kast · RHenigan · commit f818b87df769 · 2021-12-06T10:22:46.000-05:00
diff --git a/base/src/main/java/com/smartdevicelink/managers/screen/choiceset/PreloadPresentChoicesOperation.java b/base/src/main/java/com/smartdevicelink/managers/screen/choiceset/PreloadPresentChoicesOperation.java
@@ -185,6 +185,7 @@ public void onExecute() {
             choiceId = 1;
             reachedMaxIds = false;
         }
+        addListeners();
 
         DebugTool.logInfo(TAG, "Choice Operation: Executing preload choices operation");
         // Enforce unique cells and remove cells that are already loaded
@@ -343,7 +344,6 @@ private void updateKeyboardProperties(final CompletionListener listener) {
             return;
         }
 
-        addListeners();
 
         if (keyboardListener != null && choiceSet.getCustomKeyboardConfiguration() != null) {
             keyboardProperties = choiceSet.getCustomKeyboardConfiguration();

Original file line number	Diff line number	Diff line change
`@@ -185,6 +185,7 @@ public void onExecute() {`
`185`	`185`	`choiceId = 1;`
`186`	`186`	`reachedMaxIds = false;`
`187`	`187`	`}`
	`188`	`+ addListeners();`
`188`	`189`
`189`	`190`	`DebugTool.logInfo(TAG, "Choice Operation: Executing preload choices operation");`
`190`	`191`	`// Enforce unique cells and remove cells that are already loaded`
`@@ -343,7 +344,6 @@ private void updateKeyboardProperties(final CompletionListener listener) {`
`343`	`344`	`return;`
`344`	`345`	`}`
`345`	`346`
`346`		`- addListeners();`
`347`	`347`
`348`	`348`	`if (keyboardListener != null && choiceSet.getCustomKeyboardConfiguration() != null) {`
`349`	`349`	`keyboardProperties = choiceSet.getCustomKeyboardConfiguration();`