[#17463] - User list : modal view : un-activated icon not shown in certain cases
- Fixed in Code Base
- 13 Aug 2017
- Medium
- Build: staging
- Easy Test
- # 17463
- Diff
- 123monsite-regis:patch-1
User tests: Successful: Unsuccessful:
Pull Request for Issue # .
Summary of Changes
Administrator : User list : modal view : un-activated icon not shown in certain cases
field was cast to int instead if empty or not.
it is done after this patch as on default users list view : https://github.com/joomla/joomla-cms/blob/staging/administrator/components/com_users/views/users/tmpl/default.php#L137
Testing Instructions
- create a certain number of users, activated and unactivated ones
- find a modal user list view on administrator ( you have them for example on com_content > articles > author, but you can also create a new wondefull 3.7's custom field of type user)
- then you can see the modal view
Expected result
Actual result
Documentation Changes Required
none
joomla-cms-bot
- | Category | ⇒ | Administration com_users |
| Status | New | ⇒ | Pending |
| Easy | No | ⇒ | Yes |
I have tested this item
Was able to duplicate issue and resolve with the patch. :)
This comment was created with the J!Tracker Application at issues.joomla.org/tracker/joomla-cms/17463.
| Status | Pending | ⇒ | Ready to Commit |
RTC after two successful tests.
#17463 (comment) has first to be corrected before merging
| Labels |
Added:
?
|
||
I have fixed the codestyle issue that @infograf768 refers to above
thanks for the tests and the fix of the extra-space,
have a nice day.
mbabker
- | Status | Ready to Commit | ⇒ | Fixed in Code Base |
| Closed_Date | 0000-00-00 00:00:00 | ⇒ | 2017-08-13 15:41:42 |
| Closed_By | ⇒ | mbabker | |
| Labels |
Added:
?
|
||
I have tested this item✅ successfully on 55b5e5e
This comment was created with the J!Tracker Application at issues.joomla.org/tracker/joomla-cms/17463.