You need to sign in or sign up before continuing.
Commit c13817fe authored by Varun Patil's avatar Varun Patil

Disable show contact before call, add null checks to settings

parent 461ebc3c
...@@ -41,6 +41,7 @@ public class SettingsFragment extends PreferenceFragmentCompat { ...@@ -41,6 +41,7 @@ public class SettingsFragment extends PreferenceFragmentCompat {
return false; return false;
} }
}); });
showContactPref.setEnabled(false);
// Update Profile // Update Profile
profilePref = findPreference("profile"); profilePref = findPreference("profile");
...@@ -109,8 +110,10 @@ public class SettingsFragment extends PreferenceFragmentCompat { ...@@ -109,8 +110,10 @@ public class SettingsFragment extends PreferenceFragmentCompat {
@Override @Override
public void onResponse(Call<User> call, Response<User> response) { public void onResponse(Call<User> call, Response<User> response) {
if (response.isSuccessful()) { if (response.isSuccessful()) {
if(getActivity() == null || getView() == null) return;
User user = response.body(); User user = response.body();
showContactPref.setChecked(user.getShowContactNumber()); showContactPref.setChecked(user.getShowContactNumber());
showContactPref.setEnabled(true);
} }
} }
}); });
...@@ -124,6 +127,7 @@ public class SettingsFragment extends PreferenceFragmentCompat { ...@@ -124,6 +127,7 @@ public class SettingsFragment extends PreferenceFragmentCompat {
retrofitInterface.patchUserMe(Utils.getSessionIDHeader(), new UserShowContactPatchRequest(isChecked)).enqueue(new Callback<User>() { retrofitInterface.patchUserMe(Utils.getSessionIDHeader(), new UserShowContactPatchRequest(isChecked)).enqueue(new Callback<User>() {
@Override @Override
public void onResponse(Call<User> call, Response<User> response) { public void onResponse(Call<User> call, Response<User> response) {
if(getActivity() == null || getView() == null) return;
if (response.isSuccessful()) { if (response.isSuccessful()) {
showContactPref.setChecked(isChecked); showContactPref.setChecked(isChecked);
showContactPref.setEnabled(true); showContactPref.setEnabled(true);
...@@ -135,6 +139,7 @@ public class SettingsFragment extends PreferenceFragmentCompat { ...@@ -135,6 +139,7 @@ public class SettingsFragment extends PreferenceFragmentCompat {
@Override @Override
public void onFailure(Call<User> call, Throwable t) { public void onFailure(Call<User> call, Throwable t) {
if(getActivity() == null || getView() == null) return;
showContactPref.setChecked(!isChecked); showContactPref.setChecked(!isChecked);
showContactPref.setEnabled(true); showContactPref.setEnabled(true);
} }
......
Markdown is supported
0% or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment