RequiredFiles ommits files for default layout

Bug #552853 reported by Alex Harrington
14
This bug affects 2 people
Affects Status Importance Assigned to Milestone
Xibo
Fix Released
Medium
Dan Garner
1.0
Fix Released
Medium
Alex Harrington
1.1
Fix Released
Medium
Dan Garner

Bug Description

RequiredFiles omits the files required for the default layout when there is a schedule item that falls inside the "plus4hours" lookahead.

Perhaps RequiredFiles should always return the files for the default layout? (It would certainly help me on the Python Client if it did).

Alex

Changed in xibo:
status: New → Confirmed
importance: Undecided → Medium
milestone: none → 1.1.1
Revision history for this message
Alex Harrington (alexharrington) wrote :

Commenting out lines 248, 249, 252 - 256 in xmds.php fixes this for me.

248 //if ($displayInfo['inc_schedule'] == 1)
249 //{
250 $SQL .= sprintf(" AND ((schedule_detail.starttime < '%s' AND schedule_detail.endtime > '%s' )", $plus4hours, $currentdate);
251 $SQL .= " OR (schedule_detail.starttime = '2050-12-31 00:00:00' AND schedule_detail.endtime = '2050-12-31 00:00:00' ))";
252 //}
253 //else
254 //{
255 // $SQL .= sprintf(" AND (schedule_detail.starttime < '%s' AND schedule_detail.endtime > '%s' )", $plus4hours, $currentdate);
256 //}

Revision history for this message
Bartek (czajka) wrote :

Fastest bugfix that i've ever seen ;)
Worked for me too, hope there is no hidden negative effects :)

Thanks!

Revision history for this message
Dan Garner (dangarner) wrote : Re: [Bug 552853] Re: RequiredFiles ommits files for default layout

That should have the desired effect - basically forcing the media for the
default layout to always be downloaded regardless of if it is in the
schedule or not.

This will mean that if someone has a huge default layout that will take a
long time to download a fresh client will show nothing until that is
downloaded - even if the default is not going to be shown.

Changed in xibo:
status: Confirmed → Triaged
Dan Garner (dangarner)
Changed in xibo:
assignee: nobody → Dan Garner (dangarner)
status: Triaged → Fix Committed
Dan Garner (dangarner)
Changed in xibo:
status: Fix Committed → Fix Released
Revision history for this message
Alex Harrington (alexharrington) wrote :
Revision history for this message
Alex Harrington (alexharrington) wrote :
To post a comment you must log in.
This report contains Public information  
Everyone can see this information.

Other bug subscribers

Related questions

Remote bug watches

Bug watches keep track of this bug in other bug trackers.