2lenet / cruditbundle Goto Github PK

Les champs "TitleElements" dans le menu (AppMenu Provider) et les champs booléens dans les écrans Index n'apparaissent pas dans loco quand on veut les traduire.

After a fresh install of symfony and then composer require 2lenet/crudit-bundle

You have requested a non-existent parameter "default_locale". Did you mean this: "kernel.default_locale"? 

Example:
When you want to create a crudit screen on the PdfModel entity of the pdfGenerator bundle (https://github.com/2lenet/PdfGeneratorBundle), even when we make the command make:crudit Lle\PdfGeneratorBundle\Entity\PdfModel, Crudit looks for the entity in App\src\Entity.

Lorsqu'on fait un make:crudit, dans le src/Crudit/Config/EntityCrudConfig.php, ceci est créé :

private $datasource;

Le Symfony Profiler n'est pas content avec ça et râle parce qu'il veut un protected

• Choix de l'entité : C'est top

• Mode strict ou non : C'est pas la peine à mon avis on peux le forcer.

• générer ou non un contrôleur : On va forcer les controlleur. C'est plus explicite et plus clair.

• Namespace : par défaut, il faudrait que ce soit App\Controller\Crudit\ : Si je change, le répertoire n'est pas correctement généré
  

Au niveau du code :

class EspeceController extends AbstractCrudController
{
    /** @var ConfigProvider  */
    private $configProvider;

    public function __construct(ConfigProvider $configProvider)
    {
        $this->configProvider = $configProvider;
    }

    /**
     * @Route("/")
     */
    public function index(Request $request): Response
    {
        $views = $this->getBrickBuilder()->build($this->configProvider->get(EspeceCrudConfig::class), $request);
        return $this->render('@LleCrudit/crud/index.html.twig', ['views' => $views]);
    }

je pense que c'est mieux d'injecter directement le CrudConfig dans le controlleur. A quoi ça sert d'avoir un configProvider ? C'est un registre qui fait exactement la même chose que le Container. Donc pas utile.

pourquoi "DéfaultDatasource" ? getDatasource c'est suffisant. y'a pas de moyen d'en avoir plusieurs de toute manière. C'est le datasource du crud.

Dernier point : je pense qu'il faudrait rendre optionnel la possibilité de mettre le menu ou pas. Par défaut on le met.

Since this bundle is 6.4+, consider supporting AssetMapper, since Webpack is no longer recommended for new Symfony projects.

Plus, installation is greatly simplified!

Rather than supporting both, you could simply release 2.0 that is AssetMapper only.