? Success

User tests: Successful: Unsuccessful:

avatar andrepereiradasilva
andrepereiradasilva
8 Nov 2016

Pull Request for New Issue.

Summary of Changes

There are several notices in com_users usergroup edit view.

Notice: Undefined property: stdClass::$value in /path/to/joomla-staging/administrator/components/com_users/models/fields/groupparent.php on line 52

This PR solves them

Testing Instructions

  1. Latest staging
  2. Create a user with Administrator group
  3. Login with that user
  4. Go to Usergroups and edit Manager Group
  5. You will get several php notices
  6. Apply path
  7. Repeat steps and check all is fine now

Documentation Changes Required

None.

avatar andrepereiradasilva andrepereiradasilva - open - 8 Nov 2016
avatar andrepereiradasilva andrepereiradasilva - change - 8 Nov 2016
Status New Pending
avatar joomla-cms-bot joomla-cms-bot - change - 8 Nov 2016
Category Administration Components
avatar andrepereiradasilva andrepereiradasilva - change - 8 Nov 2016
The description was changed
Labels Added: ?
avatar andrepereiradasilva andrepereiradasilva - edited - 8 Nov 2016
avatar andrepereiradasilva andrepereiradasilva - change - 8 Nov 2016
The description was changed
avatar andrepereiradasilva andrepereiradasilva - edited - 8 Nov 2016
avatar andrepereiradasilva andrepereiradasilva - change - 8 Nov 2016
Title
Solve php warnings in usergroups edit view
Solve php notices in usergroups edit view
avatar andrepereiradasilva andrepereiradasilva - change - 8 Nov 2016
Title
Solve php warnings in usergroups edit view
Solve php notices in usergroups edit view
avatar andrepereiradasilva andrepereiradasilva - edited - 8 Nov 2016
avatar andrepereiradasilva andrepereiradasilva - change - 8 Nov 2016
The description was changed
avatar andrepereiradasilva andrepereiradasilva - edited - 8 Nov 2016
avatar yvesh
yvesh - comment - 13 Nov 2016

@mbabker Looks like the issue tracker is broken on this one (Missing commit SHA). We tried with 2 computers / different accounts now at Joomla! world congress..

Are we missing something?

avatar alikon
alikon - comment - 13 Nov 2016

for me is a successfull test but i was not ablr to mark on issues site

avatar ralain
ralain - comment - 13 Nov 2016

Unable to submit successful test because of missing commit SHA.

But issue confirmed and it's fixed when the patch is applied.

avatar ggppdk
ggppdk - comment - 13 Nov 2016

Successfully tested
also did code review, it is good

This now has 3 successful tests

avatar infograf768 infograf768 - change - 13 Nov 2016
Status Pending Ready to Commit
avatar infograf768
infograf768 - comment - 13 Nov 2016

As 3 successful test but there were issues with sha (that looks solve now), I mark this as RTC.


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

avatar mbabker mbabker - alter_testresult - 13 Nov 2016 - alikon: Tested successfully
avatar mbabker mbabker - alter_testresult - 13 Nov 2016 - ralain: Tested successfully
avatar mbabker mbabker - alter_testresult - 13 Nov 2016 - ggppdk: Tested successfully
avatar zero-24 zero-24 - change - 13 Nov 2016
Milestone Added:
avatar rdeutz rdeutz - reference | 8bc8984 - 15 Nov 16
avatar rdeutz rdeutz - merge - 15 Nov 2016
avatar rdeutz rdeutz - close - 15 Nov 2016
avatar rdeutz rdeutz - change - 15 Nov 2016
Status Ready to Commit Fixed in Code Base
Closed_Date 0000-00-00 00:00:00 2016-11-15 21:43:00
Closed_By rdeutz
avatar rdeutz rdeutz - close - 15 Nov 2016
avatar rdeutz rdeutz - merge - 15 Nov 2016
avatar joomla-cms-bot joomla-cms-bot - change - 15 Nov 2016
Category Administration Components Administration com_users Components
avatar andrepereiradasilva andrepereiradasilva - head_ref_deleted - 15 Nov 2016

Add a Comment

Login with GitHub to post a comment