Opened 6 years ago

Closed 6 years ago

Last modified 6 years ago

#10791 closed Patch - Bug Fix (fixed)

In mythweb.php, ':' should be changed to PATH_SEPARATOR

Reported by: karog <mythtv@…> Owned by: Rob Smith
Priority: minor Milestone: 0.26.1
Component: Plugin - MythWeb Version: 0.25-fixes
Severity: low Keywords:
Cc: Ticket locked: no

Description

In mythweb.php, include_path is set numerous times using PATH_SEPARATOR except for one case where a literal ':' is used.

        ini_set('include_path', ini_get('include_path').':'.modules_path.'/'.$Path[0]);

should be changed to

        ini_set('include_path', ini_get('include_path').PATH_SEPARATOR.modules_path.'/'.$Path[0]);

Attachments (1)

0002-change-to-PATH_SEPARATOR-in-mythweb.php.patch (998 bytes) - added by dekarl@… 6 years ago.
suggest fix, ready to commit on master

Download all attachments as: .zip

Change History (5)

Changed 6 years ago by dekarl@…

suggest fix, ready to commit on master

comment:1 Changed 6 years ago by beirdo

Type: Bug Report - GeneralPatch - Bug Fix

comment:2 Changed 6 years ago by Karl Dietz <dekarl@…>

Resolution: fixed
Status: newclosed

In f9f5a1f43f2ddfeaa038054226f6ac0aa0b973e6/mythweb:

Use PATH_SEPARATOR rather than ':' in mythweb.php.

Fixes #10791

Signed-off-by: Raymond Wagner <rwagner@…>

comment:3 Changed 6 years ago by Karl Dietz <dekarl@…>

In f82ca9521775edbbc53caf0f40f7bc183732dcc2/mythweb:

Use PATH_SEPARATOR rather than ':' in mythweb.php.

Fixes #10791

Signed-off-by: Raymond Wagner <rwagner@…>
(cherry picked from commit f9f5a1f43f2ddfeaa038054226f6ac0aa0b973e6)

comment:4 Changed 6 years ago by Raymond Wagner

Milestone: unknown0.26.1
Note: See TracTickets for help on using tickets.