Opened 13 years ago

Closed 13 years ago

Last modified 12 years ago

#1771 closed enhancement (fixed)

"All Programs" specific ascending/descending order

Reported by: jppoet@… Owned by: danielk
Priority: minor Milestone: 0.21
Component: mythtv Version: head
Severity: medium Keywords: episode sort
Cc: Ticket locked: no

Description (last modified by skamithi)

The attached patch adds the ability to control whether episodes in the "All Programs" or "Default" section are sorted by ascending or descending record date.

Attachments (7)

PlayBoxAllOrdering.patch (5.7 KB) - added by jppoet@… 13 years ago.
"All Programs" sort order
PlayBoxAllOrdering.2.patch (7.0 KB) - added by jppoet@… 13 years ago.
Version 2: ascending/descending control for "All Programs"
PlayBoxAllOrdering3.patch (7.2 KB) - added by John Poet <jppoet@…> 13 years ago.
Compatible with 11332
PlayBoxAllOrdering4.patch (7.2 KB) - added by John Poet <jppoet@…> 13 years ago.
Compatible with 11373
PlayBoxAllOrdering5.patch (7.4 KB) - added by John Poet <jppoet@…> 13 years ago.
Use single setting for both types of title sort modes.
PlayBoxAllOrdering5a.patch (7.7 KB) - added by John Poet <jppoet@…> 13 years ago.
Removed tabs.
PlayBoxAllOrdering5b.patch (7.7 KB) - added by John Poet <jppoet@…> 13 years ago.
Fix typo: effected affected

Download all attachments as: .zip

Change History (17)

Changed 13 years ago by jppoet@…

Attachment: PlayBoxAllOrdering.patch added

"All Programs" sort order

comment:1 Changed 13 years ago by skamithi

Description: modified (diff)

Jppoet. Isn't this in the code already ? I'm using the latest svn revision. It has the "List Newest Recording First" option along with the "Sort Episode" by record date option. This does the same thing your patch does in regards to the "All Program" section. Does this not work for you ? Works for me.

comment:2 Changed 13 years ago by jppoet@…

(Adding this comment to the ticket, in answer to Skamithi's comment)

This patch allows the "All Programs" or "Default" section to be controlled *independant* of the individual "titles" section.

Personally, I like the "Titles" sections to be sorted in *ascending* order by "Original Air Date" so I can easily watch the programs in the correct order. However, I like the "All Programs" section to be sorted in *descending* order so I can see what has been recorded recently. Does that make sense?

Changed 13 years ago by jppoet@…

Attachment: PlayBoxAllOrdering.2.patch added

Version 2: ascending/descending control for "All Programs"

comment:3 Changed 13 years ago by danielk

Milestone: 0.21
Owner: changed from Isaac Richards to danielk
Version: head

Changed 13 years ago by John Poet <jppoet@…>

Attachment: PlayBoxAllOrdering3.patch added

Compatible with 11332

comment:4 Changed 13 years ago by John Poet <jppoet@…>

Updated version for compatibility with svn rev 11332.

Changed 13 years ago by John Poet <jppoet@…>

Attachment: PlayBoxAllOrdering4.patch added

Compatible with 11373

comment:5 Changed 13 years ago by John Poet <jppoet@…>

Updated version for compatibility with svn rev 11373.

comment:6 Changed 13 years ago by danielk

Resolution: invalid
Status: newclosed

John, I like this functionality, but I don't like the extra configuration box. Can you make the PlayBoxOrdering? a tri-state instead?

Make it a ComboBox? and use something like:

  addSelection("Sort Descending", 0);
  addSelection("Sort Ascending",  1);
  addSelection("Sort Descending except in multiprogram views", 2);

That way no one's existing configuration is effected, and we don't add yet another configuration item...

I'm closing this for now, but please reopen it when you have an updated patch, or e-mail me with any questions or concerns.

Changed 13 years ago by John Poet <jppoet@…>

Attachment: PlayBoxAllOrdering5.patch added

Use single setting for both types of title sort modes.

comment:7 Changed 13 years ago by John Poet <jppoet@…>

Resolution: invalid
Status: closedreopened

This version gets rid of the seperate setting for the "All Program" section.

The setting for general PlayBoxOrdering? is now used to specify the different sort order possibilities for the individual title and multi-title sections.

Better, Daniel?

Changed 13 years ago by John Poet <jppoet@…>

Attachment: PlayBoxAllOrdering5a.patch added

Removed tabs.

comment:8 Changed 13 years ago by John Poet <jppoet@…>

Removed some tabs and added a comment to the help explaining that sections in parentheses are not affected by this setting.

Changed 13 years ago by John Poet <jppoet@…>

Attachment: PlayBoxAllOrdering5b.patch added

Fix typo: effected affected

comment:9 Changed 13 years ago by danielk

Resolution: fixed
Status: reopenedclosed

(In [11423]) Fixes #1771. This allows the "All Programs" list to have a different order from other program lists by applying patch from John P. Poet.

The main benefit of this is to allow you to see the latest recorded program in "All Programs" while still having series recordings ordered however you want.

The only non-formatting change from the patch is to use a little simple math to set listOrder & allOrder from DB field instead of a switch.

comment:10 Changed 12 years ago by anonymous

Type: patchenhancement
Note: See TracTickets for help on using tickets.