Doctrine Project

Doctrine Project http://www.doctrine-project.org/jira This file is an XML representation of an issue en-us

5.2.7

850

21-02-2013

[CODM-28] Adjust to new Annotation Reader http://www.doctrine-project.org/jira/browse/CODM-28 Doctrine CouchDB ODM CODM-28

Adjust to new Annotation Reader

Bug Major Open Unresolved Benjamin Eberlei Benjamin Eberlei Sat, 25 Jun 2011 09:50:24 +0000 Wed, 10 Aug 2011 06:50:48 +0000 0 0 I obtained the latest build from <tt>git://github.com/doctrine/couchdb-odm.git</tt> so this is an active bug which I believe is related to this issue. The <tt>Doctrine\ODM\CouchDB\Mapping\Driver\AnnotationDriver</tt> produced the exception <cite>Class Test is not a valid document or mapped super class</cite> for even the most basic document with an Id and one Field. The exception was thrown on line 122 of the above class. The problem: In the code segment: <tt>108: $classAnnotations = $this->reader->getClassAnnotations($reflClass);</tt> <tt>109: if (isset($classAnnotations['Doctrine\ODM\CouchDB\Mapping\Document']))</tt> the getClassAnnotations method does not return an array of annotations indexed by annotation names, it uses numeric indexes. This causes the if-statements to always fail and thus the exception is thrown. A temporary workaround: <tt>108: $classAnnotations = $this->reader->getClassAnnotations($reflClass);</tt> <tt>Add: $classAnnotations = $classAnnotations[0];</tt> <tt>109: if ($classAnnotations instanceof \Doctrine\ODM\CouchDB\Mapping\Document) {</tt> <tt>110: $documentAnnot = $classAnnotations;</tt> Rather than checking for the proper index verify the class of the annotation object (Similar to the handling of $fieldAnnot following this code block). This approach assumes that the @Document annotation is the first Doctrine related annotation in the class doc block (which is fine, since it is the only one that should be used). Standard annotations such as @author can precede the @Document one since these are ignored by the getClassAnnotations method. The same changes should be applied to all the if statements up to line 120.