Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Fix dateadded sort method in AudioLibrary.GetSongs #9809

Merged
merged 1 commit into from May 18, 2016

Conversation

@DaveTBlake
Copy link
Member

commented May 14, 2016

AudioLibrary.GetSongs is broken when using sort method dateadded, see
http://forum.kodi.tv/showthread.php?tid=273321

FieldDateAdded had been omitted from a case statement when this sort method was added to the JSON-RPC API for Jarvis.

Thanks for spotting this @Millencolin007
@Montellese, @Tolriq for interest.

Fix missing DateAdded field in case statement so that dateadded sort …
…order method can be used by AudioLibrary.GetSongs
@DaveTBlake

This comment has been minimized.

Copy link
Member Author

commented May 17, 2016

Anyone mind if I just merge this?

@Razzeee Razzeee added this to the Krypton 17.0-alpha2 milestone May 17, 2016

@Razzeee Razzeee added the v17 Krypton label May 17, 2016

@DaveTBlake DaveTBlake merged commit a935d58 into xbmc:master May 18, 2016

1 of 2 checks passed

continuous-integration/appveyor/pr AppVeyor build failed
Details
continuous-integration/travis-ci/pr The Travis CI build passed
Details

@DaveTBlake DaveTBlake deleted the DaveTBlake:JSONDateSort branch May 19, 2016

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
2 participants
You can’t perform that action at this time.