http://www.doctrine-project.org/jira This file is an XML representation of an issue en-us 5.2.7 850 21-02-2013 http://www.doctrine-project.org/jira/browse/DCOM-106 Doctrine Common <p>I was wondering if you would consider adding @todo and @fixme</p> <p>After doing some research, I'm not sure if @fixme is used anywhere other than being in Eclipse's PDT. However, @todo is definitely on the common tags of wiki page about PHPdoc as well as the phpDoc manual.</p> <p><a href="http://en.wikipedia.org/wiki/PHPDoc#Tags" class="external-link">http://en.wikipedia.org/wiki/PHPDoc#Tags</a><br/> <a href="http://manual.phpdoc.org/HTMLSmartyConverter/PHP/phpDocumentor/tutorial_tags.todo.pkg.html" class="external-link">http://manual.phpdoc.org/HTMLSmartyConverter/PHP/phpDocumentor/tutorial_tags.todo.pkg.html</a></p> DCOM-106

Add @todo and @fixme to AnnotationReader::$globalIgnoredNames

Improvement Minor Resolved Fixed Benjamin Eberlei Stephen Ostrow Wed, 12 Sep 2012 15:33:01 +0000 Wed, 21 Nov 2012 00:06:25 +0000 Wed, 21 Nov 2012 00:06:25 +0000 2.3 Annotations 0 3 <p>Todo is already built in:<br/> <a href="https://github.com/doctrine/common/blob/master/lib/Doctrine/Common/Annotations/AnnotationReader.php#L63" class="external-link">https://github.com/doctrine/common/blob/master/lib/Doctrine/Common/Annotations/AnnotationReader.php#L63</a></p> <p>Sorry about that. I normally write them as @TODO so the standout more. Then when I got that exception and started doing research I swear I had tried @todo. But now looking back, I bet I tried @todo and still got exceptions which were from other bugs I had going on. I guess we can close this unless anyone thinks @fixme should be in there as well. </p> <p>But like I said in the description, I'm not sure if @fixme is a common or just from Eclipse.</p> <p>Netbeans matches that one too</p> <p>I've prepared a fix for this issue that makes both @fixme and @TODO ignored. Pull request: <a href="https://github.com/doctrine/common/pull/223" class="external-link">https://github.com/doctrine/common/pull/223</a>.</p>