Replace sql type TEXT by LONGTEXT for profile field when using mysql …

[Mmasson-01]

Problem

mysql driver truncate data when using field type TEXT while exceeding 65535 characters

Solution

Use field type LONGTEXT when using mysql driver.

Related PR

• Replace sql type text by longtext for profile field in PdoRepository #437
• Update PdoRepository::initSchema() to create table with profile as LONGTEXT. #474
• Update profile filed type from TEXT to LONGTEXT #486

Alternative

I believe it could add a nice touch to implement an abstract class as a Driver helper.
Something like this:

<?php

namespace XHGui\Db;

abstract class PdoDriverAbstract 
{
    /* @var PdoDriverAbstract */
    private $driver;

    public function __construct(string $driverName) 
    {
        switch($driverName)
        {
            case "mysql":
                $this->driver = new PdoMysqlDriver();
                break;
            default:
                $this->driver = new PdoSqliteDriver();
        }

    }
    
    abstract public function getTableQuery(): string;
}

Then we could do something like this:

 public function __construct(PDO $pdo, string $driverName, string $table, string $tableWatch)
    {
        $this->pdo = $pdo;
        $this->driver = new PdoDriverAbstract($driverName);
        $this->table = sprintf('"%s"', $table);
        $this->tableWatches = sprintf('"%s"', $tableWatch);
        $this->initSchema();
    }

 public function initSchema(): void
    {
            $this->pdo->exec($this->driver->getTableQuery()
                ', $this->table));
   ...
}

It would be ready to scale if other exception happens in the futur...but then its a bit overkill for now.

Let me know !

[glensc]

Maybe less copy paste, and create $profileColumnType = "LONGTEXT" and use that in expression?

[Mmasson-01]

@glensc Agreed. Let me fix this real quick
Meanwhile, what about my alternative suggestion ?