brockf / hero

Hero Framework: A powerful Content Management System (CMS) and eCommerce framework built on CodeIgniter.
Other
53 stars 45 forks source link

WYSIWG Needs to include full path of filename when uploaded in KCFinder #2

Open lonnieezell opened 11 years ago

brockf commented 11 years ago

What is it including now? It should, now, be including the path RELATIVE to the root of the Hero site. This, in tandem with a <base href="{url}" /> tag in the layout.thtml template, should keep the image src links in tact.

lonnieezell commented 11 years ago

From what I recall, we had 2 or 3 people running into issues where the images would be display on the front end just fine, but would show as broken in the admin area. This was due to lacking the '/' at the beginning of the relative url it was given, so it was trying to pull it relative to the current page, which was incorrect. If we could get a full path returned, instead of a relative path, it would fix this.

brockf commented 11 years ago

Yes but then it would break on the frontend AND it would make the site break everytime it was moved from one directory to another.

The solution is having it RELATIVE to the current directory but, yes, this needs to work for both the admin panel and the frontend.

Some simple testing should show whether this is the case or not. It used to be, that I know.