From b1fdf202e706b1309c2e034432e28f5bd67f50fa Mon Sep 17 00:00:00 2001 From: zkwip Date: Wed, 10 Jun 2015 23:10:41 +0200 Subject: Removed todos if not needed anymore --- app/src/main/java/org/rssin/android/FeedsActivity.java | 2 +- app/src/main/java/org/rssin/android/FilterSettingsActivity.java | 4 ++-- app/src/main/java/org/rssin/android/InternalStorageProvider.java | 1 - .../java/org/rssin/android/NavigationDrawerManageFeedsFragment.java | 2 +- .../java/org/rssin/android/NavigationDrawerManageFiltersFragment.java | 2 +- app/src/main/java/org/rssin/rssin/Feed.java | 4 ++-- app/src/main/java/org/rssin/rssin/Filter.java | 2 +- 7 files changed, 8 insertions(+), 9 deletions(-) (limited to 'app/src') diff --git a/app/src/main/java/org/rssin/android/FeedsActivity.java b/app/src/main/java/org/rssin/android/FeedsActivity.java index a1b9656..1ac091e 100755 --- a/app/src/main/java/org/rssin/android/FeedsActivity.java +++ b/app/src/main/java/org/rssin/android/FeedsActivity.java @@ -116,7 +116,7 @@ public class FeedsActivity extends ActionBarActivity { /** * Open dialog to add new feed * For the moment, we temporarily disable rotating because we can't get it working otherwise. - * @todo make rotating possible + * Possible feature: make rotating possible */ public void openAddDialog() { setRequestedOrientation(ActivityInfo.SCREEN_ORIENTATION_NOSENSOR); diff --git a/app/src/main/java/org/rssin/android/FilterSettingsActivity.java b/app/src/main/java/org/rssin/android/FilterSettingsActivity.java index 93bffdf..ae0327a 100755 --- a/app/src/main/java/org/rssin/android/FilterSettingsActivity.java +++ b/app/src/main/java/org/rssin/android/FilterSettingsActivity.java @@ -72,7 +72,7 @@ public class FilterSettingsActivity extends ActionBarActivity { Intent intent = getIntent(); int filterHashCode = intent.getIntExtra("filter", -1); - // @todo Check on -1? Shouldn't happen anyway. + // Check on -1? Shouldn't happen anyway. filter = filtersList.getFilterFromHashCode(filterHashCode); setTitle(); @@ -205,7 +205,7 @@ public class FilterSettingsActivity extends ActionBarActivity { /** * Open dialog to edit title * For the moment, we temporarily disable rotating because we can't get it working otherwise. - * @todo make rotating possible + * Possible feature: make rotating possible */ public void openTitleDialog() { setRequestedOrientation(ActivityInfo.SCREEN_ORIENTATION_NOSENSOR); diff --git a/app/src/main/java/org/rssin/android/InternalStorageProvider.java b/app/src/main/java/org/rssin/android/InternalStorageProvider.java index 6241c2d..a09eefa 100644 --- a/app/src/main/java/org/rssin/android/InternalStorageProvider.java +++ b/app/src/main/java/org/rssin/android/InternalStorageProvider.java @@ -46,7 +46,6 @@ class InternalStorageProvider implements StorageProvider, Fi private final Context context; - // @todo redo singleton public InternalStorageProvider(Context context) { this.context = context; } diff --git a/app/src/main/java/org/rssin/android/NavigationDrawerManageFeedsFragment.java b/app/src/main/java/org/rssin/android/NavigationDrawerManageFeedsFragment.java index 3e52b1b..7f6163a 100755 --- a/app/src/main/java/org/rssin/android/NavigationDrawerManageFeedsFragment.java +++ b/app/src/main/java/org/rssin/android/NavigationDrawerManageFeedsFragment.java @@ -117,7 +117,7 @@ public class NavigationDrawerManageFeedsFragment extends Fragment { /** * Open dialog to add new feed * For the moment, we temporarily disable rotating because we can't get it working otherwise. - * @todo make rotating possible + * Possible feature: make rotating possible */ public void openAddDialog() { //setRequestedOrientation(ActivityInfo.SCREEN_ORIENTATION_NOSENSOR); diff --git a/app/src/main/java/org/rssin/android/NavigationDrawerManageFiltersFragment.java b/app/src/main/java/org/rssin/android/NavigationDrawerManageFiltersFragment.java index 84d9a0d..a0d9b55 100644 --- a/app/src/main/java/org/rssin/android/NavigationDrawerManageFiltersFragment.java +++ b/app/src/main/java/org/rssin/android/NavigationDrawerManageFiltersFragment.java @@ -124,7 +124,7 @@ public class NavigationDrawerManageFiltersFragment extends Fragment { /** * Open dialog to create new filter * For the moment, we temporarily disable rotating because we can't get it working otherwise. - * @todo make rotating possible + * Possible feature: make rotating possible */ public void openAddDialog() { //setRequestedOrientation(ActivityInfo.SCREEN_ORIENTATION_NOSENSOR); diff --git a/app/src/main/java/org/rssin/rssin/Feed.java b/app/src/main/java/org/rssin/rssin/Feed.java index 75b5616..dc56a8f 100755 --- a/app/src/main/java/org/rssin/rssin/Feed.java +++ b/app/src/main/java/org/rssin/rssin/Feed.java @@ -39,7 +39,7 @@ import java.net.URL; /** * Feed holder * @author Camil Staps - * @todo Allow users to enter links without scheme (http:// ...) + * Possible feature: Allow users to enter links without scheme (http:// ...) */ public class Feed implements Storable, Comparable, Jsonable { private static final long serialVersionUID = 0; @@ -125,7 +125,7 @@ public class Feed implements Storable, Comparable, Jsonable { /** * Cleverly make up a title based on the URL - * @todo make it really really clever + * Possible feature: make it really really clever */ public void setTitleFromURL(Context context, final FeedStorageProvider storageProvider, Fetcher fetcher, final FallibleListener titlehasChanged) { String newTitle = url.getHost(); diff --git a/app/src/main/java/org/rssin/rssin/Filter.java b/app/src/main/java/org/rssin/rssin/Filter.java index ec7a23c..8de2325 100644 --- a/app/src/main/java/org/rssin/rssin/Filter.java +++ b/app/src/main/java/org/rssin/rssin/Filter.java @@ -35,7 +35,7 @@ import java.util.List; * Filter holder * @author Camil Staps * - * @todo Write read & writeObject with JSON or so to make it easier to change this class later + * Possible feature: Write read & writeObject with JSON or so to make it easier to change this class later */ public class Filter implements Storable, Comparable, Jsonable { -- cgit v1.2.3 From 5b9c1503175bad7816cee71ef5d4d206c6ae0eb8 Mon Sep 17 00:00:00 2001 From: Size43 Date: Wed, 10 Jun 2015 23:11:42 +0200 Subject: uniqueKey() will not return the same key twice --- .../org/rssin/android/SharedPreferencesStorageProvider.java | 13 ++++++++++++- app/src/main/java/org/rssin/rssin/Feed.java | 1 + 2 files changed, 13 insertions(+), 1 deletion(-) mode change 100644 => 100755 app/src/main/java/org/rssin/android/SharedPreferencesStorageProvider.java (limited to 'app/src') diff --git a/app/src/main/java/org/rssin/android/SharedPreferencesStorageProvider.java b/app/src/main/java/org/rssin/android/SharedPreferencesStorageProvider.java old mode 100644 new mode 100755 index 40863fd..dd34e63 --- a/app/src/main/java/org/rssin/android/SharedPreferencesStorageProvider.java +++ b/app/src/main/java/org/rssin/android/SharedPreferencesStorageProvider.java @@ -119,9 +119,20 @@ class SharedPreferencesStorageProvider implements StorageProvider, FilterStorage .apply(); } + + private long lastKey = -1; @Override public Object uniqueKey() { - return Long.toString(System.currentTimeMillis()); + long key = System.currentTimeMillis(); + + // Make sure the same key isn't returned twice. + if(key <= lastKey) + { + key = lastKey + 1; + } + + lastKey = key; + return Long.toString(key); } @Override diff --git a/app/src/main/java/org/rssin/rssin/Feed.java b/app/src/main/java/org/rssin/rssin/Feed.java index 75b5616..c126ceb 100755 --- a/app/src/main/java/org/rssin/rssin/Feed.java +++ b/app/src/main/java/org/rssin/rssin/Feed.java @@ -19,6 +19,7 @@ package org.rssin.rssin; import android.content.Context; +import android.util.Log; import org.json.JSONException; import org.json.JSONObject; -- cgit v1.2.3