http://pear.php.net/bugs/search.php?cmd=display&package_name%5B0%5D=PHP_DocBlockGenerator Search Results en-us pear-webmaster@lists.php.net pear-webmaster@lists.php.net hourly 1 2000-01-01T12:00+00:00 http://pear.php.net/gifs/pearsmall.gif http://pear.php.net/bugs http://pear.php.net/bugs/16193 PHP_DocBlockGenerator Bug Reported by hm2k 2009-05-07T00:04:48+00:00 PHP: 5.2.9 OS: Package Version: 1.1.1 Description: ------------ This is my line: * @param string $path docblockgen turns that into: * @param string $path phpcs gives: Expected 1 space before variable type]]> PHP_DocBlockGenerator Bug Reported by hm2k 2009-05-07T00:04:48+00:00 PHP: 5.2.9 OS: Package Version: 1.1.1 Description: ------------ This is my line: * @param string $path docblockgen turns that into: * @param string $path phpcs gives: Expected 1 space before variable type]]> 2009-05-23T18:07:38+00:00 hm2k &#x61;&#116; php &#x64;&#111;&#x74; net PHP_DocBlockGenerator Bug http://pear.php.net/bugs/16186 PHP_DocBlockGenerator Feature/Change Request Reported by hm2k 2009-05-05T13:22:59+00:00 PHP: 5.2.9 OS: Package Version: 1.1.1 Description: ------------ It would be handy if blockdocgen could do recursion like PHP_Beautifier can. I'd find it very useful to issue a blockdocgen on an entire directory, and output it accordingly. My suggestion would be to take a look how PHP_Beautifier does it, and borrow their method as it seems to work pretty well.]]> PHP_DocBlockGenerator Feature/Change Request Reported by hm2k 2009-05-05T13:22:59+00:00 PHP: 5.2.9 OS: Package Version: 1.1.1 Description: ------------ It would be handy if blockdocgen could do recursion like PHP_Beautifier can. I'd find it very useful to issue a blockdocgen on an entire directory, and output it accordingly. My suggestion would be to take a look how PHP_Beautifier does it, and borrow their method as it seems to work pretty well.]]> 2009-05-05T13:22:59+00:00 hm2k &#x61;&#116; php &#x64;&#111;&#x74; net PHP_DocBlockGenerator Feature/Change Request http://pear.php.net/bugs/16184 PHP_DocBlockGenerator Feature/Change Request Reported by hm2k 2009-05-05T12:55:41+00:00 PHP: 5.2.9 OS: Package Version: Description: ------------ If this package made use of PHP_Parser_DocblockParser, it could streamline development of both packages. It would benefit both packages.]]> PHP_DocBlockGenerator Feature/Change Request Reported by hm2k 2009-05-05T12:55:41+00:00 PHP: 5.2.9 OS: Package Version: Description: ------------ If this package made use of PHP_Parser_DocblockParser, it could streamline development of both packages. It would benefit both packages.]]> 2009-05-05T12:55:41+00:00 hm2k &#x61;&#116; php &#x64;&#111;&#x74; net PHP_DocBlockGenerator Feature/Change Request http://pear.php.net/bugs/16181 PHP_DocBlockGenerator Bug Reported by hm2k 2009-05-05T12:31:07+00:00 PHP: 5.2.9 OS: Package Version: 1.1.1 Description: ------------ After running docblockgen, even though it sorts out the spacing, it doesn't fix the ordering outlined by PHP_CodeSniffer. eg: The @license tag is in the wrong order; the tag follows @copyright (if used) or @author The @copyright tag is in the wrong order; the tag follows @author The @link tag is in the wrong order; the tag follows @version ...]]> PHP_DocBlockGenerator Bug Reported by hm2k 2009-05-05T12:31:07+00:00 PHP: 5.2.9 OS: Package Version: 1.1.1 Description: ------------ After running docblockgen, even though it sorts out the spacing, it doesn't fix the ordering outlined by PHP_CodeSniffer. eg: The @license tag is in the wrong order; the tag follows @copyright (if used) or @author The @copyright tag is in the wrong order; the tag follows @author The @link tag is in the wrong order; the tag follows @version ...]]> 2009-05-05T12:31:07+00:00 hm2k &#x61;&#116; php &#x64;&#111;&#x74; net PHP_DocBlockGenerator Bug http://pear.php.net/bugs/15490 PHP_DocBlockGenerator Feature/Change Request Reported by hshimooka 2009-01-03T07:26:37+00:00 PHP: 5.2.8 OS: CentOS 5.2 Package Version: 1.1.1 Description: ------------ I use PHP_DocBlockGenerator-1.1.1 with PHP_CompatInfo-1.8.1 . I ran docblockgen, then displayed messages like 'print_r'. I read source codes both of PHP_DocBlockGenerator and PHP_CompatInfo, and found that the constructor of PHP_CompatInfo added PHP_CompatInfo_Renderer_Array to its listener by default. If you can, toggle thats messages with some option, or not display it. In the latter case, I think ok to pass 'null' string to the constructor of PHP_CompatInfo in PHP_DocBlockGenerator_Tokens's constructor . -=-=-=-=- $ diff /usr/local/lib/php5/pear/PHP/DocBlockGenerator/Tokens.php.org /usr/local/lib/php5/pear/PHP/DocBlockGenerator/Tokens.php 150c150 < $this->info = new PHP_CompatInfo(); --- > $this->info = new PHP_CompatInfo("null"); $ Test script: --------------- $ docblockgen /path/to/sample.php /path/to/commented.php Expected result: ---------------- $ docblockgen /path/to/sample.php /path/to/commented.php $ Actual result: -------------- $ docblockgen /path/to/sample.php /path/to/commented.php array ( 'ignored_files' => array ( ), 'ignored_functions' => array ( ), 'ignored_extensions' => array ( ), 'ignored_constants' => array ( ), 'max_version' => '', 'version' => '5.0.0', 'extensions' => array ( ), 'constants' => array ( 0 => 'PHP_EOL', ), 'tokens' => array ( 0 => 'public', ), 'cond_code' => array ( 0 => 0, ), )$]]> PHP_DocBlockGenerator Feature/Change Request Reported by hshimooka 2009-01-03T07:26:37+00:00 PHP: 5.2.8 OS: CentOS 5.2 Package Version: 1.1.1 Description: ------------ I use PHP_DocBlockGenerator-1.1.1 with PHP_CompatInfo-1.8.1 . I ran docblockgen, then displayed messages like 'print_r'. I read source codes both of PHP_DocBlockGenerator and PHP_CompatInfo, and found that the constructor of PHP_CompatInfo added PHP_CompatInfo_Renderer_Array to its listener by default. If you can, toggle thats messages with some option, or not display it. In the latter case, I think ok to pass 'null' string to the constructor of PHP_CompatInfo in PHP_DocBlockGenerator_Tokens's constructor . -=-=-=-=- $ diff /usr/local/lib/php5/pear/PHP/DocBlockGenerator/Tokens.php.org /usr/local/lib/php5/pear/PHP/DocBlockGenerator/Tokens.php 150c150 < $this->info = new PHP_CompatInfo(); --- > $this->info = new PHP_CompatInfo("null"); $ Test script: --------------- $ docblockgen /path/to/sample.php /path/to/commented.php Expected result: ---------------- $ docblockgen /path/to/sample.php /path/to/commented.php $ Actual result: -------------- $ docblockgen /path/to/sample.php /path/to/commented.php array ( 'ignored_files' => array ( ), 'ignored_functions' => array ( ), 'ignored_extensions' => array ( ), 'ignored_constants' => array ( ), 'max_version' => '', 'version' => '5.0.0', 'extensions' => array ( ), 'constants' => array ( 0 => 'PHP_EOL', ), 'tokens' => array ( 0 => 'public', ), 'cond_code' => array ( 0 => 0, ), )$]]> 2009-01-03T07:26:37+00:00 shimooka &#x61;&#116; doyouphp &#x64;&#111;&#x74; jp PHP_DocBlockGenerator Feature/Change Request http://pear.php.net/bugs/14222 PHP_DocBlockGenerator Bug Reported by unomi 2008-06-24T05:38:37+00:00 PHP: 5.2.6 OS: CentOS 4.6 Final Package Version: 1.1.1 Description: ------------ When running docblockgen on one of our files, it suddenly prompted (via Xdebug), that an index is undefined. It all happens in the guessReturn function, where $var['value'] sometimes is not defined, so a Undefined index: 'value' occurs. Another one is in the function isAVar where the $id gets incremented during the if-statements. But if all the statements fail, the 'else' false back at the end has to deal with an $id that has been incremented. But then a Undefined offset: 1 occurs. Expected result: ---------------- Sorry, I can't provide you with an example, but the description should be explainatory. The only thing I can give you that type 319 (T_ENDWHILE) of the tokens didn't had a 'value' index. Actual result: -------------- Sorry, I can't provide an actual result either. The functions mentioned are flawed and should be fixed. That's all that I can say. An extra check on the 'value' index should be sufficient and in the function isAVar there should be a default current value for $tokens[$id]. After that the $id gets incremented, but the 'else' fallback statement depends on what was current before the if statements. Like: $cur = $tokens[$id]; if (++$id stuff etc.) { elseif (++$id stuff etc.) { elseif (++$id stuff etc.) { else { $var = $cur; }]]> PHP_DocBlockGenerator Bug Reported by unomi 2008-06-24T05:38:37+00:00 PHP: 5.2.6 OS: CentOS 4.6 Final Package Version: 1.1.1 Description: ------------ When running docblockgen on one of our files, it suddenly prompted (via Xdebug), that an index is undefined. It all happens in the guessReturn function, where $var['value'] sometimes is not defined, so a Undefined index: 'value' occurs. Another one is in the function isAVar where the $id gets incremented during the if-statements. But if all the statements fail, the 'else' false back at the end has to deal with an $id that has been incremented. But then a Undefined offset: 1 occurs. Expected result: ---------------- Sorry, I can't provide you with an example, but the description should be explainatory. The only thing I can give you that type 319 (T_ENDWHILE) of the tokens didn't had a 'value' index. Actual result: -------------- Sorry, I can't provide an actual result either. The functions mentioned are flawed and should be fixed. That's all that I can say. An extra check on the 'value' index should be sufficient and in the function isAVar there should be a default current value for $tokens[$id]. After that the $id gets incremented, but the 'else' fallback statement depends on what was current before the if statements. Like: $cur = $tokens[$id]; if (++$id stuff etc.) { elseif (++$id stuff etc.) { elseif (++$id stuff etc.) { else { $var = $cur; }]]> 2008-06-24T05:38:37+00:00 aschwin &#x61;&#116; hotelspecials &#x64;&#111;&#x74; nl PHP_DocBlockGenerator Bug http://pear.php.net/bugs/14162 PHP_DocBlockGenerator Feature/Change Request Reported by hshimooka 2008-06-17T04:26:43+00:00 PHP: 5.2.6 OS: Package Version: 1.1.1 Description: ------------ If I want to use 'MY' license, perhaps I must not use '--license' option and write that licese per script file. So I made a patch against License.php: http://coderepos.org/share/browser/lang/php/PHP_DocBlockGenerator/PHP/DocBlockGenerator/License.php.patch?rev=14166&format=raw and modified License.php is: http://coderepos.org/share/browser/lang/php/PHP_DocBlockGenerator/PHP/DocBlockGenerator/License.php?rev=14166 The license files will be located in PEAR_INSTALL_DIR/data/PHP_DocBlockGenerator/: http://coderepos.org/share/browser/lang/php/PHP_DocBlockGenerator/data/PHP_DocBlockGenerator?rev=14166 In a license file, the 1st line is 'full_name', the 2nd is 'url' and 'text' is follwing.]]> PHP_DocBlockGenerator Feature/Change Request Reported by hshimooka 2008-06-17T04:26:43+00:00 PHP: 5.2.6 OS: Package Version: 1.1.1 Description: ------------ If I want to use 'MY' license, perhaps I must not use '--license' option and write that licese per script file. So I made a patch against License.php: http://coderepos.org/share/browser/lang/php/PHP_DocBlockGenerator/PHP/DocBlockGenerator/License.php.patch?rev=14166&format=raw and modified License.php is: http://coderepos.org/share/browser/lang/php/PHP_DocBlockGenerator/PHP/DocBlockGenerator/License.php?rev=14166 The license files will be located in PEAR_INSTALL_DIR/data/PHP_DocBlockGenerator/: http://coderepos.org/share/browser/lang/php/PHP_DocBlockGenerator/data/PHP_DocBlockGenerator?rev=14166 In a license file, the 1st line is 'full_name', the 2nd is 'url' and 'text' is follwing.]]> 2008-06-17T04:26:43+00:00 shimooka &#x61;&#116; doyouphp &#x64;&#111;&#x74; jp PHP_DocBlockGenerator Feature/Change Request