Zend Framework

Zend_Reflection_Docblock::__toString() must return a string value

Details

  • Type: Bug Bug
  • Status: Resolved Resolved
  • Priority: Minor Minor
  • Resolution: Fixed
  • Affects Version/s: 1.9.1, 1.9.2, 1.9.3, 1.9.4
  • Fix Version/s: 1.9.5
  • Component/s: Zend_Reflection
  • Labels:
    None

Description

 
kms > phpunit:

      [php] Evaluating PHP expression: PATH_SEPARATOR
   [delete] Deleting: /home/jkleijn/kmsbuild/kzflib/phpunit/bootstrap.php
   [append] Appending string to /home/jkleijn/kmsbuild/kzflib/phpunit/bootstrap.php
     [exec] Executing command: phpunit    --configuration /var/www/kms/testing/build/projects/kzflib/phpunit/standardConfig.xml    --bootstrap /home/jkleijn/kmsbuild/kzflib/phpunit/bootstrap.php    /home/jkleijn/kmsbuild/kzflib/source/kwd/kzflib/tests/kzflibSuite.php 2>&1
     [exec]      PHPUnit 3.3.17 by Sebastian Bergmann.
     [exec] 
     [exec] E
     [exec] 
     [exec] Time: 0 seconds
     [exec] 
     [exec] There was 1 error:
     [exec] 
     [exec] 1) testCanCreateDecorator(kwd_kzflib_tests_DecoratorGeneratorTest)
     [exec] Method Zend_Reflection_Docblock::__toString() must return a string value
     [exec] /home/jkleijn/kmsbuild/kzflib/export/dependencies/52f4c5a5877ea833f105e1b8d9047687/Zend/CodeGenerator/Php/Property.php:68
     [exec] /home/jkleijn/kmsbuild/kzflib/export/dependencies/52f4c5a5877ea833f105e1b8d9047687/Zend/CodeGenerator/Php/Class.php:124
     [exec] /home/jkleijn/kmsbuild/kzflib/export/kwd/kzflib/codegenerator/ProxyGenerator.php:28
     [exec] /home/jkleijn/kmsbuild/kzflib/export/kwd/kzflib/codegenerator/DecoratorGenerator.php:24
     [exec] /home/jkleijn/kmsbuild/kzflib/export/kwd/kzflib/tests/codegenerator/DecoratorGeneratorTest.php:20
     [exec] 
            FAILURES!
            Tests: 1, Assertions: 0, Errors: 1.

This issue exists in the 1.9.0 as well as the 1.9.1 tags.

Thanks,

John Kleijn

Activity

Ascending order - Click to sort in descending order
Hide
Permalink
John Kleijn added a comment -

Anyone? Well over a month now, two minor releases, no reponses, quick fix..

Thanks.

Show
John Kleijn added a comment - Anyone? Well over a month now, two minor releases, no reponses, quick fix.. Thanks.
Hide
Permalink
John Kleijn added a comment -

Hi anyone. It has been two months since I opened this ticket. What do I have to do get it fixed? Submit a patch? If you take 5 minutes of your time to take a look at it you'll find it is pretty trivial.

Many thanks to anyone who can get this fix in a next release.

Show
John Kleijn added a comment - Hi anyone. It has been two months since I opened this ticket. What do I have to do get it fixed? Submit a patch? If you take 5 minutes of your time to take a look at it you'll find it is pretty trivial. Many thanks to anyone who can get this fix in a next release.
Hide
Permalink
Benjamin Eberlei added a comment -

Fixed in trunk and merged back into 1.9 release branch.

Show
Benjamin Eberlei added a comment - Fixed in trunk and merged back into 1.9 release branch.
Hide
Permalink
Carlton Gibson added a comment -

The submitted fix here solves the obvious issue but when I discussed this with Ralph prior to 1.9.3 he suggested that the solution required something in line with the general policy for Reflection::export(). The trouble was at the time that we didn't know what that was...

I guess my question is have we solved that here, or are we just concealing another issue for later? (Ralph did say that he'd inquire of the Reflection maintainer but I never found out if that happened or what the results were if he did.)

Show
Carlton Gibson added a comment - The submitted fix here solves the obvious issue but when I discussed this with Ralph prior to 1.9.3 he suggested that the solution required something in line with the general policy for Reflection::export(). The trouble was at the time that we didn't know what that was... I guess my question is have we solved that here, or are we just concealing another issue for later? (Ralph did say that he'd inquire of the Reflection maintainer but I never found out if that happened or what the results were if he did.)
Hide
Permalink
John Kleijn added a comment -

Perhaps, but it at least fixes the immediate issue. IMO the whole reflection and codeGenerator packages are in desperate need of some peer review.

Show
John Kleijn added a comment - Perhaps, but it at least fixes the immediate issue. IMO the whole reflection and codeGenerator packages are in desperate need of some peer review.

People

Vote (0)
Watch (1)

Dates

  • Created:
    Updated:
    Resolved:
Atlassian JIRA (v4.4.3#663-r165197) | Report a problem
Powered by a free Atlassian JIRA open source license for Zend Framework. Try JIRA - bug tracking software for your team.