commit fba2acd72e8cbf138912295df227ee2c914158c3
author Paul Yang <TeBoring@users.noreply.github.com> Mon Apr 24 12:40:37 2017 -0700
committer GitHub <noreply@github.com> Mon Apr 24 12:40:37 2017 -0700
tree e67673627fad3a5945f90abcd6cf94b1b9d4eb67
parent e64b618b2132237f9901da0dc76bb0d50ac35d71

Add nested enum descriptor in php rumtime. (#3009)

php/src/Google/Protobuf/Internal/DescriptorPool.php

diff --git a/php/src/Google/Protobuf/Internal/DescriptorPool.php b/php/src/Google/Protobuf/Internal/DescriptorPool.php
index 1ef403c..2c00dfb 100644
--- a/php/src/Google/Protobuf/Internal/DescriptorPool.php
+++ b/php/src/Google/Protobuf/Internal/DescriptorPool.php
@@ -95,6 +95,9 @@
         foreach ($descriptor->getNestedType() as $nested_type) {
             $this->addDescriptor($nested_type);
         }
+        foreach ($descriptor->getEnumType() as $enum_type) {
+            $this->addEnumDescriptor($enum_type);
+        }
     }
 
     public function addEnumDescriptor($descriptor)

php/src/Google/Protobuf/descriptor.php

diff --git a/php/src/Google/Protobuf/descriptor.php b/php/src/Google/Protobuf/descriptor.php
index 9c744a8..fb69eda 100644
--- a/php/src/Google/Protobuf/descriptor.php
+++ b/php/src/Google/Protobuf/descriptor.php
@@ -210,6 +210,12 @@
               $nested_proto, $file_proto, $message_name_without_package));
         }
 
+        // Handle nested enum.
+        foreach ($proto->getEnumType() as $enum_proto) {
+            $desc->addEnumType(EnumDescriptor::buildFromProto(
+              $enum_proto, $file_proto, $message_name_without_package));
+        }
+
         // Handle oneof fields.
         foreach ($proto->getOneofDecl() as $oneof_proto) {
             $desc->addOneofDecl(

php/tests/generated_class_test.php

diff --git a/php/tests/generated_class_test.php b/php/tests/generated_class_test.php
index 554d2be..21ee849 100644
--- a/php/tests/generated_class_test.php
+++ b/php/tests/generated_class_test.php
@@ -839,7 +839,8 @@
     public function testMessageWithoutNamespace()
     {
         $m = new TestMessage();
-        $m->setOptionalNoNamespaceMessage(new NoNameSpaceMessage());
+        $sub = new NoNameSpaceMessage();
+        $m->setOptionalNoNamespaceMessage($sub);
         $m->getRepeatedNoNamespaceMessage()[] = new NoNameSpaceMessage();
 
         $n = new NoNamespaceMessage();