[#38058] - [5.2] Fix for missing Fulltext Image in Newsfeeds
- Pending
- Medium
- Build: 4.1-dev
- # 38058
- Diff
- AndySDH:patch-12
User tests: Successful: Unsuccessful:
As requested, Joomla 4.2 re-do of Pull Request #28195
Summary of Changes
Originally Joomla RSS Newsfeeds were completely missing the image_intro and image_fulltext. This was partially fixed with this PR: #11402
But that only introduced the image_intro, and forgot the image_fulltext.
So an article that had an image_fulltext would not use that image in the feed.
This PR simply adds the image_fulltext to the feed if the image_intro is empty, for both the category view and featured view.
| Status | New | ⇒ | Pending |
joomla-cms-bot
- | Category | ⇒ | Unit Tests Repository Administration com_admin SQL |
Quy
- | Labels |
Added:
?
|
||
| Category | Unit Tests Repository Administration com_admin SQL | ⇒ | Front End com_content |
@richard67 Should be done :)
@richard67 Should be done :)
@AndySDH Not really. There are still some complaints: https://ci.joomla.org/joomla/joomla-cms/55316/1/6
@AndySDH Still not ok: https://ci.joomla.org/joomla/joomla-cms/55320/1/6 .
Well it's going in circles a bit. If I take out those spaces, the indentation will look off. I'm not sure that's a valid complaint there, not sure.
Well it's going in circles a bit. If I take out those spaces, the indentation will look off. I'm not sure that's a valid complaint there, not sure.
@AndySDH It is a valid complaint. The indentation of the “=“ and the lines below do not need to be aligned. Furthermore the lines below should be indented with tabs. I would also suggest to put the operators “?” and “:” of the ternaries to the beginning of a line and not the end of the previous line. And I would indent the 2nd ternary by one tab more than the first one. Maybe check how it is done elsewhere.
Thanks for the suggestions :) Should be good now hopefully.
This pull requests has been automatically converted to the PSR-12 coding standard.
| Labels |
Added:
?
Maintainers Checked
?
|
||
The images are completely missing from the tags rss feed, can somebody look into this?
If you create a menu item with tags (Menu Item Type: Tagged Items) and check the RSS feed it will contain no images, also the contains the item title, not the category.
@GixxerMKD Solved with PR #38312 .
@GixxerMKD Solved with PR #38312 .
Not same issue, this is the file in question:
components/com_tags/src/View/Tag/FeedView.php
no image for item and wrong code for category:
($feeditem->category = $title;)
This pull request has been automatically rebased to 5.0-dev.
| Title |
|
||||||
| Labels |
Added:
Feature
PR-4.3-dev
PR-5.0-dev
?
Removed: ? ? |
||
Hello, I've updated the code with a cleaner and more simple approach.
Can this be reviewed and merged please?
@laoneo How comes? This is a minor new functionality, should be perfectly fine for a minor release like eg. 4.3 or 4.4.
PS: When you changed the base branch to 5.0, my PR's "commits" got totally screwed up, I only updated 2 files, this now looks like I updated 94 files, which is not the case.
| Category | Front End com_content | ⇒ | Administration com_joomlaupdate com_templates Language & Strings Modules Repository NPM Change Front End com_content Installation |
obuisard
- | Title |
|
||||||
| Category | Front End com_content Administration com_joomlaupdate com_templates Language & Strings Modules Repository NPM Change Installation | ⇒ | Repository Administration com_admin com_banners com_categories com_config com_contact com_content com_fields com_finder com_installer |
| Labels |
Added:
Language Change
NPM Resource Changed
Removed: PR-4.3-dev ? |
||
This pull request has been automatically rebased to 5.1-dev.
Confirmed this works. ✅
I created an article with only a full text image. Left the intro image blank. The feed now will show the full text image in the feed.
I have tested this item ✅ successfully on b2d557e
This comment was created with the J!Tracker Application at issues.joomla.org/tracker/joomla-cms/38058.
This pull request has been automatically rebased to 5.2-dev.
| Title |
|
||||||
| Category | com_content Administration Repository com_admin com_banners com_categories com_config com_contact com_fields com_finder com_installer | ⇒ | Front End com_content |
@AndySDH Could you fix the code style errors reported by drone here https://ci.joomla.org/joomla/joomla-cms/55308/1/6 ? Thanks in advance.