Fixed image/document not being displayed because of the __DIR__ option in the path. #557
Conversation
… displayed because of the rendered path with DIR. I tryed to make it as clean as possible.
|
Instead of creating a different pull request each time yo want to fix something, just commit in the same branch :) |
| protected function getRealLocation() | ||
| { | ||
| // get rid of the __DIR__ so it doesn't screw when displaying uploaded doc/image in the view. | ||
| return '/uploads/documents'; |
There was a problem hiding this comment.
you should not add the leading slash here otherwise the rendered link when using asset() in a template will always be /uploads/... even when your app is not at the root of the domain.
There was a problem hiding this comment.
@stof yeah but having the app at the top of the domain and not using a leading slash result in the image not rendering. Where this leading slash should be added so ? in the template ?
There was a problem hiding this comment.
As I said on the previous PR, you need to use the asset() function in your template to make your app portable.
There was a problem hiding this comment.
Ok thanks, just pushed the modifications. I've just tested and you are right using asset() automatically adds the missing leading slash when it's needed.
|
Sry for the differents pull request. I'm definitly not familiar with the web interface when editing & submitting a pull request. |
Added another method in order to prevent the document/image not being displayed because of the rendered path with DIR. I tryed to make it as clean as possible.