Pending

User tests: Successful: Unsuccessful:

avatar joomdonation
joomdonation
2 Oct 2017

Pull Request for Issue # .

Summary of Changes

This PR fixes users custom fields data lost when reset password. See #18185 for issue description

Testing Instructions

  1. Follow the issue #18185, confirm the issue
  2. Apply patch, try to reset password again, make sure users custom fields data is not lost anymore
  3. Try to edit an user account from backend, make sure custom fields data still being saved properly
avatar joomla-cms-bot joomla-cms-bot - change - 2 Oct 2017
Category Front End Plugins
avatar joomdonation joomdonation - open - 2 Oct 2017
avatar joomdonation joomdonation - change - 2 Oct 2017
Status New Pending
avatar Trossler
Trossler - comment - 2 Oct 2017

tested ... running
it works
thx

avatar wojsmol
wojsmol - comment - 3 Oct 2017
avatar franz-wohlkoenig franz-wohlkoenig - alter_testresult - 3 Oct 2017 - Trossler: Tested successfully
avatar franz-wohlkoenig
franz-wohlkoenig - comment - 3 Oct 2017

altered Test of @Trossler at Issue Tracker.

avatar laoneo
laoneo - comment - 3 Oct 2017

This pr doesn't solve the root of the issue. There is a general problem when the save functions are triggered without the custom fields (and any extension which injects data into a form). I tried to solve it in #17837. Because I think we should solve the root issue. Would be good if you can test #17837 as well as this should solve the root of the problem.

avatar joomdonation
joomdonation - comment - 3 Oct 2017

@laoneo I will look at your PR and test it later today after work.

avatar laoneo
laoneo - comment - 3 Oct 2017

Thanks. I guess we have here a similar problem to #17801.

avatar joomdonation
joomdonation - comment - 3 Oct 2017

Yes, you're right. That's why I think that we are doing in wrong way (posted it here #18185 (comment) , too).

avatar Trossler
Trossler - comment - 3 Oct 2017

it is not necessarily the way of the problem I think also. @laoneo @joomdonation

on my current page has been a huge problem with it.
300 members - almost daily was changed a member the password so also the profile fields data - deleted.
there was trouble with the members ... so your help was really great

avatar franz-wohlkoenig
franz-wohlkoenig - comment - 26 Oct 2017

@joomdonation @laoneo how to go on with this Pull request?

avatar laoneo
laoneo - comment - 26 Oct 2017

I would go with #17837 as it fixes the root cause of the issue.

avatar franz-wohlkoenig
franz-wohlkoenig - comment - 26 Oct 2017

Thanks @laoneo. If this Issue get no other Response, it will be closed at 26th November 2017.

avatar joomdonation joomdonation - change - 12 Nov 2017
Status Pending Closed
Closed_Date 0000-00-00 00:00:00 2017-11-12 10:35:19
Closed_By joomdonation
avatar joomdonation joomdonation - close - 12 Nov 2017

Add a Comment

Login with GitHub to post a comment