PR-4.0-dev ?

Pending

User tests: Successful: Unsuccessful:

avatar alikon
alikon
6 Apr 2020

Pull Request for Issue #28567 .

Summary of Changes

skip the display of joomlatoken when on registration

Testing Instructions

Enable the user joomla token plugin
Enable api authentication plugin
Enable registration on the front end
Register a new user

Expected result

no Joomla token shown

Actual result

Screenshot from 2020-04-06 10-40-39

Votes

# of Users Experiencing Issue
1/1
Average Importance Score
4.00

avatar alikon alikon - open - 6 Apr 2020
avatar alikon alikon - change - 6 Apr 2020
Status New Pending
avatar joomla-cms-bot joomla-cms-bot - change - 6 Apr 2020
Category Front End com_users
avatar alikon alikon - change - 6 Apr 2020
Labels Added: PR-4.0-dev ?
avatar joomla-cms-bot joomla-cms-bot - change - 6 Apr 2020
Category Front End com_users Front End com_users Plugins
avatar joomla-cms-bot joomla-cms-bot - change - 6 Apr 2020
Category Front End com_users Plugins Front End Plugins
avatar wilsonge
wilsonge - comment - 6 Apr 2020

If we're not going to show this data on registration (which kinda seems logical until they get assigned a user group) then we should just remove it from the list of allowed context's here https://github.com/joomla/joomla-cms/pull/28584/files#diff-618b02bb14a772431ff46eee43117ffeR190

avatar alikon
alikon - comment - 6 Apr 2020

ok removed from the array of allowed contexts

avatar toivo
toivo - comment - 8 Apr 2020

I have tested this item successfully on bbab3e4

Tested successfully in beta1-dev full version 20200408


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

avatar toivo toivo - test_item - 8 Apr 2020 - Tested successfully
avatar wilsonge wilsonge - change - 8 Apr 2020
Status Pending Fixed in Code Base
Closed_Date 0000-00-00 00:00:00 2020-04-08 09:33:51
Closed_By wilsonge
avatar wilsonge wilsonge - close - 8 Apr 2020
avatar wilsonge wilsonge - merge - 8 Apr 2020
avatar wilsonge
wilsonge - comment - 8 Apr 2020

Thanks!

Add a Comment

Login with GitHub to post a comment