Opened 9 years ago

Closed 9 years ago

#8786 closed enhancement (fixed)

Mythweb video sorting

Reported by: patrickdk@… Owned by: Rob Smith
Priority: trivial Milestone: 0.24
Component: Plugin - MythWeb Version: Master Head
Severity: low Keywords: mythweb sort episode
Cc: Ticket locked: no

Description

I don't get why you have the database sort, then call a php function to sort it, seems like a duplicate of effort. It also causes tv episodes to be really out of order. I am not sure how you want to handle this officially, but just commenting out the php sort function, and adding season and episode below to mysql sort causes it to be fixed and works nice for me.

Changed: ORDER BY title

to: ORDER BY title,season,episode

$sh = $db->query('

SELECT videometadata.intid FROM videometadata LEFT JOIN videometadatagenre

ON videometadata.intid = videometadatagenre.idvideo

' . $where . ' GROUP BY intid ORDER BY title,season,episode');

while ($intid = $sh->fetch_col()) {

$All_Videos[] = new Video($intid);

} $sh->finish();

if (count($All_Videos)) sort_programs($All_Videos, 'video_sortby');

Change History (4)

comment:1 Changed 9 years ago by anonymous

Oh sorry, this is in the end of mythweb/modules/video/handler.php

comment:2 Changed 9 years ago by robertm

Status: newassigned

comment:3 Changed 9 years ago by Rob Smith

Milestone: unknown0.24

comment:4 Changed 9 years ago by Rob Smith

Resolution: fixed
Status: assignedclosed

(In [26224]) Fixes #8786, this removes a legacy video sort function and uses new fields to sort episodic content better

Note: See TracTickets for help on using tickets.