Merge pull request #8 from algolia/feat/persistFragmentState

feat(VoiceDialogFragment): Persist suggestions

Author: q-litzler

app/src/main/java/com/algolia/instantsearch/voice/demo/MainActivity.kt

@@ -15,14 +15,15 @@ class MainActivity : AppCompatActivity(), VoiceInput.VoiceResultsListener {
         super.onCreate(savedInstanceState)
         setContentView(R.layout.activity_main)
 
+        val voiceFragment = VoiceDialogFragment()
+
         button.setOnClickListener { _ ->
             if (!Voice.hasRecordPermission(this)) {
                 PermissionDialogFragment().let {
                     it.arguments = PermissionDialogFragment.buildArguments(title = "Voice Search.")
                     it.show(supportFragmentManager, "perm")
                 }
             } else {
-                val voiceFragment = VoiceDialogFragment() //FIXME: Handle orientation changes, storing state properly
                 voiceFragment.setSuggestions("Something", "Something else")
                 voiceFragment.input.language = "en-US"
                 voiceFragment.input.maxResults = 2

instantsearch.voice/src/main/java/com/algolia/instantsearch/voice/ui/VoiceDialogFragment.kt

@@ -18,19 +18,28 @@ class VoiceDialogFragment : DialogFragment(), VoiceInput.VoiceInputPresenter {
 
     val input: VoiceInput = VoiceInput(this)
 
-    private var suggestions: List<String> = emptyList()
+    private var suggestions: Array<out String> = arrayOf()
 
     fun setSuggestions(vararg suggestions: String) {
-        this.suggestions = listOf(*suggestions)
+        this.suggestions = suggestions
     }
 
     //region Lifecycle
-    override fun onCreateView(inflater: LayoutInflater, container: ViewGroup?, savedInstanceState: Bundle?): View? =
-        inflater.inflate(R.layout.layout_voice_overlay, null)
+    override fun onCreateView(inflater: LayoutInflater, container: ViewGroup?, savedInstanceState: Bundle?): View? {
+        return inflater.inflate(R.layout.layout_voice_overlay, null)
+    }
+
+    override fun onSaveInstanceState(outState: Bundle) {
+        super.onSaveInstanceState(outState)
+        outState.putStringArray(keySuggestions, suggestions)
+    }
 
     override fun onViewCreated(view: View, savedInstanceState: Bundle?) {
         closeButton.setOnClickListener { dismiss() }
         micButton.setOnClickListener { input.toggleVoiceRecognition() }
+        savedInstanceState?.let {
+            suggestions = it.getStringArray(keySuggestions)
+        }
     }
 
     override fun onAttach(context: Context?) {
@@ -53,7 +62,8 @@ class VoiceDialogFragment : DialogFragment(), VoiceInput.VoiceInputPresenter {
     //region Helpers
     companion object {
 
-        const val SEPARATOR = "• "
+        const val separator = "• "
+        const val keySuggestions = "keySuggestions"
     }
     //endregion
 
@@ -66,7 +76,7 @@ class VoiceDialogFragment : DialogFragment(), VoiceInput.VoiceInputPresenter {
             hint.visibility = View.GONE
         } else {
             hint.visibility = View.VISIBLE
-            suggestionText.text = suggestions.fold("") { acc, it -> "$acc$SEPARATOR$it\n" }
+            suggestionText.text = suggestions.fold("") { acc, it -> "$acc$separator$it\n" }
         }
     }