Fixes Improved
This commit is contained in:
parent
bfc987f81b
commit
fd34b1a291
|
@ -16,8 +16,6 @@ import static org.schabi.newpipe.database.playlist.model.PlaylistRemoteEntity.RE
|
||||||
import static org.schabi.newpipe.database.playlist.model.PlaylistRemoteEntity.REMOTE_PLAYLIST_TABLE;
|
import static org.schabi.newpipe.database.playlist.model.PlaylistRemoteEntity.REMOTE_PLAYLIST_TABLE;
|
||||||
import static org.schabi.newpipe.database.playlist.model.PlaylistRemoteEntity.REMOTE_PLAYLIST_URL;
|
import static org.schabi.newpipe.database.playlist.model.PlaylistRemoteEntity.REMOTE_PLAYLIST_URL;
|
||||||
|
|
||||||
import android.text.TextUtils;
|
|
||||||
|
|
||||||
@Entity(tableName = REMOTE_PLAYLIST_TABLE,
|
@Entity(tableName = REMOTE_PLAYLIST_TABLE,
|
||||||
indices = {
|
indices = {
|
||||||
@Index(value = {REMOTE_PLAYLIST_NAME}),
|
@Index(value = {REMOTE_PLAYLIST_NAME}),
|
||||||
|
@ -75,19 +73,18 @@ public class PlaylistRemoteEntity implements PlaylistLocalItem {
|
||||||
@Ignore
|
@Ignore
|
||||||
public boolean isIdenticalTo(final PlaylistInfo info) {
|
public boolean isIdenticalTo(final PlaylistInfo info) {
|
||||||
/*
|
/*
|
||||||
// Returns boolean comparing the online playlist and the local copy.
|
* Returns boolean comparing the online playlist and the local copy.
|
||||||
// (False if info changed such as playlist name or track count)
|
* (False if info changed such as playlist name or track count)
|
||||||
// [Note that we don't check if the playlist creator changed:
|
* [Note that
|
||||||
// getUploader().equals(info.getUploaderName())
|
* getUploader().equals(info.getUploaderName())
|
||||||
// because this would crash the app on playlists that are auto-generated with no creator,
|
* crashes the app on playlists that are auto-generated with no creator,
|
||||||
// and the creator can not change to my knowledge.
|
* please use
|
||||||
// if you need this functionality back please use
|
* getUploader() == info.getUploaderName()
|
||||||
// getUploader() == info.getUploaderName()
|
* instead as it will work with blank names (Null value if I remember correctly).
|
||||||
// instead as it will work with blank names (Null value if I remember correctly).
|
|
||||||
*/
|
*/
|
||||||
return getServiceId() == info.getServiceId() && getName().equals(info.getName()) &&
|
return getServiceId() == info.getServiceId() && getName().equals(info.getName()) &&
|
||||||
getStreamCount() == info.getStreamCount() && getUrl().equals(info.getUrl()) &&
|
getStreamCount() == info.getStreamCount() && getUrl().equals(info.getUrl()) &&
|
||||||
getThumbnailUrl().equals(info.getThumbnailUrl());
|
getThumbnailUrl().equals(info.getThumbnailUrl()) && getUploader() == info.getUploaderName();
|
||||||
}
|
}
|
||||||
|
|
||||||
public long getUid() {
|
public long getUid() {
|
||||||
|
|
Loading…
Reference in New Issue