diff --git a/app/src/main/java/com/dkanada/gramophone/adapter/SearchAdapter.java b/app/src/main/java/com/dkanada/gramophone/adapter/SearchAdapter.java index 92069726..8ea2dd53 100644 --- a/app/src/main/java/com/dkanada/gramophone/adapter/SearchAdapter.java +++ b/app/src/main/java/com/dkanada/gramophone/adapter/SearchAdapter.java @@ -67,7 +67,7 @@ public class SearchAdapter extends RecyclerView.Adapter getSongList(@NonNull List artists) { final List songs = new ArrayList<>(); for (Artist artist : artists) { - songs.addAll(artist.getSongs()); + songs.addAll(artist.songs); } return songs; @@ -176,7 +176,7 @@ public class ArtistAdapter extends AbsMultiSelectAdapter(); } - public String getId() { - return id; - } - - public String getTitle() { - return title; - } - - public String getArtistId() { - return artistId; - } - - public String getArtistName() { - return artistName; - } - - public int getYear() { - return year; - } - - public int getSongCount() { - return songs.size(); - } - @Override public boolean equals(Object o) { if (this == o) return true; diff --git a/app/src/main/java/com/dkanada/gramophone/model/Artist.java b/app/src/main/java/com/dkanada/gramophone/model/Artist.java index 4d06a91f..cab041b0 100644 --- a/app/src/main/java/com/dkanada/gramophone/model/Artist.java +++ b/app/src/main/java/com/dkanada/gramophone/model/Artist.java @@ -52,33 +52,13 @@ public class Artist implements Parcelable { this.songs = new ArrayList<>(); } - public String getId() { - return id; - } - - public String getName() { - return name; - } - - public int getSongCount() { - return songs.size(); - } - - public int getAlbumCount() { - return albums.size(); - } - - public List getSongs() { - return songs; - } - @Override public boolean equals(Object o) { if (this == o) return true; if (o == null || getClass() != o.getClass()) return false; Artist artist = (Artist) o; - return id.equals(artist.getId()); + return id.equals(artist.id); } @Override diff --git a/app/src/main/java/com/dkanada/gramophone/ui/activities/AlbumDetailActivity.java b/app/src/main/java/com/dkanada/gramophone/ui/activities/AlbumDetailActivity.java index 663ab55a..06acd816 100644 --- a/app/src/main/java/com/dkanada/gramophone/ui/activities/AlbumDetailActivity.java +++ b/app/src/main/java/com/dkanada/gramophone/ui/activities/AlbumDetailActivity.java @@ -328,11 +328,11 @@ public class AlbumDetailActivity extends AbsSlidingMusicPanelActivity implements private void setAlbum(Album album) { this.album = album; - getSupportActionBar().setTitle(album.getTitle()); - artistTextView.setText(album.getArtistName()); - songCountTextView.setText(MusicUtil.getSongCountString(this, album.getSongCount())); + getSupportActionBar().setTitle(album.title); + artistTextView.setText(album.artistName); + songCountTextView.setText(MusicUtil.getSongCountString(this, album.songs.size())); durationTextView.setText(MusicUtil.getReadableDurationString(MusicUtil.getTotalDuration(this, album.songs))); - albumYearTextView.setText(MusicUtil.getYearString(album.getYear())); + albumYearTextView.setText(MusicUtil.getYearString(album.year)); if (album.songs.size() != 0) adapter.swapDataSet(album.songs); } diff --git a/app/src/main/java/com/dkanada/gramophone/ui/activities/ArtistDetailActivity.java b/app/src/main/java/com/dkanada/gramophone/ui/activities/ArtistDetailActivity.java index 810b14b8..6c8dbdb8 100644 --- a/app/src/main/java/com/dkanada/gramophone/ui/activities/ArtistDetailActivity.java +++ b/app/src/main/java/com/dkanada/gramophone/ui/activities/ArtistDetailActivity.java @@ -171,7 +171,7 @@ public class ArtistDetailActivity extends AbsSlidingMusicPanelActivity implement songListView.setScrollViewCallbacks(observableScrollViewCallbacks); songListView.addHeaderView(songListHeader); - songAdapter = new ArtistSongAdapter(this, getArtist().getSongs(), this); + songAdapter = new ArtistSongAdapter(this, getArtist().songs, this); songListView.setAdapter(songAdapter); final View contentView = getWindow().getDecorView().findViewById(android.R.id.content); @@ -350,10 +350,10 @@ public class ArtistDetailActivity extends AbsSlidingMusicPanelActivity implement private void setArtist(Artist artist) { this.artist = artist; - getSupportActionBar().setTitle(artist.getName()); - songCountTextView.setText(MusicUtil.getSongCountString(this, artist.getSongCount())); - albumCountTextView.setText(MusicUtil.getAlbumCountString(this, artist.getAlbumCount())); - durationTextView.setText(MusicUtil.getReadableDurationString(MusicUtil.getTotalDuration(this, artist.getSongs()))); + getSupportActionBar().setTitle(artist.name); + songCountTextView.setText(MusicUtil.getSongCountString(this, artist.songs.size())); + albumCountTextView.setText(MusicUtil.getAlbumCountString(this, artist.albums.size())); + durationTextView.setText(MusicUtil.getReadableDurationString(MusicUtil.getTotalDuration(this, artist.songs))); // TODO this activity will crash when an artist is passed with an empty album array // something in the album adapter is causing the issue