Branch: refs/heads/master
Home: https://github.com/phpmyadmin/phpmyadmin
Commit: a42803297c3110ac10247a5e5a4b663bf1849f59
https://github.com/phpmyadmin/phpmyadmin/commit/a42803297c3110ac10247a5e5a4…
Author: Kamil Tekiela <tekiela246(a)gmail.com>
Date: 2021-12-03 (Fri, 12 December 2021) -03:00
Changed paths:
M libraries/classes/Controllers/Export/ExportController.php
M libraries/classes/Database/Qbe.php
M libraries/classes/Database/Triggers.php
M libraries/classes/Display/Results.php
M libraries/classes/Plugins/Export/ExportSql.php
M libraries/classes/Server/Privileges.php
M libraries/classes/Utils/HttpRequest.php
M phpstan-baseline.neon
M psalm-baseline.xml
M test/classes/Plugins/Export/ExportSqlTest.php
M test/classes/Server/PrivilegesTest.php
Log Message:
-----------
Psalm fixes 3 (#17221)
* Update psalm-baseline.xml
Signed-off-by: Kamil Tekiela <tekiela246(a)gmail.com>
* Small bug fixes
Signed-off-by: Kamil Tekiela <tekiela246(a)gmail.com>
* non-falsy-string is always true
Signed-off-by: Kamil Tekiela <tekiela246(a)gmail.com>
* currentSearch property is nullable
Signed-off-by: Kamil Tekiela <tekiela246(a)gmail.com>
* Remove unusued parameter $columnIndex
Signed-off-by: Kamil Tekiela <tekiela246(a)gmail.com>
* Use parameters instead of globals in sendEditor()
The two globals in this method are read-only. There is an unused
parameter in the signature. Let's add one more and use them instead of globals.
Signed-off-by: Kamil Tekiela <tekiela246(a)gmail.com>
* Remove unusued parameter $crlf
Signed-off-by: Kamil Tekiela <tekiela246(a)gmail.com>
Branch: refs/heads/master
Home: https://github.com/phpmyadmin/phpmyadmin
Commit: f446e6da88645921d82e2d2ab5a3ed91f5b21604
https://github.com/phpmyadmin/phpmyadmin/commit/f446e6da88645921d82e2d2ab5a…
Author: Kamil Tekiela <tekiela246(a)gmail.com>
Date: 2021-12-03 (Fri, 12 December 2021) +00:00
Changed paths:
M libraries/classes/Controllers/Database/Structure/FavoriteTableController.php
M libraries/classes/Controllers/Database/StructureController.php
M libraries/classes/Controllers/ImportController.php
M libraries/classes/Controllers/Table/ChangeRowsController.php
M libraries/classes/Controllers/Table/ChartController.php
M libraries/classes/Controllers/Table/ExportRowsController.php
M libraries/classes/Controllers/Table/Structure/BrowseController.php
M libraries/classes/Controllers/Transformation/WrapperController.php
M libraries/classes/Database/Qbe.php
M libraries/classes/Display/Results.php
M libraries/classes/Linter.php
M libraries/classes/Plugins.php
M libraries/classes/Plugins/Auth/AuthenticationSignon.php
M libraries/classes/Plugins/Export/ExportSql.php
M libraries/classes/Plugins/Export/ExportYaml.php
M libraries/classes/Plugins/Export/Helpers/Pdf.php
M libraries/classes/Plugins/Import/ImportCsv.php
M libraries/classes/Table.php
M libraries/classes/Table/ColumnsDefinition.php
M libraries/classes/Table/Indexes.php
M libraries/classes/Tracking.php
M libraries/classes/Utils/HttpRequest.php
M phpstan-baseline.neon
M psalm-baseline.xml
Log Message:
-----------
Remove unused variables
Signed-off-by: Kamil Tekiela <tekiela246(a)gmail.com>
Commit: ef72b89fa8da9c7d842a48c53fe58fddd0d5819c
https://github.com/phpmyadmin/phpmyadmin/commit/ef72b89fa8da9c7d842a48c53fe…
Author: MaurÃcio Meneghini Fauth <mauricio(a)fauth.dev>
Date: 2021-12-03 (Fri, 12 December 2021) -03:00
Changed paths:
M libraries/classes/Controllers/Database/Structure/FavoriteTableController.php
M libraries/classes/Controllers/Database/StructureController.php
M libraries/classes/Controllers/ImportController.php
M libraries/classes/Controllers/Table/ChangeRowsController.php
M libraries/classes/Controllers/Table/ChartController.php
M libraries/classes/Controllers/Table/ExportRowsController.php
M libraries/classes/Controllers/Table/Structure/BrowseController.php
M libraries/classes/Controllers/Transformation/WrapperController.php
M libraries/classes/Database/Qbe.php
M libraries/classes/Display/Results.php
M libraries/classes/Linter.php
M libraries/classes/Plugins.php
M libraries/classes/Plugins/Auth/AuthenticationSignon.php
M libraries/classes/Plugins/Export/ExportSql.php
M libraries/classes/Plugins/Export/ExportYaml.php
M libraries/classes/Plugins/Export/Helpers/Pdf.php
M libraries/classes/Plugins/Import/ImportCsv.php
M libraries/classes/Table.php
M libraries/classes/Table/ColumnsDefinition.php
M libraries/classes/Table/Indexes.php
M libraries/classes/Tracking.php
M libraries/classes/Utils/HttpRequest.php
M phpstan-baseline.neon
M psalm-baseline.xml
Log Message:
-----------
Merge pull request #17220 from kamil-tekiela/Psalm-fixes-2
Remove unused variables
Branch: refs/heads/master
Home: https://github.com/phpmyadmin/phpmyadmin
Commit: 488e7cb5bf5e7ce1197d5bb65469c7a6157aa64f
https://github.com/phpmyadmin/phpmyadmin/commit/488e7cb5bf5e7ce1197d5bb6546…
Author: Kamil Tekiela <tekiela246(a)gmail.com>
Date: 2021-11-29 (Mon, 11 November 2021) +00:00
Changed paths:
M libraries/classes/BrowseForeigners.php
M libraries/classes/Config/FormDisplay.php
M libraries/classes/Controllers/Setup/HomeController.php
M libraries/classes/Controllers/Table/ChartController.php
M libraries/classes/Controllers/Table/ReplaceController.php
M libraries/classes/Database/CentralColumns.php
M libraries/classes/Database/Designer/Common.php
M libraries/classes/Operations.php
M libraries/classes/Plugins/Auth/AuthenticationCookie.php
M libraries/classes/Plugins/AuthenticationPlugin.php
M libraries/classes/Plugins/Export/ExportSql.php
M libraries/classes/Plugins/Schema/TableStats.php
M libraries/classes/Server/UserGroups.php
M libraries/classes/Sql.php
M libraries/classes/Table.php
M libraries/classes/Table/ColumnsDefinition.php
M psalm-baseline.xml
M test/classes/AbstractTestCase.php
M test/classes/Config/DescriptionTest.php
Log Message:
-----------
Use array_keys whenever only keys are needed in foreach
Signed-off-by: Kamil Tekiela <tekiela246(a)gmail.com>
Commit: acff73aa247e00afb84760b63c1174acd78716e5
https://github.com/phpmyadmin/phpmyadmin/commit/acff73aa247e00afb84760b63c1…
Author: MaurÃcio Meneghini Fauth <mauricio(a)fauth.dev>
Date: 2021-12-02 (Thu, 12 December 2021) -03:00
Changed paths:
M libraries/classes/BrowseForeigners.php
M libraries/classes/Config/FormDisplay.php
M libraries/classes/Controllers/Setup/HomeController.php
M libraries/classes/Controllers/Table/ChartController.php
M libraries/classes/Controllers/Table/ReplaceController.php
M libraries/classes/Database/CentralColumns.php
M libraries/classes/Database/Designer/Common.php
M libraries/classes/Operations.php
M libraries/classes/Plugins/Auth/AuthenticationCookie.php
M libraries/classes/Plugins/AuthenticationPlugin.php
M libraries/classes/Plugins/Export/ExportSql.php
M libraries/classes/Plugins/Schema/TableStats.php
M libraries/classes/Server/UserGroups.php
M libraries/classes/Sql.php
M libraries/classes/Table.php
M libraries/classes/Table/ColumnsDefinition.php
M psalm-baseline.xml
M test/classes/AbstractTestCase.php
M test/classes/Config/DescriptionTest.php
Log Message:
-----------
Merge pull request #17215 from kamil-tekiela/Psalm-fixes-1
Use array_keys whenever only keys are needed in foreach
Branch: refs/heads/master
Home: https://github.com/phpmyadmin/phpmyadmin
Commit: 1270efc2a9205f5160262ecbd33558203f9e21a1
https://github.com/phpmyadmin/phpmyadmin/commit/1270efc2a9205f5160262ecbd33…
Author: Kamil Tekiela <tekiela246(a)gmail.com>
Date: 2021-12-02 (Thu, 12 December 2021) -03:00
Changed paths:
M libraries/classes/Controllers/Table/ZoomSearchController.php
M libraries/classes/Display/Results.php
M libraries/classes/InsertEdit.php
M libraries/classes/Plugins/Export/ExportSql.php
M libraries/classes/Util.php
M phpstan-baseline.neon
M psalm-baseline.xml
M test/classes/Display/ResultsTest.php
M test/classes/UtilTest.php
Log Message:
-----------
Refactor util.php (#17212)
* Remove extra space
Signed-off-by: Kamil Tekiela <tekiela246(a)gmail.com>
* Remove null checks on properties that cannot be null
The value could be null only if the property exists in stdClass
returned by mysqli but is null. Mysqli doesn't do that. Unless the value
can be null in some other way, this check is redundant.
Signed-off-by: Kamil Tekiela <tekiela246(a)gmail.com>
* Add psalm-return
Signed-off-by: Kamil Tekiela <tekiela246(a)gmail.com>
* Remove $handle parameter of getUniqueCondition()
Signed-off-by: Kamil Tekiela <tekiela246(a)gmail.com>
* Type hint the $row parameter
Signed-off-by: Kamil Tekiela <tekiela246(a)gmail.com>
Co-authored-by: MaurÃcio Meneghini Fauth <mauricio(a)fauth.dev>
Branch: refs/heads/master
Home: https://github.com/phpmyadmin/phpmyadmin
Commit: dea4c972afce6ba8d7bc904d2706924befc503de
https://github.com/phpmyadmin/phpmyadmin/commit/dea4c972afce6ba8d7bc904d270…
Author: Kamil Tekiela <tekiela246(a)gmail.com>
Date: 2021-12-02 (Thu, 12 December 2021) -03:00
Changed paths:
M libraries/classes/Relation.php
M psalm-baseline.xml
Log Message:
-----------
Refactor relations.php (#17210)
* Refactor getComments()
Use early return and remove redundant if statement.
Signed-off-by: Kamil Tekiela <tekiela246(a)gmail.com>
* Fix indentation
Signed-off-by: Kamil Tekiela <tekiela246(a)gmail.com>
* Remove temporary variable
Signed-off-by: Kamil Tekiela <tekiela246(a)gmail.com>
* Remove redundant code
Remove redundant if statement.
Break at the end of the loop.
elseif when there is no other choice.
Redundant ! empty().
Signed-off-by: Kamil Tekiela <tekiela246(a)gmail.com>
* Collapse if statement into an elvis operator
Signed-off-by: Kamil Tekiela <tekiela246(a)gmail.com>
* Better type hinting for return array
We can tell Psalm what to expect, but we should declare the variables first.
Signed-off-by: Kamil Tekiela <tekiela246(a)gmail.com>
* Remove dead code
Signed-off-by: Kamil Tekiela <tekiela246(a)gmail.com>
* Use array_keys instead of unused variable
Signed-off-by: Kamil Tekiela <tekiela246(a)gmail.com>
* Add type declaration and psalm-return
Signed-off-by: Kamil Tekiela <tekiela246(a)gmail.com>
* Add psalm-return to checkChildForeignReferences()
Also added type declaration, and simplified the code
Signed-off-by: Kamil Tekiela <tekiela246(a)gmail.com>
* Refactor getChildReferences()
Signed-off-by: Kamil Tekiela <tekiela246(a)gmail.com>
* Improve type inference for foreignDropdown
Signed-off-by: Kamil Tekiela <tekiela246(a)gmail.com>
* [bugfix] Cast the value to a string to avoid type mismatch.
Signed-off-by: Kamil Tekiela <tekiela246(a)gmail.com>
* Add proper string type declaration to $foreign_filter
Signed-off-by: Kamil Tekiela <tekiela246(a)gmail.com>