[#16955] - [com_tags] - feed view fix Notice: Undefined property
- Fixed in Code Base
- 8 Jul 2017
- Medium
- Build: staging
- Easy Test
- # 16955
- Diff
- alikon:patch-91
User tests: Successful: Unsuccessful:
Pull Request for Issue #16951 .
Summary of Changes
fixed item property
Testing Instructions
Load tags feed via /index.php?option=com_tags&view=tags&format=feed
Expected result
feed without Notice
Actual result
Notices in PHP error log
| Status | New | ⇒ | Pending |
joomla-cms-bot
- | Category | ⇒ | com_tags Front End |
I have been unable to replicate the reported error but this change looks wrong to me
Assuming i am reading this correctly before this pr it would display
author alias or author
but now it displays
author alias or user id
and the change to remove the feed email negates the settings in global configuration
as it is now in the model we query only the #__tags table,
so we don't have an author email or author field,
i'll look if we can join #__tags and #__users tables
| Labels |
Added:
?
|
||
is ready to test now
I have tested this item
This comment was created with the J!Tracker Application at issues.joomla.org/tracker/joomla-cms/16955.
I have tested this item
This comment was created with the J!Tracker Application at issues.joomla.org/tracker/joomla-cms/16955.
| Status | Pending | ⇒ | Ready to Commit |
| Easy | No | ⇒ | Yes |
RTC after two successful tests.
rdeutz
- | Status | Ready to Commit | ⇒ | Fixed in Code Base |
| Closed_Date | 0000-00-00 00:00:00 | ⇒ | 2017-07-08 10:00:59 |
| Closed_By | ⇒ | rdeutz | |
| Labels |
Added:
?
|
||
I have tested this item✅ successfully on 4f202b1
This comment was created with the J!Tracker Application at issues.joomla.org/tracker/joomla-cms/16955.