? Success
Referenced as Related to: # 4623

User tests: Successful: Unsuccessful:

avatar joomdonation
joomdonation
13 Oct 2014

#PR summary:

This PR fixes issue #4623 reported by @javigomez and several coding errors which I found while reviewing code in view feed of com_tags component:

  1. The getListQuery method of TagsModelTags component doesn't return "author" and "author_email", so there are notice errors in the following lines of code in view feed:

https://github.com/joomla/joomla-cms/blob/staging/components/com_tags/views/tags/view.feed.php#L56
https://github.com/joomla/joomla-cms/blob/staging/components/com_tags/views/tags/view.feed.php#L74

  1. $item->displayDate in the line of code https://github.com/joomla/joomla-cms/blob/staging/components/com_tags/views/tags/view.feed.php#L57 need to be replaced with $item->created_time (there is no field displayDate in the $item object)

  2. $item in the following lines of code need to be replaced by $feeditem

https://github.com/joomla/joomla-cms/blob/staging/components/com_tags/views/tags/view.feed.php#L70
https://github.com/joomla/joomla-cms/blob/staging/components/com_tags/views/tags/view.feed.php#L74

  1. $row->author_email needs to be replaced by $item->author_email (I think it is typing mistake)

https://github.com/joomla/joomla-cms/blob/staging/components/com_tags/views/tags/view.feed.php#L74

#Testing instructions:
For some reasons, these notice errors are not displayed when I tried to view the feed, so it will be difficult for end-users to test this pull request (these errors are logged in error logs of the web server). Maybe one of PLTs (or developers who understand Joomla code) can help reviewing this code and get it merged? @mbabker Could you help review this code? The errors are clearly mentioned above.

For end-users, the only think you can test is:

  • View rss feed for tags component. You will see that for each feed item, the author and pubDate tag are not available for each feed item.
  • Apply this pull request. Check it again and you will see that author and pubDate are available now for each feed item.
avatar joomdonation joomdonation - open - 13 Oct 2014
avatar jissues-bot jissues-bot - change - 13 Oct 2014
Labels Added: ?
avatar javigomez
javigomez - comment - 13 Oct 2014

Thanks @joomdonation ^_^

avatar joomdonation
joomdonation - comment - 13 Oct 2014

You're welcome @javigomez . Hopefully this PR can be reviewed and merged (It is quite difficult to provide testing instructions for these kind of PRs :( ).

avatar brianteeman brianteeman - change - 14 Oct 2014
Category Tags
avatar nicksavov nicksavov - change - 16 Oct 2014
Labels Added: ?
avatar rajeshstarlite
rajeshstarlite - comment - 17 Oct 2014

@Test, I've tested this issue, please find my test cases as screen shots:screen shot 2014-10-17 at 05 25 30screen shot 2014-10-17 at 05 25 36screen shot 2014-10-17 at 05 25 54

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

avatar rajeshstarlite rajeshstarlite - test_item - 17 Oct 2014 - Tested unsuccessfully
avatar joomdonation
joomdonation - comment - 17 Oct 2014

Thanks @rajeshstarlite for testing this PR. Unfortunately, I deleted the original repo and now I could not update the PR with new code. Could you please help making another change and test it again:

  1. Open the file components/com_tags/views/tags/view.feed.php

  2. Find the code below:

$feeditem->link        = '/index.php?option=com_tags&view=tag&id=' . (int) $item->id;
  1. Change it to:
$feeditem->link        = JRoute::_('index.php?option=com_tags&view=tag&id=' . (int) $item->id);

After that, please help test it again. If everything goes well, I will open a new PR to fix this issue.

For some reason, I could not see the author displayed as well. Maybe it depends on the rss reader which we are using. If you view the source of the feed, you will see the is generated like below (the author tag is available but not displayed)

<item>
<title>Components</title>
<link>http://localhost/jcms/index.php?option=com_tags&amp;view=tag&amp;id=2&amp;Itemid=582</link>
<guid isPermaLink="true">http://localhost/jcms/index.php?option=com_tags&amp;view=tag&amp;id=2&amp;Itemid=582</guid>
<description><![CDATA[<p>Components</p>]]></description>
<author>tuanpn@joomdonation.com (Super User)</author>
<category>All Tags</category>
<pubDate>Mon, 13 Oct 2014 03:59:05 +0000</pubDate>

Sorry for could not reply you earlier. It was my office time, so I could not work on it until now (after dinner :) )

Thanks again

avatar joomdonation
joomdonation - comment - 29 Dec 2014

@Hackwar @wilsonge I created a new pull request #5552 to replace this one (because I deleted the original repo and don't know how I can edit code without it). Please help checking the new PR when you have some time.

avatar joomdonation joomdonation - close - 29 Dec 2014
avatar joomdonation joomdonation - change - 29 Dec 2014
Status Pending Closed
Closed_Date 0000-00-00 00:00:00 2014-12-29 14:20:08

Add a Comment

Login with GitHub to post a comment