[taler-taler-android] branch master updated: [pos] Ask for config on first start

[gnunet]

This is an automated email from the git hooks/post-receive script.

torsten-grote pushed a commit to branch master
in repository taler-android.

The following commit(s) were added to refs/heads/master by this push:
     new 92045c9  [pos] Ask for config on first start
92045c9 is described below

commit 92045c93baf45e627254246db4ea21f1eee11ce0
Author: Torsten Grote <t@grobox.de>
AuthorDate: Tue Apr 11 12:13:26 2023 -0300

    [pos] Ask for config on first start
    
    prevents connection to demo URL without user confirmation
---
 .../src/main/java/net/taler/merchantpos/MainActivity.kt        |  4 ++--
 .../java/net/taler/merchantpos/config/ConfigFetcherFragment.kt |  4 +---
 .../main/java/net/taler/merchantpos/config/ConfigFragment.kt   | 10 +++++-----
 .../main/java/net/taler/merchantpos/config/ConfigManager.kt    |  2 +-
 .../src/main/java/net/taler/merchantpos/config/PosConfig.kt    |  4 ++--
 5 files changed, 11 insertions(+), 13 deletions(-)

diff --git 
a/merchant-terminal/src/main/java/net/taler/merchantpos/MainActivity.kt 
b/merchant-terminal/src/main/java/net/taler/merchantpos/MainActivity.kt
index 5609429..4a46b27 100644
--- a/merchant-terminal/src/main/java/net/taler/merchantpos/MainActivity.kt
+++ b/merchant-terminal/src/main/java/net/taler/merchantpos/MainActivity.kt
@@ -75,8 +75,8 @@ class MainActivity : AppCompatActivity(), 
OnNavigationItemSelectedListener {
 
     override fun onStart() {
         super.onStart()
-        if (!model.configManager.config.isValid() && 
nav.currentDestination?.id != R.id.nav_settings) {
-            nav.navigate(R.id.action_global_merchantSettings)
+        if (!model.configManager.config.isValid()) {
+            if (nav.currentDestination?.id != R.id.nav_settings) 
nav.navigate(R.id.action_global_merchantSettings)
         } else if (model.configManager.merchantConfig == null && 
nav.currentDestination?.id != R.id.configFetcher) {
             nav.navigate(R.id.action_global_configFetcher)
         }
diff --git 
a/merchant-terminal/src/main/java/net/taler/merchantpos/config/ConfigFetcherFragment.kt
 
b/merchant-terminal/src/main/java/net/taler/merchantpos/config/ConfigFetcherFragment.kt
index 5564447..9e82a5a 100644
--- 
a/merchant-terminal/src/main/java/net/taler/merchantpos/config/ConfigFetcherFragment.kt
+++ 
b/merchant-terminal/src/main/java/net/taler/merchantpos/config/ConfigFetcherFragment.kt
@@ -45,9 +45,7 @@ class ConfigFetcherFragment : Fragment() {
         return ui.root
     }
 
-    @Deprecated("Deprecated in Java")
-    override fun onActivityCreated(savedInstanceState: Bundle?) {
-        super.onActivityCreated(savedInstanceState)
+    override fun onViewCreated(view: View, savedInstanceState: Bundle?) {
         configManager.fetchConfig(configManager.config, false)
         configManager.configUpdateResult.observe(viewLifecycleOwner) { result 
->
             when (result) {
diff --git 
a/merchant-terminal/src/main/java/net/taler/merchantpos/config/ConfigFragment.kt
 
b/merchant-terminal/src/main/java/net/taler/merchantpos/config/ConfigFragment.kt
index 3ee5148..9a5b7a7 100644
--- 
a/merchant-terminal/src/main/java/net/taler/merchantpos/config/ConfigFragment.kt
+++ 
b/merchant-terminal/src/main/java/net/taler/merchantpos/config/ConfigFragment.kt
@@ -49,7 +49,7 @@ class ConfigFragment : Fragment() {
         inflater: LayoutInflater,
         container: ViewGroup?,
         savedInstanceState: Bundle?
-    ): View? {
+    ): View {
         ui = FragmentMerchantConfigBinding.inflate(inflater, container, false)
         return ui.root
     }
@@ -74,11 +74,11 @@ class ConfigFragment : Fragment() {
                 password = ui.passwordView.editText!!.text.toString()
             )
             configManager.fetchConfig(config, true, 
ui.savePasswordCheckBox.isChecked)
-            configManager.configUpdateResult.observe(viewLifecycleOwner, { 
result ->
+            configManager.configUpdateResult.observe(viewLifecycleOwner) { 
result ->
                 if (onConfigUpdate(result)) {
                     
configManager.configUpdateResult.removeObservers(viewLifecycleOwner)
                 }
-            })
+            }
         }
         ui.forgetPasswordButton.setOnClickListener {
             configManager.forgetPassword()
@@ -93,8 +93,8 @@ class ConfigFragment : Fragment() {
         super.onStart()
         // focus password if this is the only empty field
         if (ui.passwordView.editText!!.text.isBlank()
-            && !ui.configUrlView.editText!!.text.isBlank()
-            && !ui.usernameView.editText!!.text.isBlank()
+            && ui.configUrlView.editText!!.text.isNotBlank()
+            && ui.usernameView.editText!!.text.isNotBlank()
         ) {
             ui.passwordView.requestFocus()
         }
diff --git 
a/merchant-terminal/src/main/java/net/taler/merchantpos/config/ConfigManager.kt 
b/merchant-terminal/src/main/java/net/taler/merchantpos/config/ConfigManager.kt
index 8986fc0..2cc0d8c 100644
--- 
a/merchant-terminal/src/main/java/net/taler/merchantpos/config/ConfigManager.kt
+++ 
b/merchant-terminal/src/main/java/net/taler/merchantpos/config/ConfigManager.kt
@@ -74,7 +74,7 @@ class ConfigManager(
     private val configurationReceivers = ArrayList<ConfigurationReceiver>()
 
     var config = Config(
-        configUrl = prefs.getString(SETTINGS_CONFIG_URL, CONFIG_URL_DEMO)!!,
+        configUrl = prefs.getString(SETTINGS_CONFIG_URL, "")!!,
         username = prefs.getString(SETTINGS_USERNAME, CONFIG_USERNAME_DEMO)!!,
         password = prefs.getString(SETTINGS_PASSWORD, CONFIG_PASSWORD_DEMO)!!
     )
diff --git 
a/merchant-terminal/src/main/java/net/taler/merchantpos/config/PosConfig.kt 
b/merchant-terminal/src/main/java/net/taler/merchantpos/config/PosConfig.kt
index 1f1ab74..556f05f 100644
--- a/merchant-terminal/src/main/java/net/taler/merchantpos/config/PosConfig.kt
+++ b/merchant-terminal/src/main/java/net/taler/merchantpos/config/PosConfig.kt
@@ -31,8 +31,8 @@ data class Config(
     val username: String,
     val password: String
 ) {
-    fun isValid() = !configUrl.isBlank()
-    fun hasPassword() = !password.isBlank()
+    fun isValid() = configUrl.isNotBlank()
+    fun hasPassword() = password.isNotBlank()
 }
 
 @Serializable

-- 
To stop receiving notification emails like this one, please contact
gnunet@gnunet.org.