Replaces isNullable with isOptional.

Signed-off-by: Johyn Papin <[email protected]>

Author: Johyn Papin

protoc_plugin/lib/src/message_generator.dart

@@ -490,7 +490,7 @@ class MessageGenerator extends ProtobufContainer {
     var defaultExpr = field.getDefaultExpr();
     var names = field.memberNames;
 
-    if (useNullable && field.isNullable) {
+    if (useNullable && field.isOptional) {
       fieldTypeString += '?';
     }
 
@@ -503,7 +503,7 @@ class MessageGenerator extends ProtobufContainer {
         defaultExpr,
         field.isRepeated,
         field.isMapField,
-        useNullable && field.isNullable);
+        useNullable && field.isOptional);
     out.printlnAnnotated(
         '$fieldTypeString get ${names!.fieldName} => $getterExpr;', [
       NamedLocation(
@@ -531,7 +531,7 @@ class MessageGenerator extends ProtobufContainer {
       _emitOverrideIf(field.overridesSetter, out);
       _emitIndexAnnotation(field.number, out);
       if (fastSetter != null) {
-        if (useNullable && field.isNullable) {
+        if (useNullable && field.isOptional) {
           fastSetter += 'Nullable';
         }
         out.printlnAnnotated(
@@ -547,7 +547,7 @@ class MessageGenerator extends ProtobufContainer {
             ]);
       } else {
         final setterName =
-            useNullable && field.isNullable ? 'setFieldNullable' : 'setField';
+            useNullable && field.isOptional ? 'setFieldNullable' : 'setField';
 
         out.printlnAnnotated(
             'set ${names.fieldName}'

protoc_plugin/lib/src/protobuf_field.dart

@@ -65,6 +65,12 @@ class ProtobufField {
   bool get isRepeated =>
       descriptor.label == FieldDescriptorProto_Label.LABEL_REPEATED;
 
+  bool get isOptional {
+    if (isRepeated) return false;
+    if (isRequired || !descriptor.proto3Optional) return false;
+    return true;
+  }
+
   /// Whether a numeric field is repeated and must be encoded with packed
   /// encoding.
   ///
@@ -140,12 +146,6 @@ class ProtobufField {
     // for example in package:protobuf/src/protobuf/mixins/well_known.dart.
   }
 
-  bool get isNullable {
-    if (isRepeated) return false;
-    if (isRequired) return false;
-    return descriptor.proto3Optional || baseType.isMessage;
-  }
-
   /// Returns the expression to use for the Dart type.
   String getDartType() {
     if (isMapField) {