diff options
| author | Adam Cozzette <acozzette@google.com> | 2016-09-27 15:36:41 -0700 |
|---|---|---|
| committer | Bo Yang <teboring@google.com> | 2016-10-10 11:44:21 -0700 |
| commit | 4a4a1627c158d976ee80f278024a49ced8b12712 (patch) | |
| tree | 7f619a542f6e83f251284b7561a789672df94b11 /js/test2.proto | |
| parent | 0bf5482712e849f67d7fe98217608f95ba60c776 (diff) | |
| download | protobuf-4a4a1627c158d976ee80f278024a49ced8b12712.tar.gz protobuf-4a4a1627c158d976ee80f278024a49ced8b12712.tar.bz2 protobuf-4a4a1627c158d976ee80f278024a49ced8b12712.zip | |
Fixed references to foreign nested messages with CommonJS-style imports
A bug was causing generated JSPB code with CommonJS-style imports to
refer incorrectly to nested messages from other .proto files. The
generated code would have things like "test_pb.InnerMessage" instead of
"test_pb.OuterMessage.InnerMessage". This commit fixes the problem by
correctly taking into account any message nesting.
Diffstat (limited to 'js/test2.proto')
| -rw-r--r-- | js/test2.proto | 6 |
1 files changed, 6 insertions, 0 deletions
diff --git a/js/test2.proto b/js/test2.proto index 44e55eff..b67f93fa 100644 --- a/js/test2.proto +++ b/js/test2.proto @@ -35,6 +35,8 @@ option java_multiple_files = true; package jspb.test; +import "test.proto"; + message TestExtensionsMessage { optional int32 intfield = 1; extensions 100 to max; @@ -52,3 +54,7 @@ extend TestExtensionsMessage { optional ExtensionMessage floating_msg_field = 101; optional string floating_str_field = 102; } + +message ForeignNestedFieldMessage { + optional Deeply.Nested.Message deeply_nested_message = 1; +} |