?
avatar mannybiker
mannybiker
7 Apr 2016

Steps to reproduce the issue

Activate the user profile plugin
Choose TOS field as mandatory
Try to register leaving or selecting "no" as choice in the TOS radio button

Expected result

The form should be rejected and user should not be created

Actual result

The form is accepted and the user is created

System information (as much as possible)

Joomla 3.5.0

Additional comments

Client side validation seems not expected but also the server side validation is not triggered due to a misspelled com_users component

Here the commit with the simple fix
TOS fix commit

avatar mannybiker mannybiker - open - 7 Apr 2016
avatar brianteeman
brianteeman - comment - 7 Apr 2016

Please can you submit that as a Pull Request so that it can be tested

On 7 April 2016 at 03:34, mannybiker notifications@github.com wrote:

Steps to reproduce the issue

Activate the user profile plugin
Choose TOS field as mandatory
Try to register leaving or selecting "no" as choice in the TOS radio button
Expected result

The form should be rejected and user should not be created
Actual result

The form is accepted and the user is created
System information (as much as possible)

Joomla 3.5.0
Additional comments

Client side validation seems not expected but also the server side
validation is not triggered due to a misspelled com_users component

Here the commit with the simple fix

mannybiker@8699552
http://url


You are receiving this because you are subscribed to this thread.
Reply to this email directly or view it on GitHub
#9777

Brian Teeman
Co-founder Joomla! and OpenSourceMatters Inc.
http://brian.teeman.net/

avatar brianteeman brianteeman - change - 7 Apr 2016
Category Plugins
avatar brianteeman brianteeman - change - 7 Apr 2016
Labels Added: ?
avatar mannybiker
mannybiker - comment - 7 Apr 2016

@brianteeman Sure Brian, it is done
#9792

avatar brianteeman
brianteeman - comment - 7 Apr 2016

Closed as we have a PR for testing

avatar brianteeman brianteeman - change - 7 Apr 2016
Status New Closed
Closed_Date 0000-00-00 00:00:00 2016-04-07 17:43:32
Closed_By brianteeman
avatar brianteeman brianteeman - close - 7 Apr 2016

Add a Comment

Login with GitHub to post a comment