Opened 10 years ago

Closed 9 years ago

#7592 closed defect (fixed)

ical link broken on Upcoming page on mythweb

Reported by: Paul <paul@…> Owned by: Rob Smith
Priority: minor Milestone: unknown
Component: Plugin - MythWeb Version: 0.22
Severity: medium Keywords:
Cc: Ticket locked: no

Description

http://mythbox/mythweb/ical/mythweb/tv/upcoming?skip_deactivated&

Warning at /var/www/html/mythweb/modules/_shared/tmpl/_errors/unknown_module.php, line 23: require(modules/_shared/tmpl/ical/header.php) [function.require]: failed to open stream: No such file or directory

Fatal error: require() [function.require]: Failed opening required 'modules/_shared/tmpl/ical/header.php' (include_path='.:/usr/share/pear:/usr/share/php') in /var/www/html/mythweb/modules/_shared/tmpl/_errors/unknown_module.php on line 23

Attachments (2)

ical_link.patch (649 bytes) - added by Nijn <info@…> 10 years ago.
proposed patch
upcoming_recordings_ical_link.diff (583 bytes) - added by nickj-fox@… 10 years ago.
ical link fix

Download all attachments as: .zip

Change History (6)

Changed 10 years ago by Nijn <info@…>

Attachment: ical_link.patch added

proposed patch

comment:1 Changed 10 years ago by Nijn <info@…>

The ical link is wrong. it uses $_SERVERREQUEST_URI? but that already includes the root. Not sure if there is already a constant for that, but if not here is my proposed fix.

comment:2 Changed 10 years ago by nickj-fox@…

I have a diff for this that is simpler on the code side and might be easy to have implemented. I have tested that this works on my local machines.

Changed 10 years ago by nickj-fox@…

ical link fix

comment:3 Changed 10 years ago by Marc Randolph <mrand@…>

Bug #7948 (with patch) is for similar problem, but for rss link

comment:4 Changed 9 years ago by Rob Smith

Resolution: fixed
Status: newclosed

(In [25083]) Fixes #7948, Fixes #7592, this modified the default REQUEST_URI to handle the root prefix automagically.

Note: See TracTickets for help on using tickets.