Valor de preferencia personalizado de Android no guardado

Question

Estoy intentando que funcione una subclase de Preferencias existente que ahorre un valor de color. No escribí la clase, pero se supone que está trabajando en android-7 y superior (estoy compilando para un destino android-9). El código fuente completo es available here. Aquí es donde se guarda la preferencia:

@Override 
public void onColorChanged(int color) { 
    if (isPersistent()) { 
     boolean ret = persistInt(color); 
    } 
    // (update preview box, other stuff) 
} 

Uso de la salida de depuración que puede decir que isPersistent() vuelve true, pero persistInt() vuelve false. De acuerdo con Android documentation, persistInt() devuelve si la preferencia es persistente; ¿Cómo pueden estos devolver diferentes valores? (Nota: setPersistent(true) se llama explícitamente desde el constructor.)

En cualquier caso, el valor no se guarda. Una llamada al getPersistedInt(defaultValue) devuelve el valor predeterminado, incluso más adelante en la misma instancia de la clase. En el código siguiente, siempre se llama a getPersistedInt() y siempre devuelve mDefaultValue.

public int getValue() { 
    try { 
     if (isPersistent()) { 
      mValue = getPersistedInt(mDefaultValue); 
     } 
    } catch (ClassCastException e) { 
     mValue = mDefaultValue; 
    } 

    return mValue; 
} 

¿Por qué es esto y cómo puedo asegurarme de que la preferencia se mantenga?

Answer 1

Después de muchas búsquedas sin remedio, finalmente encontré el problema: al valor de preferencia no se le asignó una clave, debido a un simple error en mi archivo XML de preferencias. (Utilicé android.key en lugar de android:key.)

Dado que Android no le advierte cuando se trata de persistir una preferencia que no tiene clave (pero en silencio falla en cambio,) debe llamar a la función shouldPersist() en lugar de isPersistent() antes de tratar de persistir un valor, y tal vez registrar una advertencia si shouldPersist() devuelve falso. Por ejemplo:

@Override 
public void onColorChanged(int color) { 
    mValue = color; 
    if (shouldPersist()) { 
     persistInt(color); 
    } else { 
     if (isPersistent()) 
      Log.w("myapp", "shouldPersist() returned false. Check if this preference has a key."); 
    } 
    // (update preview box, other stuff) 
}