diff options
author | orionlee <orionlee@yahoo.com> | 2019-09-22 13:28:25 -0700 |
---|---|---|
committer | orionlee <orionlee@yahoo.com> | 2019-09-22 13:28:25 -0700 |
commit | 9c8eda170dbd4c19f6b096e5553b8166b64c2f29 (patch) | |
tree | 0f4a7b21a676d5c3c65cc1c4e1ecfd5a6aaaba7b /app/src/main/java/de/danoeh | |
parent | 427477b2562bd2ea8a4a3d6a01b923303c496e53 (diff) | |
download | AntennaPod-9c8eda170dbd4c19f6b096e5553b8166b64c2f29.zip |
#2929 - review feedback: non-null feedItem requirement explicit
Diffstat (limited to 'app/src/main/java/de/danoeh')
-rw-r--r-- | app/src/main/java/de/danoeh/antennapod/activity/MediaplayerActivity.java | 10 |
1 files changed, 5 insertions, 5 deletions
diff --git a/app/src/main/java/de/danoeh/antennapod/activity/MediaplayerActivity.java b/app/src/main/java/de/danoeh/antennapod/activity/MediaplayerActivity.java index 1b6fa6bce..def2252d4 100644 --- a/app/src/main/java/de/danoeh/antennapod/activity/MediaplayerActivity.java +++ b/app/src/main/java/de/danoeh/antennapod/activity/MediaplayerActivity.java @@ -391,7 +391,7 @@ public abstract class MediaplayerActivity extends CastEnabledActivity implements return true; } else { if (media != null) { - final FeedItem feedItem = getFeedItem(media); // some options option requires FeedItem + final @Nullable FeedItem feedItem = getFeedItem(media); // some options option requires FeedItem switch (item.getItemId()) { case R.id.add_to_favorites_item: if (feedItem != null) { @@ -456,22 +456,22 @@ public abstract class MediaplayerActivity extends CastEnabledActivity implements startActivity(new Intent(Intent.ACTION_VIEW, uri)); break; case R.id.share_link_item: - if (media instanceof FeedMedia) { + if (feedItem != null) { ShareUtils.shareFeedItemLink(this, feedItem); } break; case R.id.share_download_url_item: - if (media instanceof FeedMedia) { + if (feedItem != null) { ShareUtils.shareFeedItemDownloadLink(this, feedItem); } break; case R.id.share_link_with_position_item: - if (media instanceof FeedMedia) { + if (feedItem != null) { ShareUtils.shareFeedItemLink(this, feedItem, true); } break; case R.id.share_download_url_with_position_item: - if (media instanceof FeedMedia) { + if (feedItem != null) { ShareUtils.shareFeedItemDownloadLink(this, feedItem, true); } break; |