?
avatar beat
beat
15 Jul 2020

I have found a major B/C API issue here:

public function store($updateNulls = true)

  1. This is inconsistent/incompatible with declarations of parent classes Table and TableInterface, and thus breaks the defined PHP interface.
  2. More importantly, this also breaks B/C, and Community Builder when saving user profile in frontend, when changing e.g. just the name, but keeping password same (and thus the corresponding $user->password null, which generates an exception and corresponding fatal Error "Column 'password' cannot be null". And an ugly workaround would mean to read the hashed password and re-store it on update.
avatar beat beat - open - 15 Jul 2020
avatar joomla-cms-bot joomla-cms-bot - change - 15 Jul 2020
Labels Added: ?
avatar joomla-cms-bot joomla-cms-bot - labeled - 15 Jul 2020
avatar SharkyKZ SharkyKZ - change - 15 Jul 2020
Status New Closed
Closed_Date 0000-00-00 00:00:00 2020-07-15 15:03:33
Closed_By SharkyKZ
avatar SharkyKZ SharkyKZ - close - 15 Jul 2020
avatar SharkyKZ
SharkyKZ - comment - 15 Jul 2020

Closing as there is PR #30108.

Add a Comment

Login with GitHub to post a comment