#206 svn-689: Unsupported sort order on XBox 360

[Anonymous]

Since upgrading to revision 689, I'm getting a unsupported sort-order message and, presumably caused by the empty sorting string, a failed SQL select. The XBox request upnp:class, which seems to be commented out in the latest fuppes version.

I uncommented it, and it seems to work as usual. Then again, it was probably commented out for a reason, which is why I'm posting this.

Also, I was somehow expecting fuppes to just ignore the sort order, if it is not supported. Error handling seems to be dodgy in that part (especially since dc:title was also requested, and that wasn't commented out).

[Anonymous]

I was just about to report this as well.
Output is:

unhandled sort order
your device requested a sort order that is currently not supported by FUPPES. This is not a bug as FUPPES correctly specifies it's sorting capabilities but your device seems to ignore it.
please file a feature request containing the following lines:

=== CUT ===
-upnp:class,+dc:title
=== CUT ===
SQL select error: near "limit": syntax error :: select o.OBJECT_ID, o.TYPE, o.TITLE, o.REF_ID, o.VREF_ID, d.*, CASE WHEN o.DEVICE is NULL THEN o.PATH ELSE (select PATH from OBJECTS where DEVICE is NULL and OBJECT_ID = o.VREF_ID) END AS PATH, CASE WHEN o.DEVICE is NULL THEN o.FILE_NAME ELSE (select FILE_NAME from OBJECTS where DEVICE is NULL and OBJECT_ID = o.VREF_ID) END AS FILE_NAME from OBJECTS o left join OBJECT_DETAILS d on (d.ID = o.DETAIL_ID) where o.PARENT_ID = 22 and o.TYPE < 200 and o.DEVICE is NULL and o.VISIBLE = 1 order by limit 0, 1000

[Anonymous]

i have the same bug with SVN 691 still, if u add the order type: title manually in the code it works partically. still need a fully working fix for this :(

[Giles]

Also noticed all sorting on XBOX 360 is disabled for some reason: e.g. album tracks are no longer sorted by track number.
Fixed by uncommenting out the if/else UPnPActionFactory.cpp around line 480
and uncomment adding the order by clause UPnPActionSearch.cpp line 323