Skip to content

Commit 7fe5abc

Browse files
committed
blocked channels: remove immediately
1 parent 4443622 commit 7fe5abc

File tree

3 files changed

+10
-22
lines changed

3 files changed

+10
-22
lines changed

common/src/main/java/com/liskovsoft/smartyoutubetv2/common/app/models/data/Video.java

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -311,7 +311,7 @@ public String getCardImageUrl() {
311311
}
312312

313313
public String getAuthor() {
314-
if (author != null && !Helpers.startsWith(author, "@")) {
314+
if (author != null && !Helpers.startsWith(author, "@")) { // not a channel handle (alias)
315315
return extractAuthor(author);
316316
}
317317

common/src/main/java/com/liskovsoft/smartyoutubetv2/common/app/presenters/dialogs/menu/VideoMenuPresenter.java

Lines changed: 8 additions & 20 deletions
Original file line numberDiff line numberDiff line change
@@ -407,7 +407,7 @@ private void appendBlockChannelButton() {
407407
String channelId = mVideo.channelId;
408408
String channelName = mVideo.getAuthor();
409409

410-
if (Helpers.allNulls(channelId, channelName)) {
410+
if (channelName == null) {
411411
return;
412412
}
413413

@@ -424,27 +424,15 @@ private void appendBlockChannelButton() {
424424
// Remove from blacklist
425425
blockedChannelData.removeChannel(channelId, channelName);
426426
MessageHelpers.showMessage(getContext(), R.string.channel_unblocked);
427-
if (mCallback != null) {
428-
mCallback.onItemAction(mVideo, VideoMenuCallback.ACTION_REMOVE);
429-
}
430-
mDialogPresenter.closeDialog();
431427
} else {
432-
// Show confirmation dialog before blocking
433-
String confirmMessage = getContext().getString(R.string.confirm_block_channel, channelName);
434-
435-
AppDialogUtil.showConfirmationDialog(
436-
getContext(),
437-
confirmMessage,
438-
() -> {
439-
blockedChannelData.addChannel(channelId, channelName);
440-
MessageHelpers.showMessage(getContext(), R.string.channel_blocked);
441-
442-
if (mCallback != null) {
443-
mCallback.onItemAction(mVideo, VideoMenuCallback.ACTION_REMOVE);
444-
}
445-
mDialogPresenter.closeDialog();
446-
});
428+
blockedChannelData.addChannel(channelId, channelName);
429+
MessageHelpers.showMessage(getContext(), R.string.channel_blocked);
447430
}
431+
432+
if (mCallback != null) {
433+
mCallback.onItemAction(mVideo, VideoMenuCallback.ACTION_REMOVE);
434+
}
435+
mDialogPresenter.closeDialog();
448436
}));
449437
}
450438

common/src/main/java/com/liskovsoft/smartyoutubetv2/common/prefs/BlockedChannelData.java

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -96,7 +96,7 @@ public void addChannel(String channelId, String channelName) {
9696

9797
Channel channel = new Channel(channelId, channelName);
9898
mChannels.remove(channel);
99-
mChannels.add(channel);
99+
mChannels.add(0, channel);
100100

101101
persistState();
102102
notifyListeners();

0 commit comments

Comments
 (0)