Joomla! Issue Tracker - CMS

Download
Launch
Feeling Lucky

[#5413] - JFilterInput PATH filter does not support a leading slash

? Success

User tests: Successful: Unsuccessful:

avatar mbabker
mbabker
12 Dec 2014

Test code:

$filter = JFilterInput::getInstance();
var_dump($filter->clean('/path/to/joomla-cms/index.php', 'path'));

Expected Result: '/path/to/joomla-cms/index.php'
Actual Result: ''

Apply patch and re-test

Travis-CI job which should show unit test failure - https://travis-ci.org/mbabker/joomla-cms/builds/43885074
Travis-CI job which should pass with patch (in addition to the job for this PR :smile: ) - https://travis-ci.org/mbabker/joomla-cms/builds/43885122

avatar mbabker mbabker - open - 12 Dec 2014
avatar jissues-bot jissues-bot - change - 12 Dec 2014
Labels Added: ?
avatar mbabker mbabker - change - 13 Dec 2014
Category ⇒ Libraries
avatar Bakual
Bakual - comment - 13 Dec 2014

Simple fix. Merging based on review and Travis.
Thanks!

avatar Bakual Bakual - change - 13 Dec 2014
Status Pending ⇒ Closed
Closed_Date 0000-00-00 00:00:00 ⇒ 2014-12-13 07:41:49
avatar Bakual Bakual - close - 13 Dec 2014

Add a Comment

Login with GitHub to post a comment