* Refactored tryColumnCreationQuery function inside CreateAddField to use DatabaseName type instead of string
* Refactored exportDatabase and lockTables function to use DatabaseName type instead of string for database name
Signed-off-by: Umang Patel <umang.patel@healthengine.com.au>
Co-authored-by: Maurício Meneghini Fauth <mauricio@fauth.dev>
No useless globals, no overwriting of the same value. The values seem to always be string|null so let's check explicitly for that.
Signed-off-by: Kamil Tekiela <tekiela246@gmail.com>
The crlf global variable is only diferent for SQL export method,
otherwise is always the value of the PHP_EOL constant.
Signed-off-by: Maurício Meneghini Fauth <mauricio@fauth.dev>
* Update psalm-baseline.xml
Signed-off-by: Kamil Tekiela <tekiela246@gmail.com>
* Small bug fixes
Signed-off-by: Kamil Tekiela <tekiela246@gmail.com>
* non-falsy-string is always true
Signed-off-by: Kamil Tekiela <tekiela246@gmail.com>
* currentSearch property is nullable
Signed-off-by: Kamil Tekiela <tekiela246@gmail.com>
* Remove unusued parameter $columnIndex
Signed-off-by: Kamil Tekiela <tekiela246@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@gmail.com>
* Remove unusued parameter $crlf
Signed-off-by: Kamil Tekiela <tekiela246@gmail.com>
