From 90e946c4bb60da1e7c7265c32024fca10163d4c1 Mon Sep 17 00:00:00 2001
From: Scott Jackson <daneren2005@gmail.com>
Date: Mon, 8 May 2017 08:45:52 -0700
Subject: Better share error handling

---
 .../java/github/daneren2005/dsub/fragments/SubsonicFragment.java     | 5 +++--
 app/src/main/res/values/strings.xml                                  | 1 +
 2 files changed, 4 insertions(+), 2 deletions(-)

(limited to 'app/src/main')

diff --git a/app/src/main/java/github/daneren2005/dsub/fragments/SubsonicFragment.java b/app/src/main/java/github/daneren2005/dsub/fragments/SubsonicFragment.java
index de230309..a8b21be9 100644
--- a/app/src/main/java/github/daneren2005/dsub/fragments/SubsonicFragment.java
+++ b/app/src/main/java/github/daneren2005/dsub/fragments/SubsonicFragment.java
@@ -1601,7 +1601,7 @@ public class SubsonicFragment extends Fragment implements SwipeRefreshLayout.OnR
 					Share share = shares.get(0);
 					shareExternal(share);
 				} else {
-					Util.toast(context, context.getResources().getString(R.string.playlist_error), false);
+					Util.toast(context, context.getResources().getString(R.string.share_create_error), false);
 				}
 			}
 
@@ -1611,9 +1611,10 @@ public class SubsonicFragment extends Fragment implements SwipeRefreshLayout.OnR
 				if (error instanceof OfflineException || error instanceof ServerTooOldException) {
 					msg = getErrorMessage(error);
 				} else {
-					msg = context.getResources().getString(R.string.playlist_error) + " " + getErrorMessage(error);
+					msg = context.getResources().getString(R.string.share_create_error) + " " + getErrorMessage(error);
 				}
 
+				Log.e(TAG, "Failed to create share", error);
 				Util.toast(context, msg, false);
 			}
 		}.execute();
diff --git a/app/src/main/res/values/strings.xml b/app/src/main/res/values/strings.xml
index b54e4ff2..81ee6c7a 100644
--- a/app/src/main/res/values/strings.xml
+++ b/app/src/main/res/values/strings.xml
@@ -498,6 +498,7 @@
 	<string name="shuffle.genre">Genre:</string>
 	<string name="shuffle.pick_genre">Pick a genre</string>
 
+	<string name="share.create_error">Failed to create share %s</string>
 	<string name="share.expires">Expires: %s</string>
 	<string name="share.expires_never">Never Expires</string>
 	<string name="share.deleted">Deleted share %s</string>
-- 
cgit v1.2.3