Skip to content

Fix webservice plugin to show super user group#29649

Merged
wilsonge merged 1 commit intojoomla:4.0-devfrom
wilsonge:feature/fix-groups
Jun 17, 2020
Merged

Fix webservice plugin to show super user group#29649
wilsonge merged 1 commit intojoomla:4.0-devfrom
wilsonge:feature/fix-groups

Conversation

@wilsonge
Copy link
Copy Markdown
Contributor

Summary of Changes

Allow Super User groups to always show in the webservices plugin so that they can view their api tokens.

Testing Instructions

View API plugin. select some user groups. currently super user doesn't show. Save some other user groups to access the token. Before patch a super user loose their ability to view the API token. After patch they can still view the token.

Documentation Changes Required

None

@wilsonge
Copy link
Copy Markdown
Contributor Author

@nikosdion tagging this so you can review

@uglyeoin
Copy link
Copy Markdown
Contributor

I have tested this item ✅ successfully on b7157e7

I have tested this patch. I see the "super user" group and selecting it the user can see the API key.


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

@joomla joomla deleted a comment from uglyeoin Jun 16, 2020
@Bodge-IT
Copy link
Copy Markdown
Contributor

I have tested this item ✅ successfully on b7157e7

I tested this succesfully. Super Users now available for selection/de-selection


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

@alexandreelise
Copy link
Copy Markdown
Contributor

I have tested this item ✅ successfully on b7157e7

I tested this successfully on my Linux laptop on Ubuntu 20.04 LTS with PHP 7.3.17 and a dockerized MySQL 5.7 instance and Joomla! 4.x beta 1


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

@nikosdion
Copy link
Copy Markdown
Contributor

Looks good to me.

@SharkyKZ
Copy link
Copy Markdown
Contributor

RTC.


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

@joomla-cms-bot joomla-cms-bot added the RTC This Pull Request is Ready To Commit label Jun 17, 2020
@wilsonge wilsonge merged commit af6eddf into joomla:4.0-dev Jun 17, 2020
@wilsonge
Copy link
Copy Markdown
Contributor Author

Thanks guys!

@joomla-cms-bot joomla-cms-bot removed the RTC This Pull Request is Ready To Commit label Jun 17, 2020
@wilsonge wilsonge deleted the feature/fix-groups branch June 17, 2020 10:21
@wilsonge wilsonge added this to the Joomla 4.0 milestone Jun 17, 2020
sakiss pushed a commit to sakiss/joomla-cms that referenced this pull request Oct 16, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

7 participants