baalajimaestro/NewPipe
1
0
Fork 0
Code Issues Pull requests Projects Releases 7 Packages Wiki Activity

Better FIx, Improved Comment Style

Browse source
This commit is contained in:
Peter Hindes 2019-10-15 07:19:11 -06:00
parent c93c52a58c
commit c68c35e084
1 changed files with 10 additions and 13 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

21
app/src/main/java/org/schabi/newpipe/database/playlist/model/PlaylistRemoteEntity.java
View file

@ -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() {