use prettyPrinter to extract the value of a constant

Author: jaapio

src/phpDocumentor/Reflection/Php/Factory/ClassConstantIterator.php

@@ -87,6 +87,11 @@ public function getDocComment()
         return $docComment;
     }
 
+    public function getValue()
+    {
+        return $this->classConstants->consts[$this->index]->value;
+    }
+
     /**
      * (PHP 5 &gt;= 5.0.0)<br/>
      * Return the current element

src/phpDocumentor/Reflection/Php/Factory/Constant.php

@@ -16,7 +16,9 @@
 use phpDocumentor\Reflection\Php\Constant as ConstantElement;
 use phpDocumentor\Reflection\Php\ProjectFactoryStrategy;
 use phpDocumentor\Reflection\Php\StrategyContainer;
+use phpDocumentor\Reflection\PrettyPrinter;
 use phpDocumentor\Reflection\Types\Context;
+use PhpParser\Comment\Doc;
 
 /**
  * Strategy to convert ClassConstantIterator to ConstantElement
@@ -26,6 +28,21 @@
  */
 class Constant implements ProjectFactoryStrategy
 {
+    /**
+     * @var PrettyPrinter
+     */
+    private $valueConverter;
+
+    /**
+     * Initializes the object.
+     *
+     * @param PrettyPrinter $prettyPrinter
+     */
+    public function __construct(PrettyPrinter $prettyPrinter)
+    {
+        $this->valueConverter = $prettyPrinter;
+    }
+
     /**
      * Returns true when the strategy is able to handle the object.
      *
@@ -49,6 +66,28 @@ public function matches($object)
      */
     public function create($object, StrategyContainer $strategies, Context $context = null)
     {
-        return new ConstantElement($object->getFqsen());
+        $docBlock = $this->createDocBlock($object->getDocComment(), $strategies, $context);
+        $default = null;
+        if ($object->getValue() !== null) {
+            $default = $this->valueConverter->prettyPrintExpr($object->getValue());
+        }
+
+        return new ConstantElement($object->getFqsen(), $docBlock, $default);
+    }
+
+    /**
+     * @param Doc $docBlock
+     * @param StrategyContainer $strategies
+     * @param Context $context
+     * @return null|\phpDocumentor\Reflection\DocBlock
+     */
+    private function createDocBlock(Doc $docBlock = null, StrategyContainer $strategies, Context $context = null)
+    {
+        if ($docBlock === null) {
+            return null;
+        }
+
+        $strategy = $strategies->findMatching($docBlock);
+        return $strategy->create($docBlock, $strategies, $context);
     }
 }

src/phpDocumentor/Reflection/Php/ProjectFactory.php

@@ -50,7 +50,7 @@ public static function createInstance()
             [
                 new Factory\Argument(),
                 new Factory\Class_(),
-                new Factory\Constant(),
+                new Factory\Constant(new PrettyPrinter()),
                 new Factory\DocBlock(DocBlockFactory::createInstance()),
                 new Factory\File(NodesFactory::createInstance()),
                 new Factory\Function_(),

tests/component/ProjectCreationTest.php

@@ -56,6 +56,9 @@ public function testCreateProjectWithClass()
         $this->assertArrayHasKey($fileName, $project->getFiles());
         $this->assertArrayHasKey('\\Pizza', $project->getFiles()[$fileName]->getClasses());
         $this->assertArrayHasKey('\\Pizza::PACKAGING', $project->getFiles()[$fileName]->getClasses()['\\Pizza']->getConstants());
+        $constant = $project->getFiles()[$fileName]->getClasses()['\\Pizza']->getConstants()['\\Pizza::PACKAGING'];
+
+        $this->assertEquals('box',$constant->getValue());
     }
 
     public function testWithNamespacedClass()