From 6d3cc1a9dac10e54917ec7562ba3d82d70118eec Mon Sep 17 00:00:00 2001 From: ByteHamster Date: Mon, 30 Mar 2020 14:24:55 +0200 Subject: [PATCH] Checkstyle fixes --- .../gpodnet/GPodnetServiceTest.java | 2 +- .../fragment/gpodnet/PodcastListFragment.java | 4 +- .../fragment/gpodnet/TagListFragment.java | 3 +- .../GpodderPreferencesFragment.java | 32 ++++++------ config/checkstyle/checkstyle-new-code.xml | 1 - .../antennapod/core/sync/SyncService.java | 9 ++-- .../core/sync/gpoddernet/GpodnetService.java | 34 ++++++------ .../core/sync/model/EpisodeAction.java | 52 +++++++++---------- .../core/sync/model/EpisodeActionChanges.java | 8 +-- 9 files changed, 73 insertions(+), 72 deletions(-) diff --git a/app/src/androidTest/java/de/test/antennapod/gpodnet/GPodnetServiceTest.java b/app/src/androidTest/java/de/test/antennapod/gpodnet/GPodnetServiceTest.java index 587dd38c8..e7e8c5b09 100644 --- a/app/src/androidTest/java/de/test/antennapod/gpodnet/GPodnetServiceTest.java +++ b/app/src/androidTest/java/de/test/antennapod/gpodnet/GPodnetServiceTest.java @@ -52,7 +52,7 @@ public class GPodnetServiceTest { ArrayList l = new ArrayList<>(); l.add("http://bitsundso.de/feed"); l.add("http://gamesundso.de/feed"); - service.uploadSubscriptions( "radio", l); + service.uploadSubscriptions("radio", l); } @Test diff --git a/app/src/main/java/de/danoeh/antennapod/fragment/gpodnet/PodcastListFragment.java b/app/src/main/java/de/danoeh/antennapod/fragment/gpodnet/PodcastListFragment.java index 915abdb0d..87378ae6c 100644 --- a/app/src/main/java/de/danoeh/antennapod/fragment/gpodnet/PodcastListFragment.java +++ b/app/src/main/java/de/danoeh/antennapod/fragment/gpodnet/PodcastListFragment.java @@ -116,9 +116,9 @@ public abstract class PodcastListFragment extends Fragment { @Override protected List doInBackground(Void... params) { - GpodnetService service = null; try { - service = new GpodnetService(AntennapodHttpClient.getHttpClient(), GpodnetPreferences.getHostname()); + GpodnetService service = new GpodnetService(AntennapodHttpClient.getHttpClient(), + GpodnetPreferences.getHostname()); return loadPodcastData(service); } catch (GpodnetServiceException e) { exception = e; diff --git a/app/src/main/java/de/danoeh/antennapod/fragment/gpodnet/TagListFragment.java b/app/src/main/java/de/danoeh/antennapod/fragment/gpodnet/TagListFragment.java index 613218a91..a7a0781ce 100644 --- a/app/src/main/java/de/danoeh/antennapod/fragment/gpodnet/TagListFragment.java +++ b/app/src/main/java/de/danoeh/antennapod/fragment/gpodnet/TagListFragment.java @@ -92,7 +92,8 @@ public class TagListFragment extends ListFragment { @Override protected List doInBackground(Void... params) { - GpodnetService service = new GpodnetService(AntennapodHttpClient.getHttpClient(), GpodnetPreferences.getHostname()); + GpodnetService service = new GpodnetService(AntennapodHttpClient.getHttpClient(), + GpodnetPreferences.getHostname()); try { return service.getTopTags(COUNT); } catch (GpodnetServiceException e) { diff --git a/app/src/main/java/de/danoeh/antennapod/fragment/preferences/GpodderPreferencesFragment.java b/app/src/main/java/de/danoeh/antennapod/fragment/preferences/GpodderPreferencesFragment.java index a1fa4ae3a..11afb9b4f 100644 --- a/app/src/main/java/de/danoeh/antennapod/fragment/preferences/GpodderPreferencesFragment.java +++ b/app/src/main/java/de/danoeh/antennapod/fragment/preferences/GpodderPreferencesFragment.java @@ -82,25 +82,25 @@ public class GpodderPreferencesFragment extends PreferenceFragmentCompat { return true; }); findPreference(PREF_GPODNET_SYNC).setOnPreferenceClickListener(preference -> { - SyncService.syncImmediately(getActivity().getApplicationContext()); - return true; - }); + SyncService.syncImmediately(getActivity().getApplicationContext()); + return true; + }); findPreference(PREF_GPODNET_FORCE_FULL_SYNC).setOnPreferenceClickListener(preference -> { - SyncService.fullSync(getContext()); - return true; - }); + SyncService.fullSync(getContext()); + return true; + }); findPreference(PREF_GPODNET_LOGOUT).setOnPreferenceClickListener(preference -> { - GpodnetPreferences.logout(); - Toast toast = Toast.makeText(activity, R.string.pref_gpodnet_logout_toast, Toast.LENGTH_SHORT); - toast.show(); - updateGpodnetPreferenceScreen(); - return true; - }); + GpodnetPreferences.logout(); + Toast toast = Toast.makeText(activity, R.string.pref_gpodnet_logout_toast, Toast.LENGTH_SHORT); + toast.show(); + updateGpodnetPreferenceScreen(); + return true; + }); findPreference(PREF_GPODNET_HOSTNAME).setOnPreferenceClickListener(preference -> { - GpodnetSetHostnameDialog.createDialog(activity).setOnDismissListener( - dialog -> updateGpodnetPreferenceScreen()); - return true; - }); + GpodnetSetHostnameDialog.createDialog(activity).setOnDismissListener( + dialog -> updateGpodnetPreferenceScreen()); + return true; + }); } private void updateGpodnetPreferenceScreen() { diff --git a/config/checkstyle/checkstyle-new-code.xml b/config/checkstyle/checkstyle-new-code.xml index 0076bac23..cb2129a60 100644 --- a/config/checkstyle/checkstyle-new-code.xml +++ b/config/checkstyle/checkstyle-new-code.xml @@ -176,7 +176,6 @@ - diff --git a/core/src/main/java/de/danoeh/antennapod/core/sync/SyncService.java b/core/src/main/java/de/danoeh/antennapod/core/sync/SyncService.java index 4996decd2..be35badf0 100644 --- a/core/src/main/java/de/danoeh/antennapod/core/sync/SyncService.java +++ b/core/src/main/java/de/danoeh/antennapod/core/sync/SyncService.java @@ -148,7 +148,7 @@ public class SyncService extends Worker { SharedPreferences prefs = context.getSharedPreferences(PREF_NAME, Context.MODE_PRIVATE); String json = prefs.getString(PREF_QUEUED_EPISODE_ACTIONS, "[]"); JSONArray queue = new JSONArray(json); - queue.put(action.writeToJSONObject()); + queue.put(action.writeToJsonObject()); prefs.edit().putString(PREF_QUEUED_EPISODE_ACTIONS, queue.toString()).apply(); } catch (JSONException e) { e.printStackTrace(); @@ -217,7 +217,7 @@ public class SyncService extends Worker { String json = prefs.getString(PREF_QUEUED_EPISODE_ACTIONS, "[]"); JSONArray queue = new JSONArray(json); for (int i = 0; i < queue.length(); i++) { - actions.add(EpisodeAction.readFromJSONObject(queue.getJSONObject(i))); + actions.add(EpisodeAction.readFromJsonObject(queue.getJSONObject(i))); } } catch (JSONException e) { e.printStackTrace(); @@ -297,7 +297,7 @@ public class SyncService extends Worker { synchronized (lock) { UploadChangesResponse uploadResponse = syncServiceImpl - .uploadSubscriptionChanges(queuedAddedFeeds, queuedRemovedFeeds); + .uploadSubscriptionChanges(queuedAddedFeeds, queuedRemovedFeeds); getApplicationContext().getSharedPreferences(PREF_NAME, Context.MODE_PRIVATE).edit() .putString(PREF_QUEUED_FEEDS_ADDED, "[]").apply(); getApplicationContext().getSharedPreferences(PREF_NAME, Context.MODE_PRIVATE).edit() @@ -406,6 +406,9 @@ public class SyncService extends Worker { case DELETE: // NEVER EVER call DBWriter.deleteFeedMediaOfItem() here, leads to an infinite loop break; + default: + Log.e(TAG, "Unknown action: " + action); + break; } } diff --git a/core/src/main/java/de/danoeh/antennapod/core/sync/gpoddernet/GpodnetService.java b/core/src/main/java/de/danoeh/antennapod/core/sync/gpoddernet/GpodnetService.java index 327232e1d..a35c961eb 100644 --- a/core/src/main/java/de/danoeh/antennapod/core/sync/gpoddernet/GpodnetService.java +++ b/core/src/main/java/de/danoeh/antennapod/core/sync/gpoddernet/GpodnetService.java @@ -109,7 +109,7 @@ public class GpodnetService implements ISyncService { String response = executeRequest(request); JSONArray jsonArray = new JSONArray(response); - return readPodcastListFromJSONArray(jsonArray); + return readPodcastListFromJsonArray(jsonArray); } catch (JSONException | MalformedURLException | URISyntaxException e) { e.printStackTrace(); @@ -134,7 +134,7 @@ public class GpodnetService implements ISyncService { String response = executeRequest(request); JSONArray jsonArray = new JSONArray(response); - return readPodcastListFromJSONArray(jsonArray); + return readPodcastListFromJsonArray(jsonArray); } catch (JSONException | MalformedURLException | URISyntaxException e) { e.printStackTrace(); @@ -165,7 +165,7 @@ public class GpodnetService implements ISyncService { String response = executeRequest(request); JSONArray jsonArray = new JSONArray(response); - return readPodcastListFromJSONArray(jsonArray); + return readPodcastListFromJsonArray(jsonArray); } catch (JSONException | MalformedURLException | URISyntaxException e) { e.printStackTrace(); throw new GpodnetServiceException(e); @@ -191,7 +191,7 @@ public class GpodnetService implements ISyncService { String response = executeRequest(request); JSONArray jsonArray = new JSONArray(response); - return readPodcastListFromJSONArray(jsonArray); + return readPodcastListFromJsonArray(jsonArray); } catch (JSONException | MalformedURLException e) { e.printStackTrace(); @@ -216,7 +216,7 @@ public class GpodnetService implements ISyncService { Request.Builder request = new Request.Builder().url(url); String response = executeRequest(request); JSONArray devicesArray = new JSONArray(response); - return readDeviceListFromJSONArray(devicesArray); + return readDeviceListFromJsonArray(devicesArray); } catch (JSONException | MalformedURLException | URISyntaxException e) { e.printStackTrace(); throw new GpodnetServiceException(e); @@ -393,7 +393,7 @@ public class GpodnetService implements ISyncService { String response = executeRequest(request); JSONObject changes = new JSONObject(response); - return readSubscriptionChangesFromJSONObject(changes); + return readSubscriptionChangesFromJsonObject(changes); } catch (URISyntaxException e) { e.printStackTrace(); throw new IllegalStateException(e); @@ -436,7 +436,7 @@ public class GpodnetService implements ISyncService { final JSONArray list = new JSONArray(); for (int i = from; i < to; i++) { EpisodeAction episodeAction = episodeActions.get(i); - JSONObject obj = episodeAction.writeToJSONObject(); + JSONObject obj = episodeAction.writeToJsonObject(); if (obj != null) { obj.put("device", GpodnetPreferences.getDeviceID()); list.put(obj); @@ -474,7 +474,7 @@ public class GpodnetService implements ISyncService { String response = executeRequest(request); JSONObject json = new JSONObject(response); - return readEpisodeActionsFromJSONObject(json); + return readEpisodeActionsFromJsonObject(json); } catch (URISyntaxException e) { e.printStackTrace(); throw new IllegalStateException(e); @@ -570,15 +570,15 @@ public class GpodnetService implements ISyncService { } } - private List readPodcastListFromJSONArray(@NonNull JSONArray array) throws JSONException { + private List readPodcastListFromJsonArray(@NonNull JSONArray array) throws JSONException { List result = new ArrayList<>(array.length()); for (int i = 0; i < array.length(); i++) { - result.add(readPodcastFromJSONObject(array.getJSONObject(i))); + result.add(readPodcastFromJsonObject(array.getJSONObject(i))); } return result; } - private GpodnetPodcast readPodcastFromJSONObject(JSONObject object) throws JSONException { + private GpodnetPodcast readPodcastFromJsonObject(JSONObject object) throws JSONException { String url = object.getString("url"); String title; @@ -623,15 +623,15 @@ public class GpodnetService implements ISyncService { return new GpodnetPodcast(url, title, description, subscribers, logoUrl, website, mygpoLink, author); } - private List readDeviceListFromJSONArray(@NonNull JSONArray array) throws JSONException { + private List readDeviceListFromJsonArray(@NonNull JSONArray array) throws JSONException { List result = new ArrayList<>(array.length()); for (int i = 0; i < array.length(); i++) { - result.add(readDeviceFromJSONObject(array.getJSONObject(i))); + result.add(readDeviceFromJsonObject(array.getJSONObject(i))); } return result; } - private GpodnetDevice readDeviceFromJSONObject(JSONObject object) throws JSONException { + private GpodnetDevice readDeviceFromJsonObject(JSONObject object) throws JSONException { String id = object.getString("id"); String caption = object.getString("caption"); String type = object.getString("type"); @@ -639,7 +639,7 @@ public class GpodnetService implements ISyncService { return new GpodnetDevice(id, caption, type, subscriptions); } - private SubscriptionChanges readSubscriptionChangesFromJSONObject(@NonNull JSONObject object) + private SubscriptionChanges readSubscriptionChangesFromJsonObject(@NonNull JSONObject object) throws JSONException { List added = new LinkedList<>(); @@ -664,7 +664,7 @@ public class GpodnetService implements ISyncService { return new SubscriptionChanges(added, removed, timestamp); } - private EpisodeActionChanges readEpisodeActionsFromJSONObject(@NonNull JSONObject object) + private EpisodeActionChanges readEpisodeActionsFromJsonObject(@NonNull JSONObject object) throws JSONException { List episodeActions = new ArrayList<>(); @@ -673,7 +673,7 @@ public class GpodnetService implements ISyncService { JSONArray jsonActions = object.getJSONArray("actions"); for (int i = 0; i < jsonActions.length(); i++) { JSONObject jsonAction = jsonActions.getJSONObject(i); - EpisodeAction episodeAction = EpisodeAction.readFromJSONObject(jsonAction); + EpisodeAction episodeAction = EpisodeAction.readFromJsonObject(jsonAction); if (episodeAction != null) { episodeActions.add(episodeAction); } diff --git a/core/src/main/java/de/danoeh/antennapod/core/sync/model/EpisodeAction.java b/core/src/main/java/de/danoeh/antennapod/core/sync/model/EpisodeAction.java index a8eeedd03..6154ccc84 100644 --- a/core/src/main/java/de/danoeh/antennapod/core/sync/model/EpisodeAction.java +++ b/core/src/main/java/de/danoeh/antennapod/core/sync/model/EpisodeAction.java @@ -3,6 +3,7 @@ package de.danoeh.antennapod.core.sync.model; import android.text.TextUtils; import android.util.Log; +import androidx.core.util.ObjectsCompat; import de.danoeh.antennapod.core.feed.FeedItem; import de.danoeh.antennapod.core.util.DateUtils; import org.json.JSONException; @@ -45,7 +46,7 @@ public class EpisodeAction { * @param object JSON representation * @return episode action object, or null if mandatory values are missing */ - public static EpisodeAction readFromJSONObject(JSONObject object) { + public static EpisodeAction readFromJsonObject(JSONObject object) { String podcast = object.optString("podcast", null); String episode = object.optString("episode", null); String actionString = object.optString("action", null); @@ -98,7 +99,7 @@ public class EpisodeAction { } /** - * Returns the position (in seconds) at which the client started playback + * Returns the position (in seconds) at which the client started playback. * * @return start position (in seconds) */ @@ -107,7 +108,7 @@ public class EpisodeAction { } /** - * Returns the position (in seconds) at which the client stopped playback + * Returns the position (in seconds) at which the client stopped playback. * * @return stop position (in seconds) */ @@ -126,21 +127,18 @@ public class EpisodeAction { @Override public boolean equals(Object o) { - if (this == o) return true; - if (o == null || !(o instanceof EpisodeAction)) return false; + if (this == o) { + return true; + } + if (!(o instanceof EpisodeAction)) { + return false; + } EpisodeAction that = (EpisodeAction) o; - - if (started != that.started) return false; - if (position != that.position) return false; - if (total != that.total) return false; - if (podcast != null ? !podcast.equals(that.podcast) : that.podcast != null) return false; - if (episode != null ? !episode.equals(that.episode) : that.episode != null) return false; - //if (deviceId != null ? !deviceId.equals(that.deviceId) : that.deviceId != null) - // return false; - if (action != that.action) return false; - return !(timestamp != null ? !timestamp.equals(that.timestamp) : that.timestamp != null); - + return started == that.started && position == that.position && total == that.total && action != that.action + && ObjectsCompat.equals(podcast, that.podcast) + && ObjectsCompat.equals(episode, that.episode) + && ObjectsCompat.equals(timestamp, that.timestamp); } @Override @@ -156,11 +154,11 @@ public class EpisodeAction { } /** - * Returns a JSON object representation of this object + * Returns a JSON object representation of this object. * * @return JSON object representation, or null if the object is invalid */ - public JSONObject writeToJSONObject() { + public JSONObject writeToJsonObject() { JSONObject obj = new JSONObject(); try { obj.putOpt("podcast", this.podcast); @@ -183,15 +181,15 @@ public class EpisodeAction { @Override public String toString() { - return "EpisodeAction{" + - "podcast='" + podcast + '\'' + - ", episode='" + episode + '\'' + - ", action=" + action + - ", timestamp=" + timestamp + - ", started=" + started + - ", position=" + position + - ", total=" + total + - '}'; + return "EpisodeAction{" + + "podcast='" + podcast + '\'' + + ", episode='" + episode + '\'' + + ", action=" + action + + ", timestamp=" + timestamp + + ", started=" + started + + ", position=" + position + + ", total=" + total + + '}'; } public enum Action { diff --git a/core/src/main/java/de/danoeh/antennapod/core/sync/model/EpisodeActionChanges.java b/core/src/main/java/de/danoeh/antennapod/core/sync/model/EpisodeActionChanges.java index 69c1b5280..77942ffa0 100644 --- a/core/src/main/java/de/danoeh/antennapod/core/sync/model/EpisodeActionChanges.java +++ b/core/src/main/java/de/danoeh/antennapod/core/sync/model/EpisodeActionChanges.java @@ -25,9 +25,9 @@ public class EpisodeActionChanges { @Override public String toString() { - return "EpisodeActionGetResponse{" + - "episodeActions=" + episodeActions + - ", timestamp=" + timestamp + - '}'; + return "EpisodeActionGetResponse{" + + "episodeActions=" + episodeActions + + ", timestamp=" + timestamp + + '}'; } }