Skip to content
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.

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

Allow the primitive types of integer, string and float to be stored in a Jsonb field #66

Merged
merged 2 commits into from
Apr 22, 2020

Conversation

martin-georgiev
Copy link
Owner

This PR addresses the jsonb limitation described in #58. Until now the only way to work-around it was to wrap the desired primitive value in an array, which honestly was a hack.
The changes are not perceived as backwards incompatible as full compatibility for older values is maintained.

@coveralls
Copy link

coveralls commented Apr 18, 2020

Pull Request Test Coverage Report for Build 294

  • 10 of 10 (100.0%) changed or added relevant lines in 2 files are covered.
  • No unchanged relevant lines lost coverage.
  • Overall coverage increased (+0.1%) to 94.795%

Totals Coverage Status
Change from base Build 289: 0.1%
Covered Lines: 346
Relevant Lines: 365

💛 - Coveralls

@@ -37,7 +37,7 @@ return PhpCsFixer\Config::create()
'php_unit_method_casing' => ['case' => 'snake_case'],
'php_unit_test_class_requires_covers' => false,
'phpdoc_types_order' => ['null_adjustment' => 'always_last'],
'simplified_null_return' => true,
Copy link
Owner Author

@martin-georgiev martin-georgiev Apr 18, 2020

Choose a reason for hiding this comment

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

This had to change because PHP-CS-Fixer/PHP-CS-Fixer#2349 suddenly started showing its teeth here.

🤷

@martin-georgiev martin-georgiev merged commit c7fc2b8 into master Apr 22, 2020
@martin-georgiev martin-georgiev deleted the expand-json-with-any-type branch April 22, 2020 09:31
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Bug: stored JSON value may be integer/float/string
2 participants