piotr-cz / cockpit-sql-driver Goto Github PK

https://github.com/piotr-cz/cockpit-sql-driver/blob/master/LICENSE#L3

https://piocz.cz should be https://piotr.cz

Which has been added in Cockpit v0.9.3 here.

At this moment it's used only for moving collection entries to trash.

Hey, there is a "," missing in your config.php example. After "localhost".

Tables are created on the fly and this can mess things up.

Simple test:

$this->on('cockpit.bootstrap', function() {
    $test = $this->storage->getKey('test', 'test', []);
});

Now my database has a new table named "test".

Another test with an existing table:

$this->on('cockpit.bootstrap', function() {
    $test = $this->storage->getKey('wp_users', 'test', []);
});

result:

PDOException while running query
SELECT
"document"
FROM
`wp_users`
WHERE JSON_UNQUOTE(JSON_EXTRACT(`document`, '$.\"key\"')) = 'test'
LIMIT 1

The current behaviour makes it impossible to use the same database with a different application or with custom tables in it.

If I could prefix all cockpit tables with e. g. cp_, I could still create a ton of useles tables on the fly with a typo in my storage request, but it wouldn't hurt much.

I'm not sure, if it was intentional, but the default bootstrap priority should be very high (e. g. 9999) to change the driver with the highest priority.

A simple test in addons/test/bootstrap.php:

// default priority 0
$this->on('cockpit.bootstrap', function() {
    $test = $this->storage->getKey('cockpit/options', 'multiplane', []);
    var_dump($test); // array(0) { } 
});

// priority 10
$this->on('cockpit.bootstrap', function() {
    $test = $this->storage->getKey('cockpit/options', 'multiplane', []);
    var_dump($test); // Call to a member function findOne() on null
}, 10);

If I set 'bootstrapPriority' => 9999 in the database options, the event with prio 10 works.

Hello i want to use cockpit with mysql database ,

How to install it ,
i have tryed it already buy its shown error ,
show please make a video on this in core cokpit .