Skip to content

Commit 38d4887

Browse files
committed
Undo some unneeded changes to LocalPlaylistManager
1 parent c9051d3 commit 38d4887

1 file changed

Lines changed: 3 additions & 12 deletions

File tree

app/src/main/java/org/schabi/newpipe/local/playlist/LocalPlaylistManager.java

Lines changed: 3 additions & 12 deletions
Original file line numberDiff line numberDiff line change
@@ -133,18 +133,13 @@ public Flowable<List<PlaylistStreamEntry>> getPlaylistStreams(final long playlis
133133
}
134134

135135
public Maybe<Integer> renamePlaylist(final long playlistId, final String name) {
136-
return modifyPlaylist(playlistId, name, THUMBNAIL_ID_LEAVE_UNCHANGED, false, -1);
136+
return modifyPlaylist(playlistId, name, THUMBNAIL_ID_LEAVE_UNCHANGED, false);
137137
}
138138

139139
public Maybe<Integer> changePlaylistThumbnail(final long playlistId,
140140
final long thumbnailStreamId,
141141
final boolean isPermanent) {
142-
return modifyPlaylist(playlistId, null, thumbnailStreamId, isPermanent, -1);
143-
}
144-
145-
public Maybe<Integer> updatePlaylistDisplayIndex(final long playlistId,
146-
final long displayIndex) {
147-
return modifyPlaylist(playlistId, null, THUMBNAIL_ID_LEAVE_UNCHANGED, false, displayIndex);
142+
return modifyPlaylist(playlistId, null, thumbnailStreamId, isPermanent);
148143
}
149144

150145
public long getPlaylistThumbnailStreamId(final long playlistId) {
@@ -168,8 +163,7 @@ public long getAutomaticPlaylistThumbnailStreamId(final long playlistId) {
168163
private Maybe<Integer> modifyPlaylist(final long playlistId,
169164
@Nullable final String name,
170165
final long thumbnailStreamId,
171-
final boolean isPermanent,
172-
final long displayIndex) {
166+
final boolean isPermanent) {
173167
return playlistTable.getPlaylist(playlistId)
174168
.firstElement()
175169
.filter(playlistEntities -> !playlistEntities.isEmpty())
@@ -182,9 +176,6 @@ private Maybe<Integer> modifyPlaylist(final long playlistId,
182176
playlist.setThumbnailStreamId(thumbnailStreamId);
183177
playlist.setIsThumbnailPermanent(isPermanent);
184178
}
185-
if (displayIndex != -1) {
186-
playlist.setDisplayIndex(displayIndex);
187-
}
188179
return playlistTable.update(playlist);
189180
}).subscribeOn(Schedulers.io());
190181
}

0 commit comments

Comments
 (0)