search

sonata-project / SonataUserBundle

Symfony SonataUserBundle
https://docs.sonata-project.org/projects/SonataUserBundle
MIT License
342 stars 487 forks source link

SonataUserBundle validator.xml does not adapt to MongoDB use #1579

Closed landure closed 1 year ago

landure commented 2 years ago

Environment

Ubuntu 22.04 Jammy Jellyfish

Sonata packages

show

``` $ composer show --latest 'sonata-project/*' Direct dependencies required in composer.json: sonata-project/admin-bundle 4.20.0 4.20.0 The missing Symfony Admin Generator sonata-project/doctrine-mongodb-admin-bundle 4.6.0 4.6.0 Symfony Sonata / Integrate Doctrine MongoDB ODM into the SonataAdminBundle sonata-project/user-bundle 5.4.0 5.4.0 Symfony SonataUserBundle Transitive dependencies not required in composer.json: sonata-project/block-bundle 4.19.0 4.19.0 Symfony SonataBlockBundle sonata-project/cache 2.2.0 2.2.0 Cache library Package sonata-project/cache is abandoned, you should avoid using it. No replacement was suggested. sonata-project/doctrine-extensions 2.0.1 2.0.1 Doctrine2 behavioral extensions sonata-project/exporter 3.0.0 3.0.0 Lightweight Exporter library sonata-project/form-extensions 1.18.0 1.18.0 Symfony form extensions sonata-project/twig-extensions 2.0.0 2.0.0 Sonata twig extensions ```

Symfony packages

show

``` $ composer show --latest 'symfony/*' Direct dependencies required in composer.json: symfony/asset v6.1.5 v6.1.5 Manages URL generation and versioning of web assets such as CSS stylesheets, Jav... symfony/browser-kit v6.1.3 v6.1.3 Simulates the behavior of a web browser, allowing you to make requests, click on... symfony/console v6.1.6 v6.1.6 Eases the creation of beautiful and testable command line interfaces symfony/css-selector v6.1.3 v6.1.3 Converts CSS selectors to XPath expressions symfony/debug-bundle v6.1.3 v6.1.3 Provides a tight integration of the Symfony VarDumper component and the ServerLo... symfony/doctrine-messenger v6.1.4 v6.1.4 Symfony Doctrine Messenger Bridge symfony/dotenv v6.1.0 v6.1.0 Registers environment variables from a .env file symfony/expression-language v6.1.6 v6.1.6 Provides an engine that can compile and evaluate expressions symfony/flex v2.2.3 v2.2.3 Composer plugin for Symfony symfony/form v6.1.5 v6.1.5 Allows to easily create, process and reuse HTML forms symfony/framework-bundle v6.1.6 v6.1.6 Provides a tight integration between Symfony components and the Symfony full-sta... symfony/http-client v6.1.6 v6.1.6 Provides powerful methods to fetch HTTP resources synchronously or asynchronously symfony/intl v6.1.6 v6.1.6 Provides a PHP replacement layer for the C intl extension that includes addition... symfony/mailer v6.1.5 v6.1.5 Helps sending emails symfony/maker-bundle v1.47.0 v1.47.0 Symfony Maker helps you create empty commands, controllers, form classes, tests ... symfony/mime v6.1.6 v6.1.6 Allows manipulating MIME messages symfony/monolog-bundle v3.8.0 v3.8.0 Symfony MonologBundle symfony/notifier v6.1.0 v6.1.0 Sends notifications via one or more channels (email, SMS, ...) symfony/phpunit-bridge v6.1.6 v6.1.6 Provides utilities for PHPUnit, especially user deprecation notices management symfony/process v6.1.3 v6.1.3 Executes commands in sub-processes symfony/property-access v6.1.3 v6.1.3 Provides functions to read and write from/to an object or array using a simple s... symfony/property-info v6.1.6 v6.1.6 Extracts information about PHP class' properties using metadata of popular sources symfony/proxy-manager-bridge v6.1.0 v6.1.0 Provides integration for ProxyManager with various Symfony components symfony/runtime v6.1.3 v6.1.3 Enables decoupling PHP applications from global state symfony/security-bundle v6.1.3 v6.1.3 Provides a tight integration of the Security component into the Symfony full-sta... symfony/serializer v6.1.6 v6.1.6 Handles serializing and deserializing data structures, including object graphs, ... symfony/stopwatch v6.1.5 v6.1.5 Provides a way to profile code symfony/string v6.1.6 v6.1.6 Provides an object-oriented API to strings and deals with bytes, UTF-8 code poin... symfony/translation v6.1.6 v6.1.6 Provides tools to internationalize your application symfony/twig-bundle v6.1.1 v6.1.1 Provides a tight integration of Twig into the Symfony full-stack framework symfony/validator v6.1.6 v6.1.6 Provides tools to validate values symfony/web-link v6.1.0 v6.1.0 Manages links between resources symfony/web-profiler-bundle v6.1.6 v6.1.6 Provides a development tool that gives detailed information about the execution ... symfony/yaml v6.1.6 v6.1.6 Loads and dumps YAML files Transitive dependencies not required in composer.json: symfony/cache v6.1.5 v6.1.5 Provides extended PSR-6, PSR-16 (and tags) implementations symfony/cache-contracts v3.1.1 v3.1.1 Generic abstractions related to caching symfony/config v6.1.3 v6.1.3 Helps you find, load, combine, autofill and validate configuration values of any... symfony/dependency-injection v6.1.5 v6.1.5 Allows you to standardize and centralize the way objects are constructed in your... symfony/deprecation-contracts v3.1.1 v3.1.1 A generic function and convention to trigger deprecation notices symfony/doctrine-bridge v6.1.6 v6.1.6 Provides integration for Doctrine with various Symfony components symfony/dom-crawler v6.1.4 v6.1.4 Eases DOM navigation for HTML and XML documents symfony/error-handler v6.1.6 v6.1.6 Provides tools to manage errors and ease debugging PHP code symfony/event-dispatcher v6.1.0 v6.1.0 Provides tools that allow your application components to communicate with each o... symfony/event-dispatcher-contracts v3.1.1 v3.1.1 Generic abstractions related to dispatching event symfony/filesystem v6.1.5 v6.1.5 Provides basic utilities for the filesystem symfony/finder v6.1.3 v6.1.3 Finds files and directories via an intuitive fluent interface symfony/http-client-contracts v3.1.1 v3.1.1 Generic abstractions related to HTTP clients symfony/http-foundation v6.1.6 v6.1.6 Defines an object-oriented layer for the HTTP specification symfony/http-kernel v6.1.6 v6.1.6 Provides a structured process for converting a Request into a Response symfony/messenger v6.1.6 v6.1.6 Helps applications send and receive messages to/from other applications or via m... symfony/monolog-bridge v6.1.2 v6.1.2 Provides integration for Monolog with various Symfony components symfony/options-resolver v6.1.0 v6.1.0 Provides an improved replacement for the array_replace PHP function symfony/password-hasher v6.1.3 v6.1.3 Provides password hashing utilities symfony/polyfill-intl-grapheme v1.26.0 v1.26.0 Symfony polyfill for intl's grapheme_* functions symfony/polyfill-intl-icu v1.26.0 v1.26.0 Symfony polyfill for intl's ICU-related data and classes symfony/polyfill-intl-idn v1.26.0 v1.26.0 Symfony polyfill for intl's idn_to_ascii and idn_to_utf8 functions symfony/polyfill-intl-normalizer v1.26.0 v1.26.0 Symfony polyfill for intl's Normalizer class and related functions symfony/polyfill-mbstring v1.26.0 v1.26.0 Symfony polyfill for the Mbstring extension symfony/routing v6.1.5 v6.1.5 Maps an HTTP request to a set of configuration variables symfony/security-acl v3.3.2 v3.3.2 Symfony Security Component - ACL (Access Control List) symfony/security-core v6.1.6 v6.1.6 Symfony Security Component - Core Library symfony/security-csrf v6.1.0 v6.1.0 Symfony Security Component - CSRF Library symfony/security-http v6.1.6 v6.1.6 Symfony Security Component - HTTP Integration symfony/service-contracts v3.1.1 v3.1.1 Generic abstractions related to writing services symfony/translation-contracts v3.1.1 v3.1.1 Generic abstractions related to translation symfony/twig-bridge v6.1.6 v6.1.6 Provides integration for Twig with various Symfony components symfony/var-dumper v6.1.6 v6.1.6 Provides mechanisms for walking through any arbitrary PHP variable symfony/var-exporter v6.1.3 v6.1.3 Allows exporting any serializable PHP data structure to plain PHP code ```

PHP version

$ php -v
PHP 8.1.2-1ubuntu2.6 (cli) (built: Sep 15 2022 11:30:49) (NTS)
Copyright (c) The PHP Group
Zend Engine v4.1.2, Copyright (c) Zend Technologies
    with Zend OPcache v8.1.2-1ubuntu2.6, Copyright (c), by Zend Technologies
    with Xdebug v3.1.2, Copyright (c) 2002-2021, by Derick Rethans

Subject

When using MongoDB with multiple databases as a storage for Sonata User document (using a non default manager), it is impossible to add a new user using the web UI. The add post request fail with error:

Unable to find the object manager associated with an entity of class "App\Document\User".

This is due to the UniqueEntity validators in vendor/sonata-project/user-bundle/src/Resources/config/validation.xml. These validators uses service doctrine.orm.validator.unique. For MongoDB, it should be doctrine_odm.mongodb.unique.

Steps to reproduce

Configure SonataUserBundle with MongoDB, and try to add a new User using Sonata Admin Web UI.

Expected results

The new user is added successfully.

Actual results

The add post request fail with error:

Unable to find the object manager associated with an entity of class "App\Document\User".
landure commented 2 years ago

In addition to this bug report, i've asked for a solution at SonataUserBundle with multiple doctrine database: Validators uses default doctrine manager, not the one for the User class.

Hanmac commented 2 years ago

probably needs to be split ala this in fos user bundle:

https://github.com/FriendsOfSymfony/FOSUserBundle/tree/master/Resources/config/storage-validation

VincentLanglet commented 2 years ago

Can you provide a Pr ?

Hanmac commented 2 years ago

@VincentLanglet i can make PRs for this, i just need to think if we do extra test cases, currently we don't have extra MongoDB tests?

landure commented 2 years ago

Hi, I've made the needed changes in a fork. See this commit : Add storage aware validation for User..

The changes works when integrated in my code.

Unfortunately, I'm not able nor do I have the time to run (and fix) the unit tests for the changes. I hope my changes can help you fix this issue.

VincentLanglet commented 2 years ago

@VincentLanglet i can make PRs for this, i just need to think if we do extra test cases, currently we don't have extra MongoDB tests?

Yes we don't have mongoDB tests. But since the kernel is selected here: https://github.com/sonata-project/SonataUserBundle/blob/5.x/tests/custom_bootstrap.php#L20 And we cannot have multiple users class here https://github.com/sonata-project/SonataUserBundle/blob/5.x/tests/App/Resources/config/config.yaml#L66 I dunno if we can have easily both ORM and MongoDB tests.

@jordisala1991 Do you see an easy way to have two different kernel tests ?