Commit 619d88a7 authored by Bartek Fabiszewski's avatar Bartek Fabiszewski
Browse files

Fix: print debug only if DEBUG is on

parent ef891126

app/src/main/java/net/fabiszewski/ulogger/LoggerService.java

+1 −1
Original line number Diff line number Diff line
@@ -458,7 +458,7 @@ public class LoggerService extends Service { 
                        statusString = "unknown";

                        break;

                }

                Log.d(TAG, "[location status for " + provider + " changed: " + statusString + "]");

                if (Logger.DEBUG) { Log.d(TAG, "[location status for " + provider + " changed: " + statusString + "]"); }

            }

        }

    }

app/src/main/java/net/fabiszewski/ulogger/ProviderPreference.java

+4 −4
Original line number Diff line number Diff line
@@ -67,7 +67,7 @@ class ProviderPreference extends ListPreference { 
        missingProviders = new ArrayList<>();

        final boolean existsGPS = locManager.getAllProviders().contains(LocationManager.GPS_PROVIDER);

        final boolean existsNet = locManager.getAllProviders().contains(LocationManager.NETWORK_PROVIDER);

        Log.d(TAG, "[Providers available: " + locManager.getAllProviders());

        if (Logger.DEBUG) { Log.d(TAG, "[Providers available: " + locManager.getAllProviders()); }

        defaultValue = VALUE_ALL;

        if (!existsGPS) {

            missingProviders.add(VALUE_GPS);
@@ -83,7 +83,7 @@ class ProviderPreference extends ListPreference { 
        setOnPreferenceChangeListener(new OnPreferenceChangeListener() {

            @Override

            public boolean onPreferenceChange(Preference preference, Object newValue) {

                Log.d(TAG, "[preference changed: " + newValue + "]");

                if (Logger.DEBUG) { Log.d(TAG, "[preference changed: " + newValue + "]"); }

                int providersMask = Integer.valueOf((String) newValue);

                SharedPreferences prefs = PreferenceManager.getDefaultSharedPreferences(context);

                SharedPreferences.Editor editor = prefs.edit();
@@ -106,10 +106,10 @@ class ProviderPreference extends ListPreference { 
        entries = getEntries();

        entryValues = getEntryValues();

        currentIndex = findIndexOfValue(getValue());

        Log.d(TAG, "[current value: " + currentIndex + "]");

        if (Logger.DEBUG) { Log.d(TAG, "[current value: " + currentIndex + "]"); }

        if (currentIndex == -1) {

            currentIndex = findIndexOfValue(String.valueOf(defaultValue));

            Log.d(TAG, "[using default: " + currentIndex + "]");

            if (Logger.DEBUG) { Log.d(TAG, "[using default: " + currentIndex + "]"); }

        }



        ListAdapter listAdapter = new CustomAdapter(getContext(), getEntries());