Mailing List Archive

Ticket #13611: MythWeb: Add default sort order to tv/schedules
#13611: MythWeb: Add default sort order to tv/schedules
----------------------------------+---------------------------------
Reporter: Nigel Jewell | Owner: Stuart Auchterlonie
Type: Patch - Feature | Status: new
Priority: minor | Milestone: needs_triage
Component: Plugin - MythWeb | Version: Master Head
Severity: medium | Keywords:
Ticket locked: 0 |
----------------------------------+---------------------------------
When a user navigates to /tv/schedules the very first time the results
aren't ordered. I believe that it would be good for them to be ordered by
title on first.

Exists in master, fixes/31, fixes/30 and maybe others.

--
Ticket URL: <https://code.mythtv.org/trac/ticket/13611>
MythTV <http://www.mythtv.org>
MythTV Media Center
Re: Ticket #13611: MythWeb: Add default sort order to tv/schedules [ In reply to ]
#13611: MythWeb: Add default sort order to tv/schedules
------------------------------+-------------------------------------
Reporter: Nigel Jewell | Owner: Stuart Auchterlonie
Type: Patch - Feature | Status: new
Priority: minor | Milestone: needs_triage
Component: Plugin - MythWeb | Version: Master Head
Severity: medium | Resolution:
Keywords: | Ticket locked: 0
------------------------------+-------------------------------------

Comment (by Nigel Jewell):

Pull request: [https://github.com/MythTV/mythweb/pull/32]

--
Ticket URL: <https://code.mythtv.org/trac/ticket/13611#comment:1>
MythTV <http://www.mythtv.org>
MythTV Media Center
Re: Ticket #13611: MythWeb: Add default sort order to tv/schedules [ In reply to ]
#13611: MythWeb: Add default sort order to tv/schedules
------------------------------+-------------------------------------
Reporter: Nigel Jewell | Owner: Stuart Auchterlonie
Type: Patch - Feature | Status: new
Priority: minor | Milestone: needs_triage
Component: Plugin - MythWeb | Version: Master Head
Severity: medium | Resolution:
Keywords: | Ticket locked: 0
------------------------------+-------------------------------------

Comment (by caverndigger88):

I was able to add this to my ancient fixes/29 setup...

Only Issue :
I have a recording named "The Arrival" ...

When I refresh the "mythweb/tv/schedules" page it puts "The Arrival" on
top.
If I click on "Title" it puts "The Arrival" down right before "The Outer
Limits" .

It should be the same.

since js/table_sort.js has not been updated in like forever...
I'm not sure on how to fix it.. Suggestions?

--
Ticket URL: <https://code.mythtv.org/trac/ticket/13611#comment:2>
MythTV <http://www.mythtv.org>
MythTV Media Center
Re: Ticket #13611: MythWeb: Add default sort order to tv/schedules [ In reply to ]
#13611: MythWeb: Add default sort order to tv/schedules
------------------------------+-------------------------------------
Reporter: Nigel Jewell | Owner: Stuart Auchterlonie
Type: Patch - Feature | Status: new
Priority: minor | Milestone: needs_triage
Component: Plugin - MythWeb | Version: Master Head
Severity: medium | Resolution:
Keywords: | Ticket locked: 0
------------------------------+-------------------------------------

Comment (by Nigel Jewell):

That looks like a separate issue. In my mind, clicking on "title" should
not result in everything starting "A " being ordered together, and
everything starting "The " ordered together. The "A " and "The " should
be ignored. I'll try and identify the cause.

--
Ticket URL: <https://code.mythtv.org/trac/ticket/13611#comment:3>
MythTV <http://www.mythtv.org>
MythTV Media Center
Re: Ticket #13611: MythWeb: Add default sort order to tv/schedules [ In reply to ]
#13611: MythWeb: Add default sort order to tv/schedules
------------------------------+-------------------------------------
Reporter: Nigel Jewell | Owner: Stuart Auchterlonie
Type: Patch - Feature | Status: new
Priority: minor | Milestone: needs_triage
Component: Plugin - MythWeb | Version: Master Head
Severity: medium | Resolution:
Keywords: | Ticket locked: 0
------------------------------+-------------------------------------

Comment (by Nigel Jewell):

As a side note:

this.callback_presort(table_id, heder_index);

Should be:

this.callback_presort(table_id, header_index);

In js/table_sort.js I suspect. Will come back to this.

--
Ticket URL: <https://code.mythtv.org/trac/ticket/13611#comment:4>
MythTV <http://www.mythtv.org>
MythTV Media Center
Re: Ticket #13611: MythWeb: Add default sort order to tv/schedules [ In reply to ]
#13611: MythWeb: Add default sort order to tv/schedules
------------------------------+-------------------------------------
Reporter: Nigel Jewell | Owner: Stuart Auchterlonie
Type: Patch - Feature | Status: new
Priority: minor | Milestone: needs_triage
Component: Plugin - MythWeb | Version: Master Head
Severity: medium | Resolution:
Keywords: | Ticket locked: 0
------------------------------+-------------------------------------

Comment (by Nigel Jewell):

@caverndigger88 ... try this change:

https://github.com/mymatenige/mythweb/commit/407113c024b034f5daaafa2ad893bff7f76c89b9

I'm not going to raise a pull request as it's specific to those languages
that use A, An, The articles. Nasty side effect of the fact the original
code sorts in Javascript.

--
Ticket URL: <https://code.mythtv.org/trac/ticket/13611#comment:5>
MythTV <http://www.mythtv.org>
MythTV Media Center