Git Product home page Git Product logo

Comments (3)

nreynis avatar nreynis commented on September 23, 2024

I think this is linked to: #18

Doctrine already offer the possibility to override/customize table names with custom naming strategies

from notification-bundle.

nreynis avatar nreynis commented on September 23, 2024

I use this as a temporary patch to restore Doctrine default behaviour

<?php

namespace AppBundle\ORM\Mapping;

use Doctrine\Common\EventSubscriber;
use Doctrine\Common\EventArgs;
use Doctrine\ORM\Mapping\NamingStrategy;
use Mgilet\NotificationBundle\Entity\NotifiableEntity;
use Mgilet\NotificationBundle\Entity\NotifiableNotification;
use Mgilet\NotificationBundle\Entity\Notification;

class NotificationPatch implements EventSubscriber
{
    /**
     * @var NamingStrategy
     */
    protected $namingStrategy;

    public function __construct(NamingStrategy $strategy)
    {
        $this->namingStrategy = $strategy;
    }

    public function getSubscribedEvents(): array
    {
        return ['loadClassMetadata'];
    }

    /**
     * Restore naming strategy in mgilet/notification-bundle entities
     */
    public function loadClassMetadata(EventArgs $eventArgs): void
    {
        $metadata = $eventArgs->getClassMetadata();

        if($metadata->name === NotifiableEntity::class
                || $metadata->name === NotifiableNotification::class
                || $metadata->name === Notification::class){
            $metadata->table['name'] = $this->namingStrategy->classToTableName($metadata->name);
        }
    }
}

And in service.yml

services:
    AppBundle\ORM\Mapping\NotificationPatch:
        class: AppBundle\ORM\Mapping\NotificationPatch
        # pass whatever strategy you were using in the first place
        arguments: ['@doctrine.orm.naming_strategy.underscore']
        tags:
            - { name: doctrine.event_subscriber, connection: default }

from notification-bundle.

maximilienGilet avatar maximilienGilet commented on September 23, 2024

Yes, this is definitely related to #18 . I need some time to implement a fix for this issue. Maybe you could make a pull request ?

from notification-bundle.

Related Issues (20)

Recommend Projects

  • React photo React

    A declarative, efficient, and flexible JavaScript library for building user interfaces.

  • Vue.js photo Vue.js

    🖖 Vue.js is a progressive, incrementally-adoptable JavaScript framework for building UI on the web.

  • Typescript photo Typescript

    TypeScript is a superset of JavaScript that compiles to clean JavaScript output.

  • TensorFlow photo TensorFlow

    An Open Source Machine Learning Framework for Everyone

  • Django photo Django

    The Web framework for perfectionists with deadlines.

  • D3 photo D3

    Bring data to life with SVG, Canvas and HTML. 📊📈🎉

Recommend Topics

  • javascript

    JavaScript (JS) is a lightweight interpreted programming language with first-class functions.

  • web

    Some thing interesting about web. New door for the world.

  • server

    A server is a program made to process requests and deliver data to clients.

  • Machine learning

    Machine learning is a way of modeling and interpreting data that allows a piece of software to respond intelligently.

  • Game

    Some thing interesting about game, make everyone happy.

Recommend Org

  • Facebook photo Facebook

    We are working to build community through open source technology. NB: members must have two-factor auth.

  • Microsoft photo Microsoft

    Open source projects and samples from Microsoft.

  • Google photo Google

    Google ❤️ Open Source for everyone.

  • D3 photo D3

    Data-Driven Documents codes.