RTC bug PR-5.4-dev Pending

User tests: Successful: Unsuccessful:

avatar drmenzelit
drmenzelit
23 Aug 2025

Pull Request for Issue #45833 .

Summary of Changes

If I remember correctly, in my first version of mod_articles I used the truncate function that removes html-tags, that is why it was necessary to display the field "Show Article Images" only if the introtext was not truncated.
That was changed in further modification of the module, but we forgot to check the field again.

Now I changed the module so the field "Show Article Images" always displays when the introtext is set to "show" no matter of the characters limit set.

Testing Instructions

Create an article with an image in text
Create an articles module, set "Introtext" to "Show", the field "Show Article Images" is displayed.
Play with the characters limit and check if article image is showing (or not).

Actual result BEFORE applying this Pull Request

The field "Show Article Images" only appears when the characters limit for the introtext is set to 0 (deactivated)

Expected result AFTER applying this Pull Request

The field "Show Article Images" is displayed when "Introtext" is set to "Show". If the article's image is inside the characters limit, it will be displayed in the module.

Link to documentations

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

avatar drmenzelit drmenzelit - open - 23 Aug 2025
avatar drmenzelit drmenzelit - change - 23 Aug 2025
Status New Pending
avatar joomla-cms-bot joomla-cms-bot - change - 23 Aug 2025
Category Modules Front End
avatar brianteeman
brianteeman - comment - 23 Aug 2025

If the article's image is inside the characters limit, it will be displayed in the module.

Don't you think this will confuse people?

avatar drmenzelit
drmenzelit - comment - 23 Aug 2025

Don't you think this will confuse people?

It seems people are expecting to be able to display the images inserted in text ... and at the same time truncate the introtext.
I haven't had the use case. I normally use "read more" and intro / full image on my modules.

avatar brianteeman
brianteeman - comment - 23 Aug 2025

the problem occurs when the truncating takes place and excludes the image and they think its broken

I'm with you

I normally use "read more" and intro / full image on my modules.

and would say if you want the images then either use the readmore and dont truncate or use the intro_image field

you can't please all the time

avatar drmenzelit
drmenzelit - comment - 23 Aug 2025

you can't please all the time

That is true...

avatar exlemor exlemor - test_item - 23 Aug 2025 - Tested successfully
avatar exlemor
exlemor - comment - 23 Aug 2025

I have tested this item ✅ successfully on 492491d

Hi Viviana, @drmenzelit - I have successfully tested this during PBF 23.08! :)


This comment was created with the J!Tracker Application at issues.joomla.org/tracker/joomla-cms/45969.

avatar richard67 richard67 - change - 24 Aug 2025
Title
Show field "Show Article Images" also if introtext is truncated
[5.4] Show field "Show Article Images" also if introtext is truncated
avatar richard67 richard67 - edited - 24 Aug 2025
avatar drmenzelit drmenzelit - change - 2 Sep 2025
Title
[5.4] Show field "Show Article Images" also if introtext is truncated
[5.4] mod_articles - Show field "Show Article Images" also if introtext is truncated
avatar drmenzelit drmenzelit - edited - 2 Sep 2025
avatar drmenzelit drmenzelit - change - 2 Sep 2025
The description was changed
avatar drmenzelit drmenzelit - edited - 2 Sep 2025
avatar ChristineWk ChristineWk - test_item - 2 Sep 2025 - Tested successfully
avatar ChristineWk
ChristineWk - comment - 2 Sep 2025

I have tested this item ✅ successfully on 492491d


This comment was created with the J!Tracker Application at issues.joomla.org/tracker/joomla-cms/45969.

avatar richard67 richard67 - change - 2 Sep 2025
Status Pending Ready to Commit
Labels Added: bug PR-5.4-dev
avatar richard67
richard67 - comment - 2 Sep 2025

RTC


This comment was created with the J!Tracker Application at issues.joomla.org/tracker/joomla-cms/45969.

avatar muhme muhme - change - 3 Sep 2025
Labels Added: RTC
avatar muhme
muhme - comment - 3 Sep 2025

Final test before merge with JBT graft nightly build / full package:

  • Seen error before: Field 'Show Article Images' is only shown for Introtext Limit (characters): 0
  • ✅ With the PR Field 'Show Article Images' is shown with e.g. Introtext Limit (characters):100, show and hide article image switch is working, set to 90 and image disappears (outside the char limit), increased the char limit one by one, until the image appears
avatar muhme muhme - change - 3 Sep 2025
Status Ready to Commit Fixed in Code Base
Closed_Date 0000-00-00 00:00:00 2025-09-03 13:54:06
Closed_By muhme
avatar muhme muhme - close - 3 Sep 2025
avatar muhme muhme - merge - 3 Sep 2025
avatar muhme
muhme - comment - 3 Sep 2025

Thank you @drmenzelit for your contribution. Thank you @brianteeman for your support. Thank you @exlemor and @ChristineWk for testing.

Add a Comment

Login with GitHub to post a comment