write password into new storage

and fix migration and do not store empty strings
and add missing newlines in German translation

fixes #77

Author: ostrya

app/build.gradle.kts

@@ -47,8 +47,8 @@ android {
         multiDexEnabled = true
         targetSdk = 35
         vectorDrawables.useSupportLibrary = true
-        versionCode = 57
-        versionName = "2.6.5"
+        versionCode = 58
+        versionName = "2.6.6"
         javaCompileOptions {
             annotationProcessorOptions {
                 argument("room.schemaLocation", "$projectDir/schemas")

app/src/main/java/org/ostrya/presencepublisher/PresencePublisher.java

@@ -181,6 +181,13 @@ private void migratePreferences(SharedPreferences preferences) {
         if (!devicePreferences.contains(PASSWORD) && value != null) {
             devicePreferences.putString(PASSWORD, value);
         }
+        // fix migration from 2.6.5: since the old storage returned "" if no data was stored,
+        // we did store "" into the new storage even though that's pointless
+        value = devicePreferences.getString(PASSWORD, null);
+        // so just remove it again in this case
+        if ("".equals(value)) {
+            devicePreferences.putString(PASSWORD, null);
+        }
         value = preferences.getString(MQTT_CLIENT_ID, null);
         if (!devicePreferences.contains(MQTT_CLIENT_ID) && value != null) {
             devicePreferences.putString(MQTT_CLIENT_ID, value);

app/src/main/java/org/ostrya/presencepublisher/device/DevicePreferences.java

@@ -85,7 +85,7 @@ private SecretKey getOrInitKey()
 
     @Override
     public void putString(String key, @Nullable String value) {
-        if (value != null) {
+        if (value != null && !value.isEmpty()) {
             if (Build.VERSION.SDK_INT >= Build.VERSION_CODES.M) {
                 try {
                     SecretKey secretKey = getOrInitKey();

app/src/main/java/org/ostrya/presencepublisher/preference/connection/PasswordPreference.java

@@ -15,6 +15,7 @@
 import androidx.security.crypto.MasterKey;
 
 import org.ostrya.presencepublisher.R;
+import org.ostrya.presencepublisher.device.DevicePreferences;
 import org.ostrya.presencepublisher.log.DatabaseLogger;
 import org.ostrya.presencepublisher.preference.common.TextPreferenceBase;
 
@@ -37,7 +38,7 @@ public PasswordPreference(Context context) {
                         editText.setAutofillHints(View.AUTOFILL_HINT_PASSWORD);
                     }
                 });
-        setPreferenceDataStore(new SecureDataStore(context));
+        setPreferenceDataStore(new DevicePreferences(context));
     }
 
     @Override

app/src/main/play/release-notes/de-DE/default.txt

@@ -1,6 +1,5 @@
+v2.6.6
+• Fehler beim Speichern des MQTT-Passworts behoben
+
 v2.6.5
 • neue Warnung, dass Nachrichten ohne 'conditionContent' immer versendet werden
-
-v2.6.4
-• noch mehr Probleme bei WLAN-Erkennung für Android 10 und 11 behoben
-• falsches Versenden von Nachrichten ohne passende Bedingung behoben

app/src/main/play/release-notes/en-US/default.txt

@@ -1,6 +1,5 @@
+v2.6.6
+• fixed issue when storing MQTT password
+
 v2.6.5
 • new warning about always sending messages without 'conditionContent'
-
-v2.6.4
-• fixed even more issues with wifi detection in Android 10 and 11
-• fixed wrongly sending messages if no condition matches

app/src/main/res/values-de/strings.xml

@@ -130,7 +130,7 @@
     <string name="edit_message_title_2">Nachricht bearbeiten (2)</string>
     <string name="message_name_hint">Name</string>
     <string name="message_topic_hint">Topic</string>
-    <string name="message_help_summary">Diese Liste von Nachrichten wird zum gegebenen Zeitplan versendet.\n\nTippe, um die Nachricht zu bearbeiten. Berühre länger, um einen Eintrag zu löschen.⚠️ Wenn der \"conditionContent\" Eintrag zu einer Nachricht hinzugefügt wird, so wird diese Nachricht nur versendet, wenn mindestens eine Bedingung erfüllt ist. Wenn der Eintrag nicht hinzugefügt wird, so wird die Nachricht immer versendet.</string>
+    <string name="message_help_summary">Diese Liste von Nachrichten wird zum gegebenen Zeitplan versendet.\n\nTippe, um die Nachricht zu bearbeiten. Berühre länger, um einen Eintrag zu löschen.\n\n⚠️ Wenn der \"conditionContent\" Eintrag zu einer Nachricht hinzugefügt wird, so wird diese Nachricht nur versendet, wenn mindestens eine Bedingung erfüllt ist. Wenn der Eintrag nicht hinzugefügt wird, so wird die Nachricht immer versendet.</string>
     <string name="message_format_setting_title">Nachrichtenformat</string>
     <string name="message_format_setting_summary" formatted="true">Das Format, in dem MQTT-Nachrichten versendet werden.%nAktueller Wert: %1$s</string>
     <string name="message_format_help_summary">Für weitere Details über die verschiedenen Nachrichtenformate bitte hier klicken.</string>