PHPStan fixes

Author: phpfui

src/PHPFUI/MySQLSlowQuery/BaseObject.php

@@ -41,7 +41,7 @@ public function __set(string $field, mixed $value)
 
 $this->fields[$field] = $value;
 
-return $value;
+
 }
 
 /** @return array<string, mixed> */

src/PHPFUI/MySQLSlowQuery/Entry.php

@@ -3,23 +3,44 @@
 namespace PHPFUI\MySQLSlowQuery;
 
 /**
- * @property ?string $acceptedWaiver MySQL type timestamp
- * @property string $Time
- * @property string $User
- * @property string $Host
- * @property int $Id
- * @property float $Query_time
+ * @property array<string> $Query
  * @property float $Lock_time
- * @property int $Rows_sent
+ * @property float $Query_time
+ * @property int $Bytes_sent
+ * @property int $Id
+ * @property int $Merge_passes
+ * @property int $Rows_affected
  * @property int $Rows_examined
- * @property array<string> $Query
+ * @property int $Rows_sent
  * @property int $Session
+ * @property int $Thread_id
+ * @property int $Tmp_disk_tables
+ * @property int $Tmp_table_sizes
+ * @property int $Tmp_tables
+ * @property string $explain
+ * @property string $Filesort
+ * @property string $Filesort_on_disk
+ * @property string $Full_join
+ * @property string $Full_scan
+ * @property string $Host
+ * @property string $Priority_queue
+ * @property string $QC_hit
+ * @property string $Schema
+ * @property string $Time
+ * @property string $Tmp_table
+ * @property string $Tmp_table_on_disk
+ * @property string $User
  */
 class Entry extends \PHPFUI\MySQLSlowQuery\BaseObject
 {
+/**
+ * @var array<string, string>
+ */
 private array $parameters = [];
 
-// @phpstan-ignore-next-line
+/**
+ * @param array<string, string> $parameters
+ */
 public function __construct(array $parameters = [])
 {
 $this->parameters = $parameters;

src/PHPFUI/MySQLSlowQuery/Parser.php

@@ -28,8 +28,8 @@ class Parser
  * Parse a MySQL Slow Query Log file
  */
 public function __construct(private string $fileName)
- {
- }
+{
+}
 
 /**
  * Return \PHPFUI\MySQLSlowQuery\Entry entries from file
@@ -218,9 +218,9 @@ private function parse() : void
 $previousTimeLine = $line;
 }
 }
-while ((\strlen($line = $this->getNextLine()) > 0) && ('#' === $line[0]));
+while (\strlen($line = $this->getNextLine()) > 0 && '#' === $line[0]); // @phpstan-ignore-line
 
-if ('mariadb' == $parseMode && ! $timeLineFound && $previousTimeLine)
+if ('mariadb' == $parseMode && ! $timeLineFound && $previousTimeLine) // @phpstan-ignore-line
 {
 // Always add the Time property. Assume that if it is not in the
 // log, it's the same as the previous logged query, and that the
@@ -231,8 +231,7 @@ private function parse() : void
 }
 
 // gather (more) query lines until a non-query line is reached
-// @phpstan-ignore-next-line
-while (\strlen($line = $this->getNextLine()) > 0 && '#' !== $line[0])
+while (\strlen($line = $this->getNextLine()) > 0 && '#' !== $line[0]) // @phpstan-ignore-line
 {
 if (0 === \stripos($line, self::PORT))// found a new session header
 {

src/PHPFUI/MySQLSlowQuery/Session.php

@@ -11,7 +11,7 @@
 class Session extends \PHPFUI\MySQLSlowQuery\BaseObject
 {
 /** @param array<int, string> $sessionData */
-public function __construct(array $sessionData = [], $parseMode = '')
+public function __construct(array $sessionData = [], string $parseMode = '')
 {
 $this->fields = [
 // Almost the full first line, i.e. executable and version.