contributte / doctrine-orm Goto Github PK

I can't get mapping to work using php 8 attributes.
Please add support or create documentation for this functionality.
(https://www.doctrine-project.org/projects/doctrine-orm/en/2.9/reference/attributes-reference.html)

ocramius/proxy-manager should be used instead. Look at phpstan.neon for more info (some deprecated constants)

Hi, we need to use Connection Wrapper (https://www.doctrine-project.org/projects/doctrine-dbal/en/2.7/reference/configuration.html#wrapper-class).

Is that possible with nettrine and what would be the best way? Previously in kdyby/doctrine there was a config option for that.

There are sometimes use-cases when we don't want to use mapping on Entity class in relations, but rather interface (possibly for decoupling reasons). That is why Doctrine supports ResolveTargetEntityListener (documentation on: https://www.doctrine-project.org/projects/doctrine-orm/en/2.6/cookbook/resolve-target-entity-listener.html).

It would be nice to have such configuration placed in config.neon. Symfony has its own implementation in config.yml (as described on https://symfony.com/doc/current/doctrine/resolve_target_entity.html at the bottom of the page)

We managed to do a workaround by placing the following code inside EntityManagerDecorator:

public function __construct(EntityManagerInterface $wrapped)
	{
		$resolveTargetEntityListener = new ResolveTargetEntityListener();
		$resolveTargetEntityListener->addResolveTargetEntity(UserInterface::class, User::class, []);

		$wrapped->getEventManager()->addEventListener(Events::loadClassMetadata, $resolveTargetEntityListener);
		parent::__construct($wrapped);
	}

It's good enough but people still have to remember that.

You are using current version of dev-master in our project?

On my project extension still throws this exception "AnnotationReader missing in DIC, please use Nettrine/Annotations or implement own MetadataProvider."

It's because, that getByType method returning still null, because anotation extension set anotation reader as autowired = false.
Look at the docs: https://api.nette.org/2.4/source-DI.ContainerBuilder.php.html#225

I think this is correct:

if (count($builder->findByType(AnnotationReader::class)) === 0) {
	throw new \Exception('AnnotationReader missing in DIC, please use Nettrine/Annotations or implement own MetadataProvider.');
}

Is there any specific reason, why have you omitted MetadataCommand in OrmConsoleExtension?

I'm using "orm:schema-tool:update" for generating diff, but I've to manually delete cache to have proper results.

Have I missed something in documentation? What is the preferred way to use "orm:schema-tool:update" correctly?

Simple extension to provide listeners/subscribers.

@enumag recommended Nettrine as a replacement for Kdyby/Doctrine however, I can't find any information on how to use it. Could you please provide at least some set-up instructions, or a link somewhere where this has been discussed?

Thanks

Hello, it was long time ago I was using nette, so I installed sandbox and nettrine according to https://github.com/nettrine/orm/blob/master/.docs/README.md#minimal-configuration, and it still throws error that class was not found (Class 'Nettrine\Dbal\DI\DbalExtension' not found). Could you please help me out?

My config.neon, composer.json and exception are in the zip archive.
error.zip

Hi,
I love and very often use methods from kdyby/doctrine:
Repository -> findBy, findOneBy, countBy, findPairs
QueryBuilder -> whereCriteria

I know, that you want keep light version of doctrine, so we can add this methods as trait and add replacement of built-in QueryBuilder as option to DI. I can send PR, when you fix #8

Hi,

as discussed on slack here's the minimal guide to use redis as a doctrine/nettrine cache:

First install contributte/redis package:
composer require contributte/redis

Then register it in neon:

extensions:
	redis: Contributte\Redis\DI\RedisExtension

redis:
	# Setup Tracy panel
	debug: %debugMode%

	connection:
		default:
			uri: tcp://127.0.0.1:6379

			# Options passed directly to Predis\Client
			# https://github.com/nrk/predis#client-configuration
			options: []

and lastly use it as a cache for doctrine:

nettrine.cache:
  driver: Doctrine\Common\Cache\PredisCache

Hi,

when I want to reset the entity manager via ManagerRegistry::resetManager, only the decorator gets recreated, the inner EntityManager stays the same.

I guess the solution would be to remove the orm.entityManager service altogether and create it via DI Statement:

In OrmExtension::loadEntityManagerConfiguration()

		// Entity Manager
		$original = new Statement(DoctrineEntityManager::class . '::create', [
			$builder->getDefinitionByType(Connection::class), // Nettrine/DBAL
			$this->prefix('@configuration'),
		]);

		// Entity Manager Decorator
		$builder->addDefinition($this->prefix('entityManagerDecorator'))
			->setFactory($entityManagerDecoratorClass, [$original]);

In generated DI container:

	public function createServiceOrm__entityManagerDecorator(): Nettrine\ORM\EntityManagerDecorator
	{
		return new Nettrine\ORM\EntityManagerDecorator(Doctrine\ORM\EntityManager::create(
			$this->getService('dbal.connection'),
			$this->getService('orm.configuration'),
			$this->getService('dbal.eventManager')
		));
	}

If this approach is OK (or if you suggest another approach), I would gladly prepare a PR. (I know this conflicts with open PR #47, I can wait until it is merged.)

There must be better dependency on doctrine/orm.

At this moment it triggers errors on Travis CI.

"doctrine/orm": "^2.5",

It must be higher:

"doctrine/orm": "~2.6.0",

We need to consider that EntityManager is final in doctrine/orm.

In docs, there is a statement:

But there is no mention about that orm:generate-proxies has to be run on production otherwise the application will fail.

Mine fail when fixtures were applying.

Debug panel with queries. Separate package?

Hello,

I know that using EntityManagerDecorator is recommended by Doctrine authors, but according to this issue doctrine/orm#5820 and my own experience, it's not ready to be heavily used yet.

Does someone use it successfully on large projects?

Thank you.

Versions:
nettrine/cache v0.1.0
nettrine/dbal v0.6.3
nettrine/orm v0.6.0

Bug Description

Entities from my custom entity are not loaded. It thows Doctrine\Persistence\Mapping\MappingException The class 'Example\Model\Entities\Foo' was not found in the chain configured namespaces App.

I think bug is in https://github.com/nettrine/orm/blob/v0.6.0/src/DI/Traits/TEntityMapping.php

Steps To Reproduce

Register in own DI extension:

<?php

namespace Example\Foo\DI;

use Nette\DI\CompilerExtension;
use Nettrine\ORM\DI\Traits\TEntityMapping;

class FooExtension extends CompilerExtension
{

	use TEntityMapping;

	public function loadConfiguration()
	{
		$builder = $this->getContainerBuilder();
		$config = $this->loadFromFile(__DIR__ . '/config.neon');

		$this->compiler->loadDefinitionsFromConfig($config['services']);

		$this->setEntityMappings([
			'Example\Model\Entities\Foo' => __DIR__ . '/../Model/Entities',
		]);
	}

	public function beforeCompile()
	{
		$builder = $this->getContainerBuilder();
	}
}

Possible Solution

N/A

In https://github.com/nettrine/orm/blob/master/.docs/README.md#setup

extensions:
    orm.annotations: Nettrine\ORM\DI\OrmAnnotationsExtension

orm.annotations:
    paths:
        - App/Model/Database/Entity

Instead of App/Model/Database/Entity use %appDir%/Model/Entity ?

A follow up on this nette forum discussion: https://forum.nette.org/cs/31618-nettrine-extensions-neaplikuje-se-filtr

The post mentioned it is not yet implemented and I was unable to find it in master branch code.

If there is a demand, I would be willing to make a PR - I have too much free time on my hands anyways with the current situation.

There was a BC-break introduced in doctrine/orm 2.8.2 - AbstractCommand requires EntityManager: doctrine/orm#8488.

When running db-related commands, following error is thrown:

ErrorException: assert(): assert($em instanceof EntityManager) failed in vendor/doctrine/orm/lib/Doctrine/ORM/Tools/Console/Command/SchemaTool/AbstractCommand.php:56

It is already fixed (doctrine/orm#8493), so I assume 2.8.3 will be ok.

Maybe it would be worth it to add conflict with 2.8.2 into composer.json?

Hi, planning a XML mapping support for this package?

I there way to run nettrine/orm with Nette 3?
When i try require with composer nettrine/orm a get conflict:
Problem 1
- Can only install one of: nette/di[v2.4.x-dev, 3.0.x-dev].
- Can only install one of: nette/di[v3.0.0-alpha1, v2.4.x-dev].
- Can only install one of: nette/di[v3.0.0-beta1, v2.4.x-dev].
- Can only install one of: nette/di[v3.0.0-beta2, v2.4.x-dev].
- nettrine/orm 0.3.x-dev requires nette/di ~2.4.14 -> satisfiable by nette/di[v2.4.x-dev].
- Installation request for nettrine/orm ^0.3.0@dev -> satisfiable by nettrine/orm[0.3.x-dev].
- Installation request for nette/di ^3.0 -> satisfiable by nette/di[3.0.x-dev, v3.0.0-alpha1, v3.0.0-beta1, v3.0.0-beta2].

Cannot set own cache configuration factory as service like this:

nettrine.orm.cache:
secondLevelCache: @redisCacheConfigurationFactory::create(%tempDir%)

services:
redisCacheConfigurationFactory: Eshop\Core\Cache\RedisCacheConfigurationFactory(%redis.host%, %redis.port%)

I have to create next service so working result is

services: redisCacheConfigurationFactory: Eshop\Core\Cache\RedisCacheConfigurationFactory(%redis.host%, %redis.port%) redisCacheConfiguration: @redisCacheConfigurationFactory::create(%tempDir%)

nettrine.orm.cache: secondLevelCache: @redisCacheConfiguration

For those of us who prefer the sense of stability coming from tagged releases as opposed to dev-master, would you care to tag a new one? I've stumbled upon a bug in 0.1 that breaks the default (!) configuration (wrong is_subclass_of in OrmCacheExtension) and that is hopefully fixed in master by 62526fb. Thank you :)

Hello,
is it possible to use this integration with Codeception module Doctrine2? https://codeception.com/docs/modules/Doctrine2

How to eventually configure it (getEntityManager)?

Thanx.

Laster version of doctrine orm (2.7.3) is not compatible due to namespaces changes Doctrine\Common\Persistence -> Doctrine\Persistence which is cause by dependency: "doctrine/common": "^2.11 || ^3.0",

To fix this probem, had to define doctrine orm 2.7.2 in my project.

So, just to inform you

I would like to change internal mapping driver to DriverChain. It would be more suitable for separate extensions, each could register own mapper, or for modular system.

Snippet from doc: https://www.doctrine-project.org/projects/doctrine-orm/en/2.6/reference/advanced-configuration.html#multiple-metadata-sources

$chain = new DriverChain(); 
$chain->addDriver($xmlDriver, 'Doctrine\Tests\Models\Company'); 
$chain->addDriver($yamlDriver, 'Doctrine\Tests\ORM\Mapping');

I fought with exception Missing Reader service from \Nettrine\ORM\DI\OrmAnnotationsExtension::loadConfiguration() - I discovered that the order in which the extensions are registered matters and that it makes sense to register Nettrine\Annotations\DI\AnnotationsExtension before Nettrine\ORM extensions. Maybe the exception message could advise that?

Hi,

I would like use orm.cache with memcached with other port and host. I didnt find any method. Can you help me?

I try define service
orm.cache: queryCache: @a2

and
services: a2: class:Doctrine\Common\Cache\MemcachedCache
report me

Service 'orm.cache.queryCache': Reference to missing service 'a2'.

Hi, the Nettrine\ORM\DI\OrmExtension class defines in the config schema that the configuration.repositoryFactory property must be a string, but when I use one, the generated service factory passes the string as a literal to the $service->setRepositoryFactory() call, but the method expects an instance of the repository factory. If I set the repository factory to a Neon entity instead (basically just append () to the class name) schema validation fails, because I'm no longer passing a string where one is expected.

Workaround: set the repository factory manually in services:

services:
  # ...
  nettrine.orm.configuration:
    alteration: true
    setup:
      - setRepositoryFactory(My\Repository\Factory())

I'll look into writing a PR to fix this, but it won't be today..

Hi,

Is possible to create multiple connections? I try configuration like this https://help.kdyby.org/question/?questionId=93, but it is not working.

Thank you for your help.

I've following code:

cache: @App\Factories\CacheFactory::createDoctrineCache('Doctrine.Annotations')

Exception given: The option 'orm.annotations › cache' expects to be string or null, object Nette\DI\Definitions\Statement given

Hi, nettrine/orm is not compatible with PHP 8, because:
it has dependency doctrine/common: ^2.13.1,
which is compatible with doctrine/orm version 2.7.5, because it has dependencies

php: ^7.1
doctrine/common: ^2.11 || ^3.0

but doctrine/orm version 2.8.0, has dependencies

php: ^7.2|^8.0
doctrine/common: ^3.0

and because of this clash, composer enforces usage of doctrine/orm < 2.8.0, which is not compatible with PHP 8.
I hope having
doctrine/common: ^2.13.1 || ^3.0
instead of
doctrine/common: ^2.13.1
would solve it.

Problem 1
- symfony/console v4.1.1 requires php ^7.1.3 -> your PHP version (7.1.17-0ubuntu0.17.10.1) overridden by "config.platform.php" version (7.1) does not satisfy that requirement.
- symfony/console v4.1.0 requires php ^7.1.3 -> your PHP version (7.1.17-0ubuntu0.17.10.1) overridden by "config.platform.php" version (7.1) does not satisfy that requirement.
- symfony/console v4.0.9 requires php ^7.1.3 -> your PHP version (7.1.17-0ubuntu0.17.10.1) overridden by "config.platform.php" version (7.1) does not satisfy that requirement.
- symfony/console v4.0.8 requires php ^7.1.3 -> your PHP version (7.1.17-0ubuntu0.17.10.1) overridden by "config.platform.php" version (7.1) does not satisfy that requirement.
- symfony/console v4.0.12 requires php ^7.1.3 -> your PHP version (7.1.17-0ubuntu0.17.10.1) overridden by "config.platform.php" version (7.1) does not satisfy that requirement.
- symfony/console v4.0.11 requires php ^7.1.3 -> your PHP version (7.1.17-0ubuntu0.17.10.1) overridden by "config.platform.php" version (7.1) does not satisfy that requirement.
- symfony/console v4.0.10 requires php ^7.1.3 -> your PHP version (7.1.17-0ubuntu0.17.10.1) overridden by "config.platform.php" version (7.1) does not satisfy that requirement.
- nettrine/orm v0.2.1 requires symfony/console ^4.0.8 -> satisfiable by symfony/console[v4.0.10, v4.0.11, v4.0.12, v4.0.8, v4.0.9, v4.1.0, v4.1.1].
- Installation request for nettrine/orm ^0.2.1 -> satisfiable by nettrine/orm[v0.2.1]

Hi,
is there a way to register custom DQL function like in Kdyby/Doctrine ?
https://github.com/kdyby/doctrine/blob/master/docs/en/configuring.md#user-content-orm