Declaration of getID3_cached_sqlite3

[webkader]

hi, how do i fix this bug?

Declaration of getID3_cached_sqlite3::analyze() should be compatible with getID3::analyze($filename, $filesize = NULL, $original_filename = '') in extension.cache.sqlite3.php on line 265

i think here is the cause

public function analyze($filename) {
...
$analysis = parent::analyze($filename);
...
}

[JamesHeinrich]

Fixed in d5c2918

Mirroring the similar fix (1e1f1a0) in the MySQL cache extension, I have modified the parent::analyze call

[webkader]

thanks, next issue

SQLite3Stmt::execute(): Unable to execute statement: NOT NULL constraint failed: media_cache.filesize in extension.cache.sqlite3.php on line 195

[JamesHeinrich]

I am not at all familiar with SQLite, perhaps someone who is can comment and recommend a fix.

[webkader]

Fixed

Solution this line

return "CREATE TABLE IF NOT EXISTS $this->table (filename VARCHAR(255) NOT NULL DEFAULT '', dirname VARCHAR(255) NOT NULL DEFAULT '', filesize INT(11) NOT NULL DEFAULT '0', filetime INT(11) NOT NULL DEFAULT '0', analyzetime INT(11) NOT NULL DEFAULT '0', val text not null, PRIMARY KEY (filename, filesize, filetime))";

to

return "CREATE TABLE IF NOT EXISTS $this->table (filename VARCHAR(255) DEFAULT '', dirname VARCHAR(255) DEFAULT '', filesize INT(11) DEFAULT '0', filetime INT(11) DEFAULT '0', analyzetime INT(11) DEFAULT '0', val text, PRIMARY KEY (filename, filesize, filetime))";

[webkader]

sqlite allow the use of null entries in a primary key column. least one column of the primary key to be not null

[JamesHeinrich]

Taking your word for it, updated in de973d5