Skip to content

Commit 22ed144

Browse files
author
Nigel Jewell
committed
Add support for table header sorting with recording schedules.
1 parent 30e96d5 commit 22ed144

File tree

4 files changed

+79
-14
lines changed

4 files changed

+79
-14
lines changed

‎includes/sorting.php

Lines changed: 50 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -235,3 +235,53 @@ function by_transcoder(&$a, &$b) {
235235
if ($a->transcoder == $b->transcoder) return 0;
236236
return ($a->transcoder > $b->transcoder) ? 1 : -1;
237237
}
238+
239+
function by_callsign_with_type(&$a, &$b) {
240+
// If this is an 'always on any channel' or 'find one' recording without the 'This Channel' filter, set the callsign to 'Any'
241+
if (($a->type == rectype_always || $a->type == rectype_findone) && !($a->filter & (1 << 10)))
242+
$ac = '['.t('Any').']';
243+
else
244+
$ac = $a->channel->callsign;
245+
246+
if (($b->type == rectype_always || $b->type == rectype_findone) && !($b->filter & (1 << 10)))
247+
$bc = '['.t('Any').']';
248+
else
249+
$bc = $b->channel->callsign;
250+
251+
return strnatcasecmp($ac, $bc);
252+
}
253+
254+
function by_channum_with_type(&$a, &$b) {
255+
// If this is an 'always on any channel' or 'find one' recording without the 'This Channel' filter, set the channum to 'Any'
256+
if (($a->type == rectype_always || $a->type == rectype_findone) && !($a->filter & (1 << 10)))
257+
$ac = '['.t('Any').']';
258+
else
259+
$ac = $a->channel->channum;
260+
261+
if (($b->type == rectype_always || $b->type == rectype_findone) && !($b->filter & (1 << 10)))
262+
$bc = '['.t('Any').']';
263+
else
264+
$bc = $b->channel->channum;
265+
266+
return strnatcasecmp($ac, $bc);
267+
}
268+
269+
function by_endoffset(&$a, &$b) {
270+
if ($a->endoffset == $b->endoffset) return 0;
271+
return ($a->endoffset > $b->endoffset) ? 1 : -1;
272+
}
273+
274+
function by_last_record(&$a, &$b) {
275+
if ($a->last_record == $b->last_record) return 0;
276+
return ($a->last_record > $b->last_record) ? 1 : -1;
277+
}
278+
279+
function by_startoffset(&$a, &$b) {
280+
if ($a->startoffset == $b->startoffset) return 0;
281+
return ($a->startoffset > $b->startoffset) ? 1 : -1;
282+
}
283+
284+
function by_storagegroup(&$a, &$b) {
285+
if ($a->storagegroup == $b->storagegroup) return 0;
286+
return ($a->storagegroup > $b->storagegroup) ? 1 : -1;
287+
}

‎modules/tv/classes/Schedule.php

Lines changed: 9 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -112,6 +112,15 @@ public static function findAll($sort = true) {
112112
case 'airdate':
113113
$orderby .= 'starttime';
114114
break;
115+
case 'callsign_with_type':
116+
$orderby .= 'CASE WHEN (type = '.rectype_always.' OR type = '.rectype_findone.') AND !(filter & (1 << 10)) THEN "" ELSE callsign END';
117+
break;
118+
case 'channum_with_type':
119+
$orderby .= 'CASE WHEN (type = '.rectype_always.' OR type = '.rectype_findone.') AND !(filter & (1 << 10)) THEN 0 ELSE channum END';
120+
break;
121+
case 'last_record':
122+
$orderby .= 'record.last_record';
123+
break;
115124
case 'recpriority':
116125
$orderby .= 'record.recpriority';
117126
break;

‎modules/tv/tmpl/default/schedules.php

Lines changed: 15 additions & 12 deletions
Original file line numberDiff line numberDiff line change
@@ -28,7 +28,7 @@
2828

2929
// Print the page contents
3030
$group_field = $_GET['sortby'];
31-
if ($group_field == 'title' || !in_array($group_field, array('title', 'channum', 'type', 'profile', 'recgroup')))
31+
if ($group_field == 'title' || !in_array($group_field, array('title', 'callsign', 'channum', 'type', 'profile', 'recgroup')))
3232
$group_field = '';
3333
?>
3434

@@ -71,17 +71,17 @@ function attempt_to_show_tip(element) {
7171
<thead>
7272
<tr class="menu">
7373
<?php if ($group_field != '') echo "<th class=\"list\">&nbsp;</th>\n"; ?>
74-
<th class="x-title"><?php echo t('Title' ); ?></th>
75-
<th class="x-priority"><?php echo t('Recording Priority' ); ?></th>
76-
<th class="x-channel" sort_hint="sortMythwebChannel"><?php echo t('Channel' ); ?></th>
77-
<th class="x-profile"><?php echo t('Profile' ); ?></th>
78-
<th class="x-transcoder"><?php echo t('Transcoder'); ?></th>
79-
<th class="x-group"><?php echo t('Recording Group'); ?></th>
80-
<th class="x-type"><?php echo t('Type'); ?></th>
81-
<th class="x-sgroup"><?php echo t('Storage Group'); ?></th>
82-
<th class="x-startoffset"><?php echo t('Start Early'); ?></th>
83-
<th class="x-endoffset"><?php echo t('End Late'); ?></th>
84-
<th class="x-lastrec"><?php echo t('Last Recorded'); ?></th>
74+
<th class="x-title" sort_hint="sortMythwebTitle"><?php echo get_sort_link('title', t('Title' )); ?></th>
75+
<th class="x-priority"><?php echo get_sort_link('recpriority', t('Recording Priority' )); ?></th>
76+
<th class="x-channel" sort_hint="sortMythwebChannel"><?php echo get_sort_link($_SESSION["prefer_channum"] ? 'channum_with_type' : 'callsign_with_type', t('Channel' )); ?></th>
77+
<th class="x-profile"><?php echo get_sort_link('profile', t('Profile' )); ?></th>
78+
<th class="x-transcoder"><?php echo get_sort_link('transcoder', t('Transcoder')); ?></th>
79+
<th class="x-group"><?php echo get_sort_link('recgroup', t('Recording Group')); ?></th>
80+
<th class="x-type"><?php echo get_sort_link('type', t('Type')); ?></th>
81+
<th class="x-sgroup"><?php echo get_sort_link('storagegroup', t('Storage Group')); ?></th>
82+
<th class="x-startoffset"><?php echo get_sort_link('startoffset', t('Start Early')); ?></th>
83+
<th class="x-endoffset"><?php echo get_sort_link('endoffset', t('End Late')); ?></th>
84+
<th class="x-lastrec"><?php echo get_sort_link('last_record', t('Last Recorded')); ?></th>
8585
</tr>
8686
</thead>
8787
<?php
@@ -98,6 +98,7 @@ function attempt_to_show_tip(element) {
9898
// If this is an 'always on any channel' or 'find one' recording without the 'This Channel' filter, set the channel name to 'Any'
9999
if (($schedule->type == rectype_always || $schedule->type == rectype_findone) && !($schedule->filter & (1 << 10))) {
100100
$schedule->channel->name = '[ '.t('Any').' ]';
101+
$schedule->channel->callsign = null;
101102
$schedule->channel->channum = 0;
102103
}
103104
// A program id counter for popup info
@@ -109,6 +110,8 @@ function attempt_to_show_tip(element) {
109110
// Print a dividing row if grouping changes
110111
if ($group_field == 'type')
111112
$cur_group = $schedule->texttype;
113+
elseif ($group_field == 'callsign')
114+
$cur_group = ($schedule->channel->callsign ? $schedule->channel->callsign.' - ' : '').$schedule->channel->name;
112115
elseif ($group_field == 'channum')
113116
$cur_group = ($schedule->channel->channum ? $schedule->channel->channum.' - ' : '').$schedule->channel->name;
114117
elseif ($group_field == 'profile')

‎modules/tv/tmpl/lite/schedules.php

Lines changed: 5 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -31,7 +31,7 @@
3131

3232
// Print the page contents
3333
$group_field = $_GET['sortby'];
34-
if ($group_field == 'title' || !in_array($group_field, array('title', 'channum', 'type', 'profile', 'recgroup')))
34+
if ($group_field == 'title' || !in_array($group_field, array('title', 'callsign', 'channum', 'type', 'profile', 'recgroup')))
3535
$group_field = '';
3636
?>
3737

@@ -40,7 +40,7 @@
4040
<?php if ($group_field != '') echo "<td class=\"list\">&nbsp;</td>\n" ?>
4141
<td><?php echo get_sort_link('title', t('title')) ?></td>
4242
<td><?php echo get_sort_link('recpriority', t('recpriority')) ?></td>
43-
<td><?php echo get_sort_link($_SESSION["prefer_channum"] ? 'channum' : 'callsign', t('channel')) ?></td>
43+
<td><?php echo get_sort_link($_SESSION["prefer_channum"] ? 'channum_with_type' : 'callsign_with_type', t('channel')) ?></td>
4444
<td><?php echo get_sort_link('profile', t('profile')) ?></td>
4545
<td><?php echo get_sort_link('transcoder', t('transcoder')) ?></td>
4646
<td><?php echo get_sort_link('recgroup', t('recgroup')) ?></td>
@@ -56,12 +56,15 @@
5656
// If this is an 'always on any channel' or 'find one' recording without the 'This Channel' filter, set the channel name to 'Any'
5757
if (($schedule->type == rectype_always || $schedule->type == rectype_findone) && !($schedule->filter & (1 << 10))) {
5858
$schedule->channel->name = '[ '.t('Any').' ]';
59+
$schedule->channel->callsign = null;
5960
$schedule->channel->channum = 0;
6061
}
6162

6263
// Print a dividing row if grouping changes
6364
if ($group_field == 'type')
6465
$cur_group = $schedule->texttype;
66+
elseif ($group_field == 'callsign')
67+
$cur_group = ($schedule->channel->callsign ? $schedule->channel->callsign.' - ' : '').$schedule->channel->name;
6568
elseif ($group_field == 'channum')
6669
$cur_group = ($schedule->channel->channum ? $schedule->channel->channum.' - ' : '').$schedule->channel->name;
6770
elseif ($group_field == 'profile')

0 commit comments

Comments
 (0)