› ›
CRITICAL SECURITY VERSION UPDATE:
https://github.com/daylightstudio/FUEL-CMS/releases/tag/1.4.13

HTTP links to PDFs converted to pdf_path()

almostcompletelyalmostcompletely
in Bug Reports

Fuel 1.5.1 & CKEditor

If I add a link to some text using the "link" button and that link is an absolute url (eg. http://www.foo.bar/test.pdf) then it is always returned to the editor wrapped in a pdf_path() ( eg. {pdf_path('http%3A//www.foo.bar/test.pdf')} ). Obviously, when rendered for the front-end, it results in a malformed link.

I've tracked this down to line 492 of jquery.markitup.set.js

if (selectedUrl.match(/\.pdf$/)){

I've edited this line to ignore absolute urls

if (selectedUrl.match(/\.pdf$/) && !isHTTP){

Seems to do the trick?

Comments

Sign In or Register to comment.