Small code refactoring

This commit is contained in:
Dima-1 2019-12-31 11:34:25 +02:00
parent 9bd1945ade
commit b7a4c7947a
6 changed files with 15 additions and 22 deletions

View file

@ -3248,7 +3248,7 @@
<string name="profile_alert_need_save_title">حفظ التغييرات</string>
<string name="profile_alert_need_save_msg">احفظ التغييرات في ملف التعريف أولاً</string>
<string name="profile_alert_delete_title">حذف الملف الشخصي</string>
<string name="profile_alert_delete_msg">هل تريد بالتأكيد حذف ملف التعريف \"٪ s\"</string>
<string name="profile_alert_delete_msg">هل تريد بالتأكيد حذف ملف التعريف \"%s\"</string>
<string name="select_base_profile_dialog_title">حدد ملف تعريف أساسي</string>
<string name="select_base_profile_dialog_message">إسناد ملف التعريف المخصص الخاص بك إلى أحد ملفات تعريف التطبيق الافتراضية ، يحدد الإعداد الأساسي مثل الرؤية الافتراضية لعناصر واجهة المستخدم ووحدات السرعة والمسافة. وهذه هي ملفات تعريف التطبيق الافتراضية ، إلى جانب أمثلة لملفات تعريف مخصصة يمكن تمديدها إلى:</string>
<string name="select_nav_profile_dialog_title">حدد نوع الملاحة</string>

View file

@ -133,7 +133,6 @@ import net.osmand.plus.search.QuickSearchDialogFragment.QuickSearchType;
import net.osmand.plus.settings.BaseSettingsFragment;
import net.osmand.plus.settings.BaseSettingsFragment.SettingsScreenType;
import net.osmand.plus.settings.DataStorageFragment;
import net.osmand.plus.settings.MainSettingsFragment;
import net.osmand.plus.settings.ProfileAppearanceFragment;
import net.osmand.plus.views.AddGpxPointBottomSheetHelper.NewGpxPoint;
import net.osmand.plus.views.AnimateDraggingMapThread;
@ -684,10 +683,6 @@ public class MapActivity extends OsmandActionBarActivity implements DownloadEven
return;
}
}
MainSettingsFragment mainSettingsFragment = getMainSettingsFragment();
if (mainSettingsFragment != null) {
mainSettingsFragment.close();
}
if (mapContextMenu.isVisible() && mapContextMenu.isClosable()) {
if (mapContextMenu.getCurrentMenuState() != MenuState.HEADER_ONLY && !isLandscapeLayout()) {
mapContextMenu.openMenuHeaderOnly();
@ -2420,10 +2415,6 @@ public class MapActivity extends OsmandActionBarActivity implements DownloadEven
return getFragment(QuickActionListFragment.TAG);
}
public MainSettingsFragment getMainSettingsFragment() {
return getFragment(MainSettingsFragment.TAG);
}
<T> T getFragment(String fragmentTag){
Fragment fragment = getSupportFragmentManager().findFragmentByTag(fragmentTag);
return fragment != null && !fragment.isDetached() && !fragment.isRemoving() ? (T) fragment : null;

View file

@ -772,4 +772,11 @@ public abstract class BaseSettingsFragment extends PreferenceFragmentCompat impl
}
return false;
}
void updateRouteInfoMenu() {
MapActivity mapActivity = getMapActivity();
if (mapActivity != null) {
mapActivity.getMapRouteInfoMenu().updateMenu();
}
}
}

View file

@ -135,10 +135,7 @@ public class ConfigureProfileFragment extends BaseSettingsFragment {
TextView title = switchContainer.findViewById(R.id.switchButtonText);
title.setText(isChecked ? R.string.shared_string_on : R.string.shared_string_off);
MapActivity mapActivity = getMapActivity();
if (mapActivity != null) {
mapActivity.getMapRouteInfoMenu().updateMenu();
}
updateRouteInfoMenu();
}
@Override

View file

@ -16,7 +16,6 @@ import net.osmand.plus.ApplicationMode;
import net.osmand.plus.OsmandApplication;
import net.osmand.plus.R;
import net.osmand.plus.UiUtilities;
import net.osmand.plus.activities.MapActivity;
import net.osmand.plus.profiles.ProfileDataObject;
import net.osmand.plus.profiles.SelectProfileBottomSheetDialogFragment;
import net.osmand.plus.profiles.SelectProfileBottomSheetDialogFragment.SelectProfileListener;
@ -200,10 +199,9 @@ public class MainSettingsFragment extends BaseSettingsFragment {
return selectProfileListener;
}
public void close() {
MapActivity mapActivity = getMapActivity();
if (mapActivity != null) {
mapActivity.getMapRouteInfoMenu().updateMenu();
}
@Override
public void onPause() {
updateRouteInfoMenu();
super.onPause();
}
}

View file

@ -148,11 +148,11 @@ public class ProfileAppearanceFragment extends BaseSettingsFragment {
suffix++;
newName = oldName.substring(0, i + 1) + divider + suffix;
}
while (hasThatName(newName));
while (hasProfileWithName(newName));
return newName;
}
boolean hasThatName(String newName) {
private boolean hasProfileWithName(String newName) {
for (ApplicationMode m : ApplicationMode.allPossibleValues()) {
if (m.toHumanString(app).equals(newName)) {
return true;