Merge pull request #5793 from ByteHamster/better-download-notification
Simplify download notification title
This commit is contained in:
commit
f6a7985901
|
@ -52,18 +52,41 @@ public class DownloadServiceNotification {
|
|||
return null;
|
||||
}
|
||||
|
||||
String contentTitle = context.getString(R.string.download_notification_title);
|
||||
String downloadsLeft = (downloads.size() > 0)
|
||||
String contentTitle;
|
||||
if (typeIsOnly(downloads, Feed.FEEDFILETYPE_FEED)) {
|
||||
contentTitle = context.getString(R.string.download_notification_title_feeds);
|
||||
} else if (typeIsOnly(downloads, FeedMedia.FEEDFILETYPE_FEEDMEDIA)) {
|
||||
contentTitle = context.getString(R.string.download_notification_title_episodes);
|
||||
} else {
|
||||
contentTitle = context.getString(R.string.download_notification_title);
|
||||
}
|
||||
String contentText = (downloads.size() > 0)
|
||||
? context.getResources().getQuantityString(R.plurals.downloads_left, downloads.size(), downloads.size())
|
||||
: context.getString(R.string.completing);
|
||||
String bigText = compileNotificationString(downloads);
|
||||
if (!bigText.contains("\n")) {
|
||||
contentText = bigText;
|
||||
}
|
||||
|
||||
notificationCompatBuilder.setContentTitle(contentTitle);
|
||||
notificationCompatBuilder.setContentText(downloadsLeft);
|
||||
notificationCompatBuilder.setContentText(contentText);
|
||||
notificationCompatBuilder.setStyle(new NotificationCompat.BigTextStyle().bigText(bigText));
|
||||
return notificationCompatBuilder.build();
|
||||
}
|
||||
|
||||
private boolean typeIsOnly(List<Downloader> downloads, int feedFileType) {
|
||||
for (Downloader downloader : downloads) {
|
||||
if (downloader.cancelled) {
|
||||
continue;
|
||||
}
|
||||
DownloadRequest request = downloader.getDownloadRequest();
|
||||
if (request.getFeedfileType() != feedFileType) {
|
||||
return false;
|
||||
}
|
||||
}
|
||||
return true;
|
||||
}
|
||||
|
||||
private static String compileNotificationString(List<Downloader> downloads) {
|
||||
StringBuilder stringBuilder = new StringBuilder();
|
||||
for (int i = 0; i < downloads.size(); i++) {
|
||||
|
|
|
@ -279,6 +279,8 @@
|
|||
</plurals>
|
||||
<string name="completing">Completing…</string>
|
||||
<string name="download_notification_title">Downloading podcast data</string>
|
||||
<string name="download_notification_title_feeds">Refreshing podcasts</string>
|
||||
<string name="download_notification_title_episodes">Downloading episodes</string>
|
||||
<string name="download_log_title_unknown">Unknown Title</string>
|
||||
<string name="download_type_feed">Feed</string>
|
||||
<string name="download_type_media">Media file</string>
|
||||
|
|
Loading…
Reference in New Issue