[DDC-1909] Getting Fatal error Call to undefined method Doctrine\ORM\Mapping\ClassMetadata::getSqlExecutor() Created: 05/Jul/12  Updated: 29/Jul/12  Resolved: 29/Jul/12

Status: Resolved
Project: Doctrine 2 - ORM
Component/s: ORM
Affects Version/s: 2.2.2
Fix Version/s: 2.2.3
Security Level: All

Type: Bug Priority: Major
Reporter: Justinas Assignee: Benjamin Eberlei
Resolution: Fixed Votes: 0
Labels: None
Environment:

PHP 5.3.6-13ubuntu3.8 with Suhosin-Patch (cli) (built: Jun 13 2012 17:19:54)
Copyright (c) 1997-2011 The PHP Group
Zend Engine v2.3.0, Copyright (c) 1998-2011 Zend Technologies
with Xdebug v2.2.0, Copyright (c) 2002-2012, by Derick Rethans


Attachments: Zip Archive Bisna.zip     File DoctrineCache.php     File options.ini    

 Description   
( ! ) Fatal error: Call to undefined method Doctrine\ORM\Mapping\ClassMetadata::getSqlExecutor() in /home/www/scms/library/Doctrine/ORM/Query.php on line 241
Call Stack
#	Time	Memory	Function	Location
1	0.0000	327100	{main}( )	../index.php:0
2	0.1485	4174724	Zend_Application->run( )	../index.php:47
3	0.1485	4174724	Zend_Application_Bootstrap_Bootstrap->run( )	../Application.php:366
4	0.1486	4174796	Zend_Controller_Front->dispatch( ???, ??? )	../Bootstrap.php:97
5	0.1545	4488224	Zend_Controller_Router_Rewrite->route( object(Zend_Controller_Request_Http)[333] )	../Front.php:911
6	0.1550	4488968	SCMS_Controller_Router_Route->match( string(6), ??? )	../Rewrite.php:398
7	0.1567	4570028	Doctrine\ORM\AbstractQuery->getOneOrNullResult( long )	../Route.php:72
8	0.1567	4570192	Doctrine\ORM\AbstractQuery->execute( array(0), long )	../AbstractQuery.php:571
9	0.1567	4570660	Doctrine\ORM\Query->_doExecute( )	../AbstractQuery.php:733
Variables in local scope (#9)

$executor =

	Undefined

$paramMappings =

	Undefined

$sqlParams =

	Undefined

$types =

	Undefined

error appears out of the blue, after a few refreshes it disappears, but this happens constantly.



 Comments   
Comment by Marco Pivetta [ 05/Jul/12 ]

Can you try namespacing your caches (metadata/query/results) and see if the problem persists?
Also, it would be interesting to see your bootstrap configuration...

Comment by Benjamin Eberlei [ 05/Jul/12 ]

We need the complete bootstrap configuratino and options.

Comment by Justinas [ 09/Jul/12 ]

DoctrineCache.php is a controller plugin that runs first and initializes doctrine cache usually memcache servers
options.ini contains all configuration options for doctrine

Bisna initializes Doctrine based on options and integrates it into ZF

as you can see all caches are namespaced, and i'm currently the only one using system locally so no cache conflicts should occur even without namespaces

Comment by Benjamin Eberlei [ 12/Jul/12 ]

Can you show one of your controller actions/model services where this error occurs, explicitly the code that generates the ResultCache keys.

Comment by Justinas [ 12/Jul/12 ]
if ($em instanceof \Doctrine\ORM\EntityManager) {
			//search for matching vanity urls from most accurate to least accurate
			//matching full slug first /tv/samsung/lcd then /tv/samsung, and then /tv
			foreach ($urls as $possible_match) {
				/* @var $q Doctrine\ORM\Query */
				$q = $em->createQuery('SELECT r
				FROM SCMS\Entity\Route r
				WHERE r.full_slug = :slug');
				$q->setMaxResults(1)
						->setParameter('slug', $possible_match)
						->useResultCache(true, \SCMS\DbCache::TTL_ROUTE, $possible_match);

				$result = $q->getOneOrNullResult(\Doctrine\ORM\Query::HYDRATE_ARRAY);
Comment by Marco Pivetta [ 12/Jul/12 ]

You should definitely namespace your cache (with something like Doctrine\Common\Cache\ApcCache#setNamespace() for example), since you're manually defining your cache key here...

Is the explicit usage of the cache ID intended?

Benjamin Eberlei, should the fetched cache items be checked for their type?

Comment by Benjamin Eberlei [ 12/Jul/12 ]

What are the contents of $urls and $possible_match? You should namespace them even further in your code:

$q->useResultCache(true, \SCMS\DbCache::TTL_ROUTE, "my_query_type_something_" . $possible_match);
Comment by Benjamin Eberlei [ 12/Jul/12 ]

This isn't related to the result cache, but to the query cache though i just realized. However i don't see any way this is possible, the DQL Query Cache key is generated as a hash, the Metadata cache entry is generated using the ClassName as key. I don't udnerstand how the Metadata could end up in a Query Cache key.

@ocramius As a fix, in AbstractQuery and ClassMetadataFactory, we should check if the return value is really instanceof ClassMetadata or ParserResult. And if that happens throw an error. Also we should prefix the metadata and DQL queries ourself.

Comment by Justinas [ 12/Jul/12 ]

$urls is an array of URIs array('/tv/samsung/lcd', '/tv/samsung', '/tv');
$possible_match is the value of array ex.: '/tv/samsung/lcd'

i'm setting the namespace in the configuration, and providing a unique id for each result cache. the documentation isn't clear
is namespace prefixed automatically if you set your cache id or is namespace ignored if you set cache id yourself.
so i always assumed that namespace is prefixed automatically, and that you have to set cacheId yourself for same query with different parameters, so that query searching for '/tv/samsung' would not return the same result as searching for '/tv';

Comment by Benjamin Eberlei [ 12/Jul/12 ]

can you try setting different namespace for Query and Metadata cache? They seem to use the same prefix. Only the Result cache uses its own prefix.

Comment by Marco Pivetta [ 12/Jul/12 ]

Justinas, setting different caches for query, results and metadata allows you to have, as an example, 3 caches writing/reading from APC. If you set a namespace for those caches, those won't collide (everything is handled transparently).

Anyway Benjamin Eberlei is correct when he states that this shouldn't happen here. Even if your approach isn't correct (you don't need to explicitly set any cache key here, plus you should be using namespaced caches) the issue seems to be valid...

Does this happen also with a clean cache?

Comment by Justinas [ 12/Jul/12 ]

i'm not sure, about the clean cache, it just doesn't happen all the time so it's hard to check. If i remember correctly i flushed the cache, and the error appeared, after a few refreshes it disappeared, without flushing cache. it sorta appears and disappears from time to time.

even if i don't set the namespace for my caches i have a local memcache, so no cache collision from other projects shouldn't occur

Comment by Benjamin Eberlei [ 29/Jul/12 ]

Added a guard to avoid this problem. Its not a real fix, but i cannot come up with the a way to reproduce the problem that you have exactly.

Generated at Mon Jul 28 06:33:59 UTC 2014 using JIRA 6.2.3#6260-sha1:63ef1d6dac3f4f4d7db4c1effd405ba38ccdc558.