? Pending

User tests: Successful: Unsuccessful:

avatar rdeutz
rdeutz
14 Apr 2017

Pull Request for Issue #15194

Summary of Changes

When activating a account we don't need to process the field check/save procedure

Testing Instructions

  • Add a text field to a user e.g. Company
  • Register a user (have set: New user account activation to administrator)
  • process the activation

Data get's lost

Apply patch and do the same process

Data will not get lost

avatar rdeutz rdeutz - open - 14 Apr 2017
avatar rdeutz rdeutz - change - 14 Apr 2017
Status New Pending
avatar joomla-cms-bot joomla-cms-bot - change - 14 Apr 2017
Category Front End Plugins
avatar AlexRed AlexRed - test_item - 14 Apr 2017 - Tested successfully
avatar AlexRed
AlexRed - comment - 14 Apr 2017

I have tested this item successfully on 8e66e18

Patch ok for me


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

avatar rdeutz rdeutz - change - 14 Apr 2017
Labels Added: ?
avatar franz-wohlkoenig franz-wohlkoenig - test_item - 15 Apr 2017 - Tested successfully
avatar franz-wohlkoenig
franz-wohlkoenig - comment - 15 Apr 2017

I have tested this item successfully on f6126d3


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

avatar franz-wohlkoenig
franz-wohlkoenig - comment - 15 Apr 2017

@AlexRed can you please retest?

avatar laoneo
laoneo - comment - 15 Apr 2017

I guess for now, we are good to go. But this issue revealed a problem which I'm discussing right now with @Bakual to find a proper solution. The we will probably revert that change ?

avatar laoneo laoneo - test_item - 15 Apr 2017 - Tested successfully
avatar laoneo
laoneo - comment - 15 Apr 2017

I have tested this item successfully on e15aad7


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

avatar rdeutz
rdeutz - comment - 15 Apr 2017

we will probably revert that change

Never ever I will merge it ;-)

avatar Bakual Bakual - change - 15 Apr 2017
Status Pending Fixed in Code Base
Closed_Date 0000-00-00 00:00:00 2017-04-15 15:21:13
Closed_By Bakual
avatar Bakual Bakual - close - 15 Apr 2017
avatar Bakual Bakual - merge - 15 Apr 2017
avatar Bakual
Bakual - comment - 15 Apr 2017

I think for this case the fix is fine anyway. Regardless if we need some handling elsewhere as well.

Add a Comment

Login with GitHub to post a comment