Added the annotation namespace registration with the AnnotationRegistry #10

Merged
merged 1 commit into from Apr 4, 2012

Conversation

Projects
None yet
4 participants
Contributor

marnusw commented Mar 21, 2012

Since the AnnotationReaders use the AnnotationRegistry to load annotation classes rather than the registered autoloaders the annotation namespace must be registered on the AnnotationRegistry to load properly. This did not reflect in the docs.

Contributor

treffynnon commented Mar 28, 2012

+1

This has cost me too much time. Was about to fork and fix the docs myself.

The error that is returned from couchdb-odm is also incredibly vague on this one. Including it here in case it helps someone find it from a search engine:

Fatal error: Uncaught exception 'Doctrine\ODM\CouchDB\Mapping\MappingException' with message 'Class TestDocument is not a valid document or mapped super class.' in
/couchdb/vendor/doctrine/couchdb-odm/lib/Doctrine/ODM/CouchDB/Mapping/MappingException.php on line 23

Member

lsmith77 commented Apr 4, 2012

it should be noted that this is obviously only necessary when using annotations

beberlei added a commit that referenced this pull request Apr 4, 2012

Merge pull request #10 from marnusw/annot_registry_namespace
Added the annotation namespace registration with the AnnotationRegistry

@beberlei beberlei merged commit c144db2 into doctrine:master Apr 4, 2012

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment