Nette + Doctrine2 + Gedmo

Myslím, že je chyba jen v entitě, nechceš ji ukázat? A hodila by se i ona globální entita.

EDIT: První věc co mě napadla: Změnil jsi i anotaci @Entity na @ORM\Entity?

Editoval CherryBoss (15. 3. 2013 10:07)

mas definovanej use?
use Doctrine\ORM\Mapping as ORM

takhle nějak používám gedmo:

<?php
namespace Foo\Bar;

use Gedmo\Mapping\Annotation as Gedmo;
use Doctrine\ORM\Mapping as ORM;

/**
 * @Gedmo\Tree(type="nested")
 * @ORM\Table(name="directory_entry")
 * @ORM\Entity(repositoryClass="EntryRepository")
 */
class EntryEntity
{

	/**
	 * @ORM\Id
	 * @ORM\GeneratedValue
	 * @ORM\Column(type="integer")
	 * @var int
	 */
	private $id;

	/**
	 * @ORM\Column(name="name", type="string", length=64)
	 */
	private $name;

	/**
	 * @Gedmo\TreeLeft
	 * @ORM\Column(name="lft", type="integer")
	 */
	private $lft;

	/**
	 * @Gedmo\TreeLevel
	 * @ORM\Column(name="lvl", type="integer")
	 */
	private $lvl;

	/**
	 * @Gedmo\TreeRight
	 * @ORM\Column(name="rgt", type="integer")
	 */
	private $rgt;


	/**
	 * @Gedmo\TreeParent
	 * @ORM\ManyToOne(targetEntity="EntryEntity", inversedBy="children")
	 * @ORM\JoinColumn(name="parent_id", referencedColumnName="id", onDelete="SET NULL")
	 */
	private $parent;

	/**
	 * @ORM\OneToMany(targetEntity="EntryEntity", mappedBy="parent")
	 * @ORM\OrderBy({"lft" = "ASC"})
	 */
	private $children;


	/**
	 * @Gedmo\TreeRoot
	 * @ORM\Column(name="root_id")

	 */
	private $root;
....
}

koukni na https://github.com/…xtension.php a koukni jestli ti tam neco nechybi, pripadne si rovnou nainstaluj z https://github.com/nella doctrine a gedmo extensions a mas po problemech :)
(pokud jedes na dev verzi nette, tak mrkni na moje https://github.com/…/nette-gedmo a https://github.com/…tte-doctrine jsou to jen forky s kompatibilitou na dev verzi

Takže problém se mě podařilo na 99% odhalit, jen zatím nevím jak jej vyřešit. Problém je v tom že se používá simpleAnotationReader, když jej natvrdo vypnu tak to projde dále. Ten simple anotationreader pak špatně skládá názvy tříd.

jak tedy aktivovat tu extension co je v nellaframeworku?

class Extension extends \Nette\Config\CompilerExtension {}

a následně gedmo

class GedmoExtension extends \Nette\Config\CompilerExtension {}

edit:
předpokládám tedy nějak takto v bootstrapu:

$configurator->onCompile[] = function ($configurator, $compiler) {
    $compiler->addExtension('doctrine', new Nella\Doctrine\Config\Extension);
};

Editoval akadlec (15. 3. 2013 21:59)

Super, zatím mi tedy vše funguje jen jsem narazil na další problém ;) Snažím se použít extension Blameable. V anotaci je vše ok, při krokování jsem zjistil že tenhle listener funguje ok, ale problém je ten že v tom listeneru není přiřazen user který by se uložil do DB.

Potřeboval bych nějak zavolat něco takového:

$listener->setUserValue($user);

otaázkou tedy je jak se dostat k tomu konkrétnímu listeneru? Je definovaný v service doctrineEventManager pomocí addEventSubscriber

		doctrineEventManager:
			class: Doctrine\Common\EventManager
			setup:
				- addEventListener('loadClassMetadata', @tablePrefix)
				- addEventSubscriber(Doctrine\DBAL\Event\Listeners\MysqlSessionInit('utf8', 'utf8_general_ci'))
				- addEventSubscriber(Gedmo\Timestampable\TimestampableListener())
				- addEventSubscriber(Gedmo\Blameable\BlameableListener())
				- addEventSubscriber(Gedmo\Loggable\LoggableListener())
				- addEventSubscriber(Gedmo\Sluggable\SluggableListener())
				- addEventSubscriber(Gedmo\Tree\TreeListener())

No tu bych i rád použil bohužel se mi jí nedaří zprovoznit :( a sandbox nebo nějaké příklady jsem nenašel :(

No chvíli zkoušení a laborování a podařilo se mi nella extenze rozjet. Hodím to sem kdyby náhodou někdo stejně tápal jako já.

Extensions sem zaregistroval v bootstrapu

$configurator->onCompile[] = function ($configurator, $compiler) {
    $compiler->addExtension('doctrine', new Nella\Doctrine\Config\Extension);
    $compiler->addExtension('gedmo', new Nella\Doctrine\Config\GedmoExtension);
};
$container = $configurator->createContainer();

A v neonu jsem upravil konfigurace. Původně byly v sekci parameters, nyní je to pro každou extension zválšť

common:
	doctrine:
		entityDirs:	['%libsDir%/IPub/CMS/Models']
		proxy:
			dir			: %tempDir%/proxies
			namespace		: 'App\Model\Proxies'
			autogenerate	: true
		connection:
			prefix:		smart_
			driver:		pdo_mysql
			user:		root
			password:		root
			dbname:		nette_smartstart
			host:		localhost
			charset:		utf8
			collation:	utf8_general_ci

	gedmo:
		orm:
			em			: @doctrine.entityManager
			loggable		: true
			sluggable		: true
			softDeleteable	: true
			sortable		: true
			timestampable	: true
			translatable	: true
			tree			: true
			uploadable	: true

Snad jsem to realizoval správně. Gedmo extenze již funguji tak jak mají, ještě tedy musím dodělat tu Blameable která v nella není.

Editoval akadlec (16. 3. 2013 18:01)

Tenhle zacarovany kruh jsem take resil (psal jsem si sve vlastni Blameable rozsireni) ale potreboval jsem to vyresit asap takze nakonec jsem to udelal tak, ze jsem mel vlastni tridu User (uz jsem ji stejne mel z jinych duvodu), ktera mela metodu getUserEntity(EntityManager $em), takze v tom listeneru jsem ji toho entity managera predal (ten je v tech EventArgs $args)… rozhodne na to nejsem pysny, ale fungovalo to… to ale asi neni tvuj pripad, pac bys ten BlameableListener musel trochu prepsat…

Ale napada me, k cemu je vlastne CompilerExtension::afterCompile(Nette\Utils\PhpGenerator\ClassType $class), nedala by se zneuzit k donastaveni sluzby po jejich vytvoreni? fakt netusim…

Problem je v tom ze to „circular reference detected“ bys tam mel porad:

LISTENER needs USER needs FACADE needs ENTITY_MANAGER needs LISTENER :)

jestli jsem ti teda porozumel… presne tak jsem to chtel take udelat, ale pak mi nezbylo nic jineho, nez tomu USEROVI rict o ENTITY_MANAGER az kdyz se jsem se ho na tu entitu ptal

Ale ten user by tu fasadu mel injektovanou uz v dobe vytvoreni (standardni cestou), takze si stejne myslim ze by se to zakruhovalo… leda ze bys ji tam nejak nainjektoval az pozdeji

Kazdopadne az to vyresis tak se rad necham inspirovat :)

Kdybys použil kdyby/events tak se ti container zkompiluje tak, že ti tuhle circural reference přeruší (platí pouze na listenery) ;)

Jenom jsem nezkoušel, jestli je kdyby/events kompatibilní s Nellou (myslím že není) :)

Předpokládám, že článek si četl, takže rozvedu pouze to s tím fixováním circural reference.

Protože listenery mívají různé závislosti a né vždy se pustí všechny, tak je kdyby/events optimalizují takovým způsobem, že všechny je zanalyzuje, vytvoří mapu ve které je popsané jaký listener pouslouchá jakou událost a pak zruší jejich přímé provázání s EventManagerem (zahodí všechny metody setup které přidávají listenery). Tahle mapa společně s DI Containerem se předá do EventManageru a on si pak vytahuje listenery podle jména služby až když jsou volány.

Příklad jak to vypadá fyzicky, když se to zkompiluje, můžeš vidět v testech.

@Filip Procházka
Tak sem si na to zkusil sednout a pochopit. A pokud jsem pochopil správně tak:

1. V bootsrapu si zaregistruji EventsExtension

$configurator->onCompile[] = function ($configurator, $compiler) {
    $compiler->addExtension('events', new Kdyby\Events\DI\EventsExtension);
    .....
};

2. Vytvořím si svůj listener kde zadefinuji jednolivé eventy onNěco
3. Tento listener zaregistruji v configu, např. ten tvuj newRelic:

services:
	# LISTENERS
	newRelicListener:
		class: Addons\Events\Listeners\NewRelic\NewRelicProfilingListener
		tags: [kdyby.subscriber]

a mám z půle vyhráno? Abych to pak mohl aplikovat na to GEDMO tak musím vzít ty listenery gedma a upravit je aby je volala ta tva extension?

Editoval akadlec (11. 4. 2013 20:15)

Sry za možná hloupý dotaz, ale jenom takto?

services:
    # LISTENERS
    gedmoBlameableListener:
        class: Gedmo\Blameable\BlameableListener
        tags: [kdyby.subscriber]

Takže zkusil jsem to a nefachči :(

services:
    # LISTENERS
    timestampableListener:
	class: Gedmo\Timestampable\TimestampableListener
	tags: [kdyby.subscriber]

upravil sem i ten listener:

class TimestampableListener extends MappedEventSubscriber implements Kdyby\Events\Subscriber
{
	....
}

Je to tento listener: https://github.com/…Listener.php

Doplnění:
když to Gedmo rozšíření Timestampable vyhodím z inicializace dle Nella a udělám to pomocí Kdyby tak to přestane zavěšovat ty události, v tom listeneru je událost onFlush která se nespustí a když se dívam do zaregistrovaných listenerů tak tam není. Zkoušel jsem i upravit listener takto:

namespace Gedmo\Timestampable;

use Kdyby;

use Doctrine\Common\EventArgs,
    Gedmo\Mapping\MappedEventSubscriber,
    Doctrine\Common\NotifyPropertyChanged,
    Gedmo\Exception\UnexpectedValueException;

class TimestampableListener extends MappedEventSubscriber implements Kdyby\Events\Subscriber
{
    /**
     * Specifies the list of events to listen
     *
     * @return array
     */
    public function getSubscribedEvents()
    {
        return array(
            '\\Doctrine\\ORM\\Events::prePersist',
            '\\Doctrine\\ORM\\Events::onFlush',
            '\\Doctrine\\ORM\\Events::loadClassMetadata'
        );
    }

Editoval akadlec (12. 4. 2013 21:35)

@solution to upravoval, aby mu to jelo s Kdyby/Doctrine, koukni: https://gist.github.com/…tion/5389050

Takže pokud se dívám mělo by stači vyřešit to registrování listenerů co použil @Solution v classe GedmoExtension

@Solution
takže se ti povedlo rozchodit to blameable? Aby to přidávalo samo usera co upravuje/pracuje s entitou?

Tak vcera jsem zrovna integroval blameable via Nella\Doctrine a Kdyby\Events a zjistil jsem par postrehu, ktere by ti mohly pomoci. Do Nella\Doctrine muzes predat tvuj EventManager (https://github.com/…xtension.php#L46). Ten logicky musi byt pro celou aplikaci (nebo alepson souvisejici cast) pouze jeden, v pripade Kdyby\Events je to
@events.manager:

doctrine:
	eventManager: @events.manager

pak se ti zacnou ty eventy spoustet spravne…

Nicmene ale i po teto integraci jsem stale zasekly na Circular reference detected... Tentokrat ale tak, ze blameableListener je zavisly sam na sobe (?!). Po hlubsim zkoumani fce DI\Container::getService($name) jsem zjistil, ze je to asi tim, ze v pripade blameable se ten blameableListener vola dvakrat (createdBy a updatedBy) a snazi se nejak nakonfigurovat… pri odstraneni jedne z techto akci to jiz funguje…

Debugovanim jsem dosel k tomu, ze pri inicializaci listeneru v LazyEventManageru (https://github.com/…tManager.php#L94) se zavola v urcite zavislosti DI\Container::getService($name) vicekrat (v pripade blameable 2×), a pokazde se snazi inicializovat, na podruhe asi uz v zavislosti na tom prvnim… Uplne presne nevim, co se tam deje, uz byla pozdni nocni hodina, ja chtel jit spat a tak jsem provedl pouze to, ze jsem si v tom EventManageru udelal registr nainicializovanych sluzeb a spoustel to jen pro ty nove:

	/** @var array already inicialized listeners */
	private $inicialized = array();

	/**
	 * @param string $eventName
	 */
	private function initializeListener($eventName)
	{
		foreach ($this->listenerIds[$eventName] as $serviceName) {
			if(!array_key_exists($serviceName,$this->initialized))	{
				$this->initialized[$serviceName] = true;
				$subscriber = $this->container->getService($serviceName);
				/** @var Doctrine\Common\EventSubscriber $subscriber */

				$this->addEventSubscriber($subscriber);
			}

		}

		unset($this->listenerIds[$eventName]);
	}

Ted nedokazu rict, zda jsem delal nekde neco spatne pri definici/registraci listeneru, ale zdalo se mi to vse OK… tak nevim :( Nenapada vas nekoho neco?

Diky

@akadlec: presne tak… to by melo stacit a ty doctrine eventy se razem zacnou spoustet… ovsem jak rikam u me stale v pripade blameable vladla cirkularkova reference :)

jen jsem to tak zkousel, tzn nejdriv jsem zkousel vytvorit service: userEntity, ktera dostane Nette\Security\User a UserRepository a ma fci getEntity a vysledek teto fce jsem predal do blameableListeneru.

Z patra (nemam to ted u sebe, jde o princip)

services:
	userRepository: @doctrine::getRepository('User')
	userEntity:
		class: UserEntity
		setup:
			- setUser(@user)
			- setUserRepository(@userRepository)
	blameableListener:
		class: Gedmo\BlameableListener
		setup:
			- setUserValue(@userEntity::getEntity())
		tags: ['kdyby.subscriber']

pak jsem zkousel to predavat i ruzne jinak, ale porad jsem tam mel circular… Ovsem z ladenky bylo videt, ze se vytvareni teto sluzby inicializuje opravdu az on-demand (tzn vsechny zavislosti uz mely existovat)

Kompatibilita s Kdyby/Events bude opravena viz. report. Příště to rovnou nahlaš :)

Na integraci Kdyby/Doctrine a Gedmo budu průběžně pracovat tento měsíc, zatím mám chuť to přepsat nebo rovnou vzdát s Kdyby návrhem se to trochu bije (Tree) :).

Balík je Rixxi/Gedmo kód tradičně na github, kdyby někdo chtěl pomoci budu rád!

V Kdyby/Events je stále ten bug AFAIK takže je třeba pro tree úprava listeneru jinak to funguje ± samozřejmě stromy mají vlastní repozitáře nekompatibilní s Kdyby\Doctrine\EntityDao tak se na to musí dávat bacha aby je člověk netahal přímo přes Dao (háže vyjímku). Soft-deleteable se musí registrovat ručně a to je nejspíš všechno :)

Stále přemýšlím nad tím jak dořešit GEDMO rozšíření a napadlo mě, je nějak možné se dostat k těm zaregistrovaným listenerům? Třeba ve startup metodě? Aby se pak nad tím listnerem dala zavolat jeho metoda na přiřazení uživatele nebo definice locale?

mishak: no na tomhle projektu dělám po večerech a o chvílích volna. Jinak ale s mými znalostni nemám šanci to rozšíření napsat tak aby jej posvětil hosiplan :-)

Dočasně sem si to tedy vyřešil v startupu v base presenteru:

$this->context->getService('gedmo.gedmo.blameable')->setUserValue($this->userEntity);
$this->context->getService('gedmo.gedmo.translatable')->setTranslatableLocale($this->languageEntity->locale);

Editoval akadlec (3. 10. 2013 20:02)