Model relation use class rather than id #70
Annotations
10 errors and 1 warning
phpstan:
lib/ActiveRecord.php#L188
Access to an undefined property ICanBoogie\ActiveRecord\Schema\Column::$auto_increment.
|
phpstan:
lib/ActiveRecord.php#L211
Parameter #1 $primary_key of method ICanBoogie\ActiveRecord<TKey of array<string>|int|string>::update_primary_key() expects array<string>|int|string, mixed given.
|
phpstan:
lib/ActiveRecord/Config/AssociationBuilder.php#L95
Parameter $through of method ICanBoogie\ActiveRecord\Config\AssociationBuilder::has_many() expects class-string<ICanBoogie\ActiveRecord>|null, class-string|null given.
|
phpstan:
lib/ActiveRecord/Connection.php#L71
Property ICanBoogie\ActiveRecord\Connection::$driver is never written, only read.
|
phpstan:
lib/ActiveRecord/Connection.php#L89
Property ICanBoogie\ActiveRecord\Connection::$profiling type has no value type specified in iterable type array.
|
phpstan:
lib/ActiveRecord/Connection.php#L130
Parameter #1 $statement of method ICanBoogie\ActiveRecord\Connection::query() expects string, mixed given.
|
phpstan:
lib/ActiveRecord/Connection.php#L220
Parameter #1 $mode of method PDOStatement::setFetchMode() expects int, mixed given.
|
phpstan:
lib/ActiveRecord/ConnectionCollection.php#L52
Method ICanBoogie\ActiveRecord\ConnectionCollection::get_definitions() return type has no value type specified in iterable type array.
|
phpstan:
lib/ActiveRecord/ConnectionCollection.php#L54
Method ICanBoogie\ActiveRecord\ConnectionCollection::get_definitions() should return array<string, array> but returns array<string, ICanBoogie\ActiveRecord\Config\ConnectionDefinition>.
|
phpstan:
lib/ActiveRecord/Driver/TableRenderer.php#L39
Method ICanBoogie\ActiveRecord\Driver\TableRenderer::render_table_options() invoked with 1 parameter, 0 required.
|
phpstan
The following actions uses node12 which is deprecated and will be forced to run on node16: actions/checkout@v2, actions/cache@v2. For more info: https://github.blog/changelog/2023-06-13-github-actions-all-actions-will-run-on-node16-instead-of-node12-by-default/
|