1 files changed, 14 insertions, 20 deletions

diff --git a/main/src/cgeo/geocaching/LogCacheActivity.java b/main/src/cgeo/geocaching/LogCacheActivity.java
index c739067..cb565bd 100644
--- a/main/src/cgeo/geocaching/LogCacheActivity.java
+++ b/main/src/cgeo/geocaching/LogCacheActivity.java
@@ -46,7 +46,6 @@ import java.util.ArrayList;
 import java.util.Calendar;
 import java.util.Date;
 import java.util.List;
-import java.util.Locale;
 
 public class LogCacheActivity extends AbstractLoggingActivity implements DateDialog.DateDialogParent {
     static final String EXTRAS_GEOCODE = "geocode";
@@ -79,7 +78,7 @@ public class LogCacheActivity extends AbstractLoggingActivity implements DateDia
     private ILoggingManager loggingManager;
 
     // Data to be saved while reconfiguring
-    private double rating;
+    private float rating;
     private LogType typeSelected;
     private Calendar date;
     private String imageCaption;
@@ -214,13 +213,13 @@ public class LogCacheActivity extends AbstractLoggingActivity implements DateDia
         if (!postButton.isEnabled()) {
             return res.getString(R.string.log_post_not_possible);
         }
-        if (!Settings.isGCvoteLogin() || !cache.supportsGCVote()) {
+        if (!GCVote.isVotingPossible(cache)) {
             return res.getString(R.string.log_post);
         }
-        if (rating == 0) {
-            return res.getString(R.string.log_post_no_rate);
+        if (GCVote.isValidRating(rating)) {
+            return res.getString(R.string.log_post_rate) + " " + GCVote.getRatingText(rating) + "*";
         }
-        return res.getString(R.string.log_post_rate) + " " + ratingTextValue(rating) + "*";
+        return res.getString(R.string.log_post_no_rate);
     }
 
     @Override
@@ -261,7 +260,7 @@ public class LogCacheActivity extends AbstractLoggingActivity implements DateDia
 
         // Restore previous state
         if (savedInstanceState != null) {
-            rating = savedInstanceState.getDouble(SAVED_STATE_RATING);
+            rating = savedInstanceState.getFloat(SAVED_STATE_RATING);
             typeSelected = LogType.getById(savedInstanceState.getInt(SAVED_STATE_TYPE));
             date.setTimeInMillis(savedInstanceState.getLong(SAVED_STATE_DATE));
             imageCaption = savedInstanceState.getString(SAVED_STATE_IMAGE_CAPTION);
@@ -341,7 +340,7 @@ public class LogCacheActivity extends AbstractLoggingActivity implements DateDia
 
     private void setDefaultValues() {
         date = Calendar.getInstance();
-        rating = 0.0;
+        rating = GCVote.NO_RATING;
         if (cache.isEventCache()) {
             final Date eventDate = cache.getHiddenDate();
             boolean expired = DateUtils.daysSince(eventDate.getTime()) >= 0;
@@ -424,8 +423,7 @@ public class LogCacheActivity extends AbstractLoggingActivity implements DateDia
     public boolean onPrepareOptionsMenu(Menu menu) {
         super.onPrepareOptionsMenu(menu);
 
-        final boolean voteAvailable = Settings.isGCvoteLogin() && StringUtils.isNotBlank(cache.getGuid()) && cache.supportsGCVote();
-        menu.findItem(SUBMENU_VOTE).setVisible(voteAvailable);
+        menu.findItem(SUBMENU_VOTE).setVisible(GCVote.isVotingPossible(cache));
 
         return true;
     }
@@ -438,9 +436,9 @@ public class LogCacheActivity extends AbstractLoggingActivity implements DateDia
 
         final int id = item.getItemId();
         if (id >= 10 && id <= 19) {
-            rating = (id - 9) / 2.0;
-            if (rating < 1) {
-                rating = 0;
+            rating = (id - 9) / 2.0f;
+            if (!GCVote.isValidRating(rating)) {
+                rating = GCVote.NO_RATING;
             }
             updatePostButtonText();
             return true;
@@ -449,10 +447,6 @@ public class LogCacheActivity extends AbstractLoggingActivity implements DateDia
         return false;
     }
 
-    private static String ratingTextValue(final double rating) {
-        return String.format(Locale.getDefault(), "%.1f", rating);
-    }
-
     @Override
     protected void onSaveInstanceState(final Bundle outState) {
         super.onSaveInstanceState(outState);
@@ -539,7 +533,7 @@ public class LogCacheActivity extends AbstractLoggingActivity implements DateDia
                 final LogResult logResult = loggingManager.postLog(cache, typeSelected, date, log, logPwd, trackables);
 
                 if (logResult.getPostLogResult() == StatusCode.NO_ERROR) {
-                    final LogEntry logNow = new LogEntry(date, typeSelected, log);
+                    final LogEntry logNow = new LogEntry(date.getTimeInMillis(), typeSelected, log);
 
                     cache.getLogs().add(0, logNow);
 
@@ -556,7 +550,7 @@ public class LogCacheActivity extends AbstractLoggingActivity implements DateDia
                             Twitter.postTweetCache(geocode);
                         }
                     }
-                    if (rating > 0) {
+                    if (GCVote.isValidRating(rating)) {
                         GCVote.setRating(cache, rating);
                     }
 
@@ -572,7 +566,7 @@ public class LogCacheActivity extends AbstractLoggingActivity implements DateDia
                 }
 
                 return logResult.getPostLogResult();
-            } catch (Exception e) {
+            } catch (RuntimeException e) {
                 Log.e("VisitCacheActivity.Poster.doInBackgroundInternal", e);
             }