Opened 14 years ago
Closed 14 years ago
Last modified 14 years ago
#10944 closed Patch - Bug Fix (fixed)
unbreak starting playback on a frontend fromt the recording details
| Reported by: | Owned by: | danielk | |
|---|---|---|---|
| Priority: | minor | Milestone: | 0.26 |
| Component: | Plugin - MythWeb | Version: | Master Head |
| Severity: | medium | Keywords: | |
| Cc: | Ticket locked: | no |
Description
appears to have been missed in the UTC conversion
Attachments (1)
Change History (9)
by , 14 years ago
| Attachment: | 10944-unbreak-playback-on-frontend.patch added |
|---|
comment:1 by , 14 years ago
| Owner: | changed from to |
|---|---|
| Status: | new → assigned |
comment:2 by , 14 years ago
| Milestone: | unknown → 0.26 |
|---|---|
| Status: | assigned → infoneeded |
comment:3 by , 14 years ago
I was debugging why the playback on frontend x function did not work and fixed both errors while on it.
Hitting the play on frontend button resulted in the following error
[Thu Jul 26 14:23:38 2012] [error] [client 192.168.66.137] PHP Fatal error: require_once(): Failed opening required 'tv.php' (include_path='/usr/local/share/mythtv/bindings/php/:/usr/share/mythtv/bindings/php/:.:/usr/share/php:/usr/share/pear:/usr/share/mythtv/mythweb/modules/remote') in /usr/share/mythtv/mythweb/modules/remote/handler.php on line 100, referer: http://mythmaster.breitlacher/mythweb/tv/detail/2065/1337288700
looking at the (removed in the meantime) iPad template revealed that the relative path led to calling http://mythmaster.breitlacher/mythweb/remote/tv/detail/2065/1337288700 instead of just http://mythmaster.breitlacher/mythweb/tv/detail/2065/1337288700 (without the remote/ component) which was avoided by making the path absolute (by prepending "root.")
But that appears to just be a cosmetic error / red herring so I continued to debug which led me to the UTC issue. I never tested what happens if just the latter issues gets fixed.
comment:4 by , 14 years ago
| Status: | infoneeded → assigned |
|---|
comment:5 by , 14 years ago
Unfortuntely, I seem to be unable to test this. I don't have any frontends listed as available for playback duties.
comment:6 by , 14 years ago
| Resolution: | → fixed |
|---|---|
| Status: | assigned → closed |
comment:7 by , 14 years ago
Dekarl, the .root. stuff doesn't appear to be necessary. I'm not really the right person to review those types of changes, so I've just applied the necessary bit in bindings/php/MythFrontend.php.
dekarl, the second part of the patch looks like a UTC issue, what is the root stuff about though?