Better user feedback on location button presses; more reliable map po…

…int retrieval
StAResComp · Jun 6, 2018 · d5e395e · d5e395e
1 parent c7e1556
commit d5e395e
Show file tree

Hide file tree

Showing 3 changed files with 10 additions and 4 deletions.
diff --git a/catch/src/main/java/uk/ac/masts/sifids/activities/Fish1FormsActivity.java b/catch/src/main/java/uk/ac/masts/sifids/activities/Fish1FormsActivity.java
@@ -164,15 +164,21 @@ public void startTrackingLocation(View v) {
 
     public void stopTrackingLocation(View v) {
         stopService(new Intent(this, CatchLocationService.class));
+        Toast.makeText(getBaseContext(), getString(R.string.stopped_tracking_location),
+                Toast.LENGTH_LONG).show();
     }
 
     public void startFishing(View v) {
         ((CatchApplication) this.getApplication()).setFishing(true);
         this.startTrackingLocation(v);
+        Toast.makeText(getBaseContext(), getString(R.string.started_fishing),
+                Toast.LENGTH_LONG).show();
     }
 
     public void stopFishing(View v) {
         ((CatchApplication) this.getApplication()).setFishing(false);
+        Toast.makeText(getBaseContext(), getString(R.string.stopped_fishing),
+                Toast.LENGTH_LONG).show();
     }
 
     private void startLocationService() {

diff --git a/catch/src/main/java/uk/ac/masts/sifids/activities/MapActivity.java b/catch/src/main/java/uk/ac/masts/sifids/activities/MapActivity.java
@@ -107,10 +107,7 @@ private void getPoints() {
                 @Override
                 public void run() {
                     Calendar cal = Calendar.getInstance();
-                    cal.set(Calendar.DAY_OF_WEEK, Calendar.SUNDAY);
-                    if (cal.after(new Date())) {
-                        cal.add(Calendar.DATE, -7);
-                    }
+                    cal.add(Calendar.DATE, -1 * (cal.get(Calendar.DAY_OF_WEEK) - 1));
                     cal.set(Calendar.HOUR_OF_DAY, 0);
                     cal.set(Calendar.MINUTE, 0);
                     cal.set(Calendar.SECOND, 0);

diff --git a/catch/src/main/res/values/strings.xml b/catch/src/main/res/values/strings.xml
@@ -41,6 +41,9 @@
     <string name="stop_fishing">Stop Fishing</string>
     <string name="submit_track">Submit last 1000 location points</string>
     <string name="fish_1_form_summary">%s\n%s -\n%s</string>
+    <string name="stopped_tracking_location">Stopped tracking location</string>
+    <string name="started_fishing">Recorded location information will now indicate that you are fishing</string>
+    <string name="stopped_fishing">Recorded location information will no longer indicate that you are fishing</string>
 
     <!-- FISH1 Form Creation -->