?
avatar andrepereiradasilva
andrepereiradasilva
15 Mar 2016

Steps to reproduce the issue

  1. Use 3.5.0 RC3
  2. Ativate "User - Profile" plugin
  3. Edit a user in the backend
  4. Try to save.
  5. Error! Please read the Terms of Service. You will not be able to register if you do not agree with them.

Expected result

User changes saved.

Actual result

Error! Please read the Terms of Service. You will not be able to register if you do not agree with them.

System information (as much as possible)

3.5.0 RC3

Additional comments

I think was caused by #9405

@alikon can you have a look

avatar andrepereiradasilva andrepereiradasilva - open - 15 Mar 2016
avatar andrepereiradasilva
andrepereiradasilva - comment - 15 Mar 2016

@brianteeman can you reproduce this also?

avatar franz-wohlkoenig
franz-wohlkoenig - comment - 15 Mar 2016

i'm not @brianteeman but can reproduce this.

avatar andrepereiradasilva
andrepereiradasilva - comment - 15 Mar 2016

@brianteeman 3.5.0 blocker label here?

avatar alikon
alikon - comment - 15 Mar 2016

i'm afraid was my fault in #9405 let me fix.....

avatar brianteeman
brianteeman - comment - 15 Mar 2016

This was reported before

On 15 March 2016 at 18:43, Nicola Galgano notifications@github.com wrote:

i'm afraid was my fault in #9405
#9405 let me fix.....


You are receiving this because you were mentioned.
Reply to this email directly or view it on GitHub
#9438 (comment)

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

avatar brianteeman
brianteeman - comment - 15 Mar 2016

Please see #9405


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

avatar andrepereiradasilva
andrepereiradasilva - comment - 15 Mar 2016

@brianteeman #9405 is a different thing.

avatar brianteeman
brianteeman - comment - 15 Mar 2016

I am confused @alikon 9405 was supposed to be the fix


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

avatar brianteeman
brianteeman - comment - 15 Mar 2016

ok mea culpa for being confused then - it happens - its called old age

On 15 March 2016 at 18:56, andrepereiradasilva notifications@github.com
wrote:

@brianteeman https://github.com/brianteeman #9405
#9405 is a different thing.


You are receiving this because you were mentioned.
Reply to this email directly or view it on GitHub
#9438 (comment)

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

avatar alikon
alikon - comment - 15 Mar 2016

mea culpa for the mistake :innocent:

// Check that the tos are valid.
if ((!($data['profile']['tos'])) && ($this->params->get('register-require_tos', 0) == 2))

as usual my quick dirty trick #9439

avatar wilsonge wilsonge - close - 15 Mar 2016
avatar brianteeman brianteeman - close - 15 Mar 2016
avatar wilsonge wilsonge - change - 15 Mar 2016
Status New Closed
Closed_Date 0000-00-00 00:00:00 2016-03-15 19:24:15
Closed_By wilsonge
avatar wilsonge wilsonge - close - 15 Mar 2016
avatar wilsonge
wilsonge - comment - 15 Mar 2016

Closing as we have a PR

avatar andrepereiradasilva
andrepereiradasilva - comment - 15 Mar 2016

@aikon only those that don't change don't make mistakes.

avatar brianteeman brianteeman - change - 22 Apr 2016
Labels Added: ?

Add a Comment

Login with GitHub to post a comment