Opened 12 years ago

Closed 9 years ago

#4517 closed patch (Fixed)

sorting in upnp

Reported by: Kevin Kalmbach Owned by: beirdo
Priority: minor Milestone: 0.25
Component: MythTV - UPnP Version: Master Head
Severity: medium Keywords:
Cc: Ticket locked: no

Description (last modified by beirdo)

I already put this in the mailing list, but this might be a better forum.

I noticed that in the latest incarnation of upnp stuff, the names are not sorted. I put in a quick sorting, don't know if it's the correct way or worthy of committing, but I thought I would throw it out there. If basically adds a sortColumn to the UPnpCDSRootInfo, then adds that in the select.

Thanks -Kevin

Attachments (2)

upnp-sorting.patch (3.6 KB) - added by beirdo 9 years ago.
0001-Reworked-patch-from-4517-sort-items-in-UPnP-video.patch (4.5 KB) - added by beirdo 9 years ago.

Download all attachments as: .zip

Change History (10)

comment:1 Changed 12 years ago by greg

Owner: changed from dblain to greg
Status: newassigned

comment:2 Changed 11 years ago by Nigel

Description: modified (diff)

comment:3 Changed 10 years ago by stuartm

Component: upnpMythTV - UPnP

comment:4 Changed 9 years ago by beirdo

Owner: changed from greg to beirdo

I'm gonna steal this. Greg, if you want it back, let me know, but as I'm in this stuff for #3580 anyways...

Changed 9 years ago by beirdo

Attachment: upnp-sorting.patch added

comment:5 Changed 9 years ago by beirdo

Description: modified (diff)

Patch moved from description to an attachment

comment:6 Changed 9 years ago by beirdo

Patch redone to trunk [25695].

Changed 9 years ago by beirdo

comment:7 Changed 9 years ago by beirdo

Version: headTrunk Head

comment:8 Changed 9 years ago by beirdo

Milestone: unknown0.25
Resolution: Fixed
Status: assignedclosed

Committed to master in e270c1046

Note: See TracTickets for help on using tickets.