Branch: refs/heads/master
Home: https://github.com/phpmyadmin/phpmyadmin
Commit: 5acfb452628f4044ff2f096ca4b3c080c73102bd
https://github.com/phpmyadmin/phpmyadmin/commit/5acfb452628f4044ff2f096ca4b…
Author: Maurício Meneghini Fauth <mauricio(a)fauth.dev>
Date: 2022-08-23 (Tue, 08 August 2022) -03:00
Changed paths:
M libraries/classes/Common.php
M psalm-baseline.xml
Log Message:
-----------
Fix some errors in Common class found by Psalm
Signed-off-by: Maurício Meneghini Fauth <mauricio(a)fauth.dev>
Branch: refs/heads/master
Home: https://github.com/phpmyadmin/phpmyadmin
Commit: f7f8722e1f013afbb0332cd9d03999ef54c293fa
https://github.com/phpmyadmin/phpmyadmin/commit/f7f8722e1f013afbb0332cd9d03…
Author: Kamil Tekiela <tekiela246(a)gmail.com>
Date: 2022-08-22 (Mon, 08 August 2022) -03:00
Changed paths:
A libraries/classes/EditField.php
M libraries/classes/Controllers/Table/ReplaceController.php
M libraries/classes/InsertEdit.php
M phpstan-baseline.neon
M psalm-baseline.xml
M test/classes/InsertEditTest.php
M test/classes/Stubs/DbiDummy.php
Log Message:
-----------
Refactor insert edit (#17684)
This is a major redesign of the code that handles these four actions: in-place edit(AJAX), edit of multiple rows, copying of rows, and insertion of new rows. The goal is to make the code easier to read and more understandable.
I introduced a new DTO for better readability.
Some of the methods were made private to the model
The controller has access to two methods for getting the value for INSERT and for UPDATE
Unfortunately, I have introduced method envy on EditField, but I do not want to move the functionality away from InsertEdit.php
Unit tests have not improved in readability but I added helpful comments explaining what we are testing. There's probably a way to improve it too.
I will probably merge all commits once I decide it's ready for review.
Signed-off-by: Kamil Tekiela <tekiela246(a)gmail.com>