Skip to content

Adding new column types to the ExpectedValues #70

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
roxblnfk merged 3 commits into from
Feb 15, 2023
Merged

Adding new column types to the ExpectedValues #70

Changes from all commits
Commits
Show all changes
3 commits
Select commit Hold shift + click to select a range
75ac908
Add column types to the ExpectedValues
msmakouz Feb 15, 2023
f96f7da
Fix CS
msmakouz Feb 15, 2023
2129c58
Add comment
msmakouz Feb 15, 2023
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
18 changes: 15 additions & 3 deletions src/Annotation/Column.php
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -24,6 +24,12 @@ final class Column

/**
* @param non-empty-string $type Column type. {@see \Cycle\Database\Schema\AbstractColumn::$mapping}
* Column types `smallPrimary`, `timetz`, `timestamptz`, `interval`, `bitVarying`, `int4range`, `int8range`,
* `numrange`, `tsrange`, `tstzrange`, `daterange`, `jsonb`, `point`, `line`, `lseg`, `box`, `path`,
* `polygon`, `circle`, `cidr`, `inet`, `macaddr`, `macaddr8`, `tsvector`, `tsquery` are related
* to the PostgreSQL only {@see \Cycle\Database\Driver\Postgres\Schema\PostgresColumn::$mapping}
* Column type `datetime2` is related to the SQL Server only
* {@see \Cycle\Database\Driver\SQLServer\Schema\SQLServerColumn::$mapping}
* @param non-empty-string|null $name Column name. Defaults to the property name.
* @param non-empty-string|null $property Property that belongs to column. For virtual columns.
* @param bool $primary Explicitly set column as a primary key.
Expand All @@ -38,9 +44,15 @@ final class Column
*/
public function __construct(
#[ExpectedValues(values: ['primary', 'bigPrimary', 'enum', 'boolean',
'integer', 'tinyInteger', 'smallInteger', 'bigInteger',
'string', 'text', 'tinyText', 'longText', 'double', 'float', 'decimal', 'datetime', 'date', 'time',
'timestamp', 'binary', 'tinyBinary', 'longBinary', 'json', 'uuid',
'integer', 'tinyInteger', 'smallInteger', 'bigInteger', 'string', 'text', 'tinyText', 'longText', 'double',
'float', 'decimal', 'datetime', 'date', 'time', 'timestamp', 'binary', 'tinyBinary', 'longBinary', 'json',
'uuid', 'bit',
// PostgreSQL
'smallPrimary', 'timetz', 'timestamptz', 'interval', 'bitVarying', 'int4range', 'int8range', 'numrange',
'tsrange', 'tstzrange', 'daterange', 'jsonb', 'point', 'line', 'lseg', 'box', 'path', 'polygon', 'circle',
'cidr', 'inet', 'macaddr', 'macaddr8', 'tsvector', 'tsquery',
// SQL Server
'datetime2',
])]
private string $type,
private ?string $name = null,
Expand Down