Skip to content

[Serializer] allow more default context configuration via yaml #33420

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

Jump to bottom
Closed
rimas-kudelis opened this issue Sep 2, 2019 · 1 comment
Closed

[Serializer] allow more default context configuration via yaml #33420

rimas-kudelis opened this issue Sep 2, 2019 · 1 comment
Labels
Feature FrameworkBundle Serializer

Comments

@rimas-kudelis
Copy link

Description
Serializer context may have numerous keys, but only a handful are configurable via yaml:

    # serializer configuration
    serializer:
        enabled:              true
        enable_annotations:   true
        name_converter:       ~
        circular_reference_handler: ~
        max_depth_handler:    ~
        mapping:
            paths:                []

It would be nice if other options were supported. In particular, I find it quite inconvenient that while I can configure max_depth_handler via yaml config, I cannot enable_max_depth that same way. Without the latter, the former appears to me hardly useful.

Example

    # serializer configuration
    serializer:
        enabled:              true
        enable_annotations:   true
        enable_max_depth:     true
        (...other options?..)
        name_converter:       ~
        circular_reference_handler: ~
        max_depth_handler:    ~
        mapping:
            paths:                []
@rimas-kudelis
Copy link
Author

Apparently this is already being fixed: #28927

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
Feature FrameworkBundle Serializer
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
2 participants
@xabbuh @rimas-kudelis