User tests: Successful: Unsuccessful:
Option to use or not, the intro image field or the full image field of the article
The module shows or not, the fields "Intro image" or "Full article image" of the article correctly
The current option only display images in the article context.
New module option added.
|Category||⇒||Language & Strings Modules Front End|
|Status||Pending||⇒||Ready to Commit|
please remove the RTC status as this PR is not backwards compatible and it also changes the way that the module works
Create an article with an image INSIDE the article
create a newsflash module and set the show images to NO
Test and you will see that there are no images displayed
Apply this PR and do NOT change anything in the module
Check the module on the front end and you will see that images ARE displayed
There is no longer a way to not display images that are INSIDE the article
The no image option only stops the display of intro and full images
the intro image option will display the images INSIDE the article and the intro image
The full image option will display the images INSIDE the article and the full image
|Status||Ready to Commit||⇒||Pending|
@franz-wohlkoenig if a status is changed on the issue tracker it will be updated on github but only when a comment is made. Your comment triggered the change. So in future if a change is made on the issue tracker you should always make a comment explaining that yu made the change and then it will work here on github too
thanks Brian, so its vica versa working to set RTC and wrote here > haven't thought about it :-
@brianteeman the issue you talk about is more related to the user context than the module. Why need the module remove any image you put INSIDE the article? If I want my article to have or not an image in the introtext, then I just put it or not, INSIDE the introtext.
Anyway, can be another param asking for this, in case the user wants also, show or not, the image INSIDE the article
Also, in my opinion, I think the option to put an image that belongs to the text of the article, is an option only a few users use it.
No, I do not have a magic ball, that is why I wrote "in my opinion", just counting with my small knowledge about this option.
Ok, what I will do is, revert the change, and make a new param leaving the old intact. In this way, we are only adding a new option to the module, without modifying it
I revert the change. Now, this option is with a new param. Also, I modify the description for the old one, trying to avoid confusion, if is possible change this description.
B/C images all fixed now.
I think it would be best if you updated the old string "Show Images" to "Show article images" or something like that - just so that people dont think it is related to the other options below
|Status||Pending||⇒||Ready to Commit|
|Status||Ready to Commit||⇒||Fixed in Code Base|
|Closed_Date||0000-00-00 00:00:00||⇒||2018-05-28 14:59:22|
Please help (my Joomla's up to date):
The Article-Newsflash Module doesn't show the Intro or Full image, as offered in the field.
What I found out - sorry, I'm not a programmer:
In the options field there's an english-german language mixup - see the screenshot:
Doesn't the module find the appropriate value?
I couldn't find the file where the values are translated and assigned to the corresponding keys.
I only found the german translation file de-DE_mod_articles-news_ini where the following lines are missing:
MOD_ARTICLES_NEWS_FIELD_IMAGES_ARTICLE_DESC="... german translation ..." MOD_ARTICLES_NEWS_FIELD_IMAGES_ARTICLE_LABEL="... german translation ..."
This is not a big issue, it's only the translation of the label and the description - so I tried and added it on my site. Of course, it didn't help with the bug.
Thanks for your patience!