[#37674] - [4.1] fix parks image
- Closed
- 15 May 2022
- Medium
- Build: 4.1-dev
- # 37674
- Diff
- N6REJ:parks
User tests: Successful: Unsuccessful:
Pull Request for Issue # .
Summary of Changes
Parks image doesn't display properly due to improper relative path.
Testing Instructions
Install sample testing data.
Click on "parks" on the menu at the top
Notice that the image does not display.
Verify the url for the image is "http://j4.local/index.php/images/sampledata/parks/banner_cradle.jpg" ( your uri will be different of course )
view that the image displays.
verify url for the image is "http://j4.local/images/sampledata/parks/banner_cradle.jpg"
Actual result BEFORE applying this Pull Request
Expected result AFTER applying this Pull Request
Documentation Changes Required
None
| Status | New | ⇒ | Pending |
joomla-cms-bot
- 
| Category | ⇒ | Front End Plugins |
My local testing is in a subdirectory http://localhost/joomla-cms-4.1-dev/ and not in the root so this PR will not be a correct fix.
Enable Use URL Rewriting and it will not be an issue.
I have tested this item
This comment was created with the J!Tracker Application at issues.joomla.org/tracker/joomla-cms/37674.
My local testing is in a subdirectory
http://localhost/joomla-cms-4.1-dev/and not in the root so this PR will not be a correct fix. EnableUse URL Rewritingand it will not be an issue.
Mine is a sub as well...
www/j4/.....
We had this exact same issue several years ago when the router was changed.
I have tested this item
? unsuccessfully on cccd072This comment was created with the J!Tracker Application at issues.joomla.org/tracker/joomla-cms/37674.
How was it unsuccessful, please show the failure. Just because you don't like the fix doesn't mean it doesn't work.
Before PR
After PR
this really should be closed
| Labels |
Added:
?
|
||
No, it shouldn't.
The "default" dataset returns
which is incorrect.
"http://j4.local/index.php/images/sampledata/parks/banner_cradle.jpg"
is off yet is clearly being left on even with .htaccess and url rewriting going on.
IF infact we do remove the testing sample dataset then that is a different issue.
- Add suffix to URL is nothing to do with this
- I already showed that this PR does not work
your test doesn't even make sense... if the image showed pre-fix why would I change it? and how would adding a "/" change the url so dramatically. I'll retest yet again with a clean ( again ) install.
sample "test" data option has been removed from 4.1-x So I'm closing this
| Status | Pending | ⇒ | Closed |
| Closed_Date | 0000-00-00 00:00:00 | ⇒ | 2022-05-15 10:48:34 |
| Closed_By | ⇒ | N6REJ |
sample "test" data has NEVER EVER been part of any joomla release
see #37663