Skip to content

Conversation

@MirakuSan
Copy link

Adds a new jsonb_document Doctrine type that leverages DBAL 4.3.0+ native JsonbType.

The type is conditionally registered only when DBAL 4.3.0+ is available, ensuring backward compatibility.

Shared serialization logic has been extracted into a trait to avoid duplication.

Closes #152


This is my first contribution to this project, hope I did it right! 🙂

New jsonb_document type leveraging native DBAL JsonbType.
Shared serialization logic extracted into JsonDocumentTypeTrait.
Type is conditionally registered when DBAL 4.3.0+ is available.
@MirakuSan MirakuSan force-pushed the feat/jsonb-document-type branch from 34ad85f to 3532d76 Compare December 2, 2025 14:50
@MirakuSan MirakuSan marked this pull request as draft December 2, 2025 14:50
Move ProductJsonb entity to separate directory with conditional mapping.
Enable native lazy objects for PHP 8.4+ compatibility.
@MirakuSan MirakuSan force-pushed the feat/jsonb-document-type branch 2 times, most recently from 0fe6963 to f327692 Compare December 2, 2025 15:30
@MirakuSan MirakuSan force-pushed the feat/jsonb-document-type branch from f327692 to d679dc7 Compare December 2, 2025 15:49
@MirakuSan MirakuSan marked this pull request as ready for review December 2, 2025 16:12
@MirakuSan
Copy link
Author

@dunglas I tried something here, not sure it is the right think to do.
The tests are passing in the CI, but it failes anyway.
Don't really now why...


namespace Dunglas\DoctrineJsonOdm\Tests;

use Doctrine\DBAL\Types\JsonbType;
Copy link

Choose a reason for hiding this comment

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

It seems this class is not used in this file.

Copy link
Author

Choose a reason for hiding this comment

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

Yes totaly, good catch

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.

Add new type jsonb_document

2 participants