[5.4] mod_articles - Show field "Show Article Images" also if introtext is truncated

[drmenzelit]

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

[brianteeman]

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?

[drmenzelit]

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.

[brianteeman]

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

[drmenzelit]

you can't please all the time

That is true...

[exlemor]

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.}

[ChristineWk]

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.}

[richard67]

RTC

---

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

[muhme]

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

[muhme]

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