Package home | Report new bug | New search | Development Roadmap Status: Open | Feedback | All | Closed Since Version 1.5.0a1

Bug #12201 Misleading error message
Submitted: 2007-10-08 07:56 UTC Modified: 2007-12-09 01:11 UTC
From: yunosh Assigned: ashnazg
Status: Closed Package: PhpDocumentor (version 1.4.0)
PHP Version: Irrelevant OS:
Roadmaps: 1.4.1    
Subscription  


 [2007-10-08 07:56 UTC] yunosh (Jan Schneider)
Description: ------------ Running phpdoc on the SOAP package generates a lot of these warnings: Warning on line 26 - package default is already in category Web-Services, will now replace with category Web-Services Beside that I don't have any @category tags that don't have a @package tag in the same doc block, the code that raises this warning looks suspicious. Line 1126 of IntermediateParser.inc says: if (isset($this->packagecategories[$this->package]) && $this->packagecategories[$this->package] != $this->category) addWarning(PDERROR_PACKAGECAT_SET,$this->package, $this->packagecategories[$this->package], $data->category); The test is against $this->category, but the error message refers to $data->category.

Comments

 [2007-10-08 07:59 UTC] yunosh (Jan Schneider)
Another issue that might be related is that the @category tag is actually "Web Services", while the error message uses "Web-Services".
 [2007-12-08 21:10 UTC] ashnazg (Chuck Burgess)
Noticed this oddity about @category myself before... Testing Jan's theory about that $data needing to be $this...
 [2007-12-08 23:10 UTC] ashnazg (Chuck Burgess)
Looks to me like whenever an @category value has spaces in its value, PhpDocumentor replaces the spaces with dashes, thereby avoiding having to die due to an invalid category value... I'm still tinkering and testing on this, but at this point I think the $this->category in the IF condition is what should be $data->category instead...
 [2007-12-09 01:11 UTC] ashnazg (Chuck Burgess)
Yep, that was it... Committed fix to CVS.