J3 Issue ?
avatar sozzled
sozzled
15 Mar 2018

Steps to reproduce the issue

Testing site at http://joomla37.enduring.com.au/index.php/news-feed-tests

The sources for these news feeds—the XML files—can be examined.

Expected result

The URLs for should generate URLs based on the <link> element for each <item> in the XML feed.

Actual result

The URLs for generate URLs based on the <guid> element for each <item> in the XML feed.

For the test site: "News Feed 4" generates a 404 error (and I don't know why). The URLs for "News Feed 1" and "3" and "5" generate valid URLs that will display the external content; the URLs for "News Feed 2" and "6" generate invalid URLs.

System information (as much as possible)

See full discussion on the Joomla forum at https://forum.joomla.org/viewtopic.php?f=706&t=960129

Additional comments

Lines 94-95 of /components/com_newsfeeds/views/newsfeed/tmpl/default.php probably apply

avatar sozzled sozzled - open - 15 Mar 2018
avatar joomla-cms-bot joomla-cms-bot - labeled - 15 Mar 2018
avatar sozzled sozzled - change - 15 Mar 2018
The description was changed
avatar sozzled sozzled - edited - 15 Mar 2018
avatar sozzled sozzled - change - 15 Mar 2018
The description was changed
avatar sozzled sozzled - edited - 15 Mar 2018
avatar sozzled sozzled - change - 15 Mar 2018
The description was changed
avatar sozzled sozzled - edited - 15 Mar 2018
avatar franz-wohlkoenig franz-wohlkoenig - change - 16 Mar 2018
Build J! 3.x 3.x
avatar pmleconte
pmleconte - comment - 21 Mar 2018

As mentionned in above forum's discussion, component com_newfeed and module mod_feed are not consistent:
com-newfeed line (line 94)
$uri = !empty($this->rssDoc[$i]->guid) || $this->rssDoc[$i]->guid !== null ? trim($this->rssDoc[$i]->guid) : trim($this->rssDoc[$i]->uri);
mod-feed (line 93)
$uri = (!empty($feed[$i]->uri) || $feed[$i]->uri !== null) ? trim($feed[$i]->uri) : trim($feed[$i]->guid);

Please note that this is still like this in 4.0 aplha 2.

Pascal

avatar franz-wohlkoenig franz-wohlkoenig - change - 21 Mar 2018
Status New Discussion
avatar brianteeman brianteeman - change - 25 Mar 2018
Labels Added: J3 Issue
avatar brianteeman brianteeman - labeled - 25 Mar 2018
avatar SharkyKZ
SharkyKZ - comment - 11 Jun 2018

@sozzled see PR #20717.

avatar franz-wohlkoenig franz-wohlkoenig - change - 11 Jun 2018
Status Discussion Closed
Closed_Date 0000-00-00 00:00:00 2018-06-11 13:28:49
Closed_By franz-wohlkoenig
avatar joomla-cms-bot joomla-cms-bot - change - 11 Jun 2018
Closed_By franz-wohlkoenig joomla-cms-bot
avatar joomla-cms-bot joomla-cms-bot - close - 11 Jun 2018
avatar joomla-cms-bot
joomla-cms-bot - comment - 11 Jun 2018
avatar franz-wohlkoenig
franz-wohlkoenig - comment - 11 Jun 2018

closed as having Pull Request #20717

avatar sozzled
sozzled - comment - 11 Jun 2018

Thanks.

Add a Comment

Login with GitHub to post a comment