Donate to e Foundation | Murena handsets with /e/OS | Own a part of Murena! Learn more

Commit beb763b6 authored by Danny Baumann's avatar Danny Baumann Committed by Ricardo Cerqueira
Browse files

Fix certain setting writes being incorrectly skipped. 

If a value that previously had a very long value (over the 500 char
threshold) is now set to null, isRedundantSetValue would incorrectly
return true even though the value was changed.

Change-Id: Ia5304743024ec01024be98aa51ed260d38caf93c
parent df67c391

packages/SettingsProvider/src/com/android/providers/settings/SettingsProvider.java

+1 −1
Original line number Diff line number Diff line
@@ -1171,7 +1171,7 @@ public class SettingsProvider extends ContentProvider { 
            if (cache == null) return false;

            synchronized (cache) {

                Bundle bundle = cache.get(name);

                if (bundle == null) return false;

                if (bundle == null || bundle == TOO_LARGE_TO_CACHE_MARKER) return false;

                String oldValue = bundle.getPairValue();

                if (oldValue == null && value == null) return true;

                if ((oldValue == null) != (value == null)) return false;