User tests: Successful: Unsuccessful:
This PR fixes issue #4623 and several coding errors which I found while reviewing code in view feed of com_tags component:
https://github.com/joomla/joomla-cms/blob/staging/components/com_tags/views/tags/view.feed.php#L68
https://github.com/joomla/joomla-cms/blob/staging/components/com_tags/views/tags/view.feed.php#L72
4. $row->author_email needs to be replaced by $item->author_email
https://github.com/joomla/joomla-cms/blob/staging/components/com_tags/views/tags/view.feed.php#L72
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?
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.
Labels |
Added:
?
|
Category | ⇒ | Tags |
Rel_Number | ⇒ | 4623 | |
Relation Type | ⇒ | Pull Request for |
Status | Pending | ⇒ | Closed |
Closed_Date | 0000-00-00 00:00:00 | ⇒ | 2015-12-08 07:05:41 |
Closed_By | ⇒ | joomdonation |