Skip to content

Commit 8e631b2

Browse files
committed
Music: Add composer support.
1 parent c810be0 commit 8e631b2

File tree

5 files changed

+33
-19
lines changed

5 files changed

+33
-19
lines changed

JMPDComm/backends/android/src/main/java/org/a0z/mpd/item/MusicParcelable.java

+4-3
Original file line numberDiff line numberDiff line change
@@ -62,9 +62,9 @@ public MusicParcelable(final Music music) {
6262
* @param in The {@link android.os.Parcel} that contains our object
6363
*/
6464
protected MusicParcelable(final Parcel in) {
65-
super(in.readString(), in.readString(), in.readString(), in.readString(), in.readInt(),
66-
in.readLong(), in.readString(), in.readLong(), in.readString(), in.readInt(),
67-
in.readInt(), in.readInt(), in.readInt(), in.readString());
65+
super(in.readString(), in.readString(), in.readString(), in.readString(), in.readString(),
66+
in.readInt(), in.readLong(), in.readString(), in.readLong(), in.readString(),
67+
in.readInt(), in.readInt(), in.readInt(), in.readInt(), in.readString());
6868
}
6969

7070
@Override
@@ -77,6 +77,7 @@ public void writeToParcel(final Parcel dest, final int flags) {
7777
dest.writeString(getAlbum());
7878
dest.writeString(getArtist());
7979
dest.writeString(getAlbumArtist());
80+
dest.writeString(getComposer());
8081
dest.writeString(getFullPath());
8182
dest.writeInt(getDisc());
8283
dest.writeLong(getDate());

JMPDComm/src/main/java/org/a0z/mpd/item/Music.java

+19-6
Original file line numberDiff line numberDiff line change
@@ -104,6 +104,8 @@ public int compare(final Music lhs, final Music rhs) {
104104

105105
private final String mArtist;
106106

107+
private final String mComposer;
108+
107109
private final long mDate;
108110

109111
private final int mDisc;
@@ -130,6 +132,7 @@ public Music() {
130132
this(null, /** Album */
131133
null, /** Artist */
132134
null, /** AlbumArtist */
135+
null, /** Composer */
133136
null, /** FullPath */
134137
UNDEFINED_INT, /** Disc */
135138
-1L, /** Date */
@@ -145,20 +148,21 @@ public Music() {
145148
}
146149

147150
protected Music(final Music music) {
148-
this(music.mAlbum, music.mArtist, music.mAlbumArtist, music.mFullPath, music.mDisc,
149-
music.mDate, music.mGenre, music.mTime, music.mTitle,
151+
this(music.mAlbum, music.mArtist, music.mAlbumArtist, music.mComposer, music.mFullPath,
152+
music.mDisc, music.mDate, music.mGenre, music.mTime, music.mTitle,
150153
music.mTotalTracks, music.mTrack, music.mSongId, music.mSongPos, music.mName);
151154
}
152155

153156
protected Music(final String album, final String artist, final String albumArtist,
154-
final String fullPath, final int disc, final long date, final String genre,
155-
final long time, final String title, final int totalTracks,
157+
final String composer, final String fullPath, final int disc, final long date,
158+
final String genre, final long time, final String title, final int totalTracks,
156159
final int track, final int songId, final int songPos, final String name) {
157160
super();
158161

159162
mAlbum = album;
160163
mArtist = artist;
161164
mAlbumArtist = albumArtist;
165+
mComposer = composer;
162166
mFullPath = fullPath;
163167
mDisc = disc;
164168
mDate = date;
@@ -176,6 +180,7 @@ static Music build(final Collection<String> response) {
176180
String album = null;
177181
String artist = null;
178182
String albumArtist = null;
183+
String composer = null;
179184
String fullPath = null;
180185
int disc = UNDEFINED_INT;
181186
long date = -1L;
@@ -210,6 +215,9 @@ static Music build(final Collection<String> response) {
210215
case "Artist":
211216
artist = pair[VALUE];
212217
break;
218+
case "Composer":
219+
composer = pair[VALUE];
220+
break;
213221
case "Date":
214222
try {
215223
final Matcher matcher = DATE_DELIMITER.matcher(pair[VALUE]);
@@ -289,8 +297,8 @@ static Music build(final Collection<String> response) {
289297
}
290298
}
291299

292-
return new Music(album, artist, albumArtist, fullPath, disc, date, genre, time, title,
293-
totalTracks, track, songId, songPos, name);
300+
return new Music(album, artist, albumArtist, composer, fullPath, disc, date, genre, time,
301+
title, totalTracks, track, songId, songPos, name);
294302
}
295303

296304
/**
@@ -494,6 +502,7 @@ public boolean equals(final Object o) {
494502
{mAlbum, music.mAlbum},
495503
{mAlbumArtist, music.mAlbumArtist},
496504
{mArtist, music.mArtist},
505+
{mComposer, music.mComposer},
497506
{mGenre, music.mGenre},
498507
{mName, music.mName},
499508
{mTitle, music.mTitle}
@@ -585,6 +594,10 @@ public Artist getArtistAsArtist() {
585594
return new Artist(mArtist);
586595
}
587596

597+
public String getComposer() {
598+
return mComposer;
599+
}
600+
588601
public long getDate() {
589602
return mDate;
590603
}

MPDroid/src/main/java/com/namelessdev/mpdroid/models/AbstractPlaylistMusic.java

+2-2
Original file line numberDiff line numberDiff line change
@@ -25,12 +25,12 @@ public abstract class AbstractPlaylistMusic extends Music {
2525
private boolean mForceCoverRefresh = false;
2626

2727
protected AbstractPlaylistMusic(final String album, final String artist,
28-
final String albumartist,
28+
final String albumartist, final String composer,
2929
final String fullpath, final int disc, final long date, final String genre,
3030
final long time, final String title,
3131
final int totalTracks, final int track, final int songId, final int pos,
3232
final String name) {
33-
super(album, artist, albumartist, fullpath, disc, date, genre, time, title,
33+
super(album, artist, albumartist, composer, fullpath, disc, date, genre, time, title,
3434
totalTracks, track, songId, pos, name);
3535
}
3636

MPDroid/src/main/java/com/namelessdev/mpdroid/models/PlaylistSong.java

+4-4
Original file line numberDiff line numberDiff line change
@@ -27,10 +27,10 @@
2727
public class PlaylistSong extends AbstractPlaylistMusic {
2828

2929
public PlaylistSong(final Music music) {
30-
super(music.getAlbum(), music.getArtist(), music.getAlbumArtist(), music.getFullPath(),
31-
music.getDisc(), music.getDate(), music.getGenre(), music.getTime(),
32-
music.getTitle(), music.getTotalTracks(), music.getTrack(), music.getSongId(),
33-
music.getPos(), music.getName());
30+
super(music.getAlbum(), music.getArtist(), music.getAlbumArtist(), music.getComposer(),
31+
music.getFullPath(), music.getDisc(), music.getDate(), music.getGenre(),
32+
music.getTime(), music.getTitle(), music.getTotalTracks(), music.getTrack(),
33+
music.getSongId(), music.getPos(), music.getName());
3434
}
3535

3636
@Override

MPDroid/src/main/java/com/namelessdev/mpdroid/models/PlaylistStream.java

+4-4
Original file line numberDiff line numberDiff line change
@@ -22,10 +22,10 @@
2222
public class PlaylistStream extends AbstractPlaylistMusic {
2323

2424
public PlaylistStream(final Music music) {
25-
super(music.getAlbum(), music.getArtist(), music.getAlbumArtist(), music.getFullPath(),
26-
music.getDisc(), music.getDate(), music.getGenre(), music.getTime(),
27-
music.getTitle(), music.getTotalTracks(), music.getTrack(), music.getSongId(),
28-
music.getPos(), music.getName());
25+
super(music.getAlbum(), music.getArtist(), music.getAlbumArtist(), music.getComposer(),
26+
music.getFullPath(), music.getDisc(), music.getDate(), music.getGenre(),
27+
music.getTime(), music.getTitle(), music.getTotalTracks(), music.getTrack(),
28+
music.getSongId(), music.getPos(), music.getName());
2929
}
3030

3131
@Override

0 commit comments

Comments
 (0)