Skip to content

Conversation

@benno-truevault
Copy link
Collaborator

Take an example array of ['a', 'b']

In a SET column it will stored as 'a,b'

In any other column type, it will be stored as a JSON string: '["a","b"]'

Take an example array of ['a', 'b']

In a SET column it will stored as 'a,b'

In any other column type, it will be stored as a JSON string:
'["a","b"]'
Copy link
Collaborator

@gwillz gwillz left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Very tight.

A few missing @param tags but they're on protected methods so I'm not too fussed.

* @return void
* @throws JsonException
*/
public function update($config)
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Would be good to chuck a @inheritdoc in here.

Actually we could start using the new #[\Override] attribute, which might communicate things even better.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants