User tests: Successful: Unsuccessful:
Pull Request for Issue # 44898
Made changes in this file components/com_tags/tmpl/tag/default_items.php
remove htmlencode for space i.e %20 and replaced with blank space
Image was not visible
image should be visible
Please select:
Documentation link for docs.joomla.org:
No documentation changes for docs.joomla.org needed
Pull Request link for manual.joomla.org:
No documentation changes for manual.joomla.org needed
Status | New | ⇒ | Pending |
Category | ⇒ | com_tags Front End |
Labels |
Added:
PR-5.3-dev
|
Title |
|
successfully tested. I manually edited the file and now the image is shown
Issue fixed after patch.
@webfeuerflo @dynamicsites Can you please mark your test as successfully and submit the test result at the Issue Tracker so the test count.
This is not the correct fix. It should be using the layout
I have tested this item ✅ successfully on b6625f0
I have tested this item ✅ successfully on b6625f0
This is not the correct fix. It should be using the layout
Not setting to RTC despite of successful tests for this reason.
@tekvishal Could you modify this PR or make a new one to do it the right way? That would be much appreciated.
Sure I'll update the PR
On Sat, 22 Feb, 2025, 6:54 pm Richard Fath, @.***>
wrote:
This is not the correct fix. It should be using the layout
Not setting to RTC despite of successful tests for this reason.
@tekvishal https://github.com/tekvishal Could you modify this PR or
make a new one to do it the right way? That would be much appreciated.—
Reply to this email directly, view it on GitHub
#44960 (comment),
or unsubscribe
https://github.com/notifications/unsubscribe-auth/BMKL6536XOMWAMKPIIDALHD2RB3ARAVCNFSM6AAAAABXU2Y5QCVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDMNZWGIYTCMJYGQ
.
You are receiving this because you were mentioned.Message ID:
@.***>
[image: richard67]richard67 left a comment (#44960)
#44960 (comment)This is not the correct fix. It should be using the layout
Not setting to RTC despite of successful tests for this reason.
@tekvishal https://github.com/tekvishal Could you modify this PR or
make a new one to do it the right way? That would be much appreciated.—
Reply to this email directly, view it on GitHub
#44960 (comment),
or unsubscribe
https://github.com/notifications/unsubscribe-auth/BMKL6536XOMWAMKPIIDALHD2RB3ARAVCNFSM6AAAAABXU2Y5QCVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDMNZWGIYTCMJYGQ
.
You are receiving this because you were mentioned.Message ID:
@.***>
Labels |
Added:
Updates Requested
|
(as mentioned in the Issue conversation -
Since this is not recommended for accessibility, SEO, and browser consistency reasons why are we even allowing it / entertaining allowing spaces in filenames? (I know I am going to get beaten up for that but figured I'd ask lol)
As mentioned by PixedBo, why don't we sanitize the file name...
It appears we have this:
use Joomla\CMS\Filesystem\File;
$safeFilename = File::makeSafe($originalFilename);
or update that function if need be.
(I realize I'm clueless in coding so I apologize in advance).
"This is not the correct fix. It should be using the layout"
Can you please help me understand what exact changes are required here?
I would expect this to use the layout
LayoutHelper::render('joomla.content.intro_image')
I have tested this item ✅ successfully on b6625f0
Tested on Joomla! 5.3-dev on the test server
Patch Applied using Joomla! Patch Tester
Followed instructions, works as described
This comment was created with the J!Tracker Application at issues.joomla.org/tracker/joomla-cms/44960.