Optional fields do not generate correct defaults in Pydantic about protobuf_to_pydantic HOT 4 CLOSED

Thanks for the feedback, I will fix this in version 0.2.0.1.

from protobuf_to_pydantic.

@so1n There's another bug created that's caused by the fix.

For fields such as:

repeated string a = 1;

It now generates this:

a: typing.Optional[typing.List[str]] = Field(default_factory=list, default=None)

which is not allowed in Pydantic. You cannot have default_factory and a default.

from protobuf_to_pydantic.

@so1n There's another bug created that's caused by the fix.

For fields such as:

repeated string a = 1;

It now generates this:

a: typing.Optional[typing.List[str]] = Field(default_factory=list, default=None)

which is not allowed in Pydantic. You cannot have default_factory and a default.

Please provide the Protobuf file or Message, I can't reproduce the problem at the moment

from protobuf_to_pydantic.

@so1n Here it is:

syntax = "proto3";

message A {
    optional string a = 2;
    repeated string b = 3;
}

Output:

# This is an automatically generated file, please do not change
# gen by protobuf_to_pydantic[v0.2.0.1](https://github.com/so1n/protobuf_to_pydantic)
# Protobuf Version: 4.24.3
# Pydantic Version: 1.10.12
from google.protobuf.message import Message  # type: ignore
from pydantic import BaseModel
from pydantic import Field
import typing


class A(BaseModel):
    a: typing.Optional[str] = Field(default="")
    b: typing.Optional[typing.List[str]] = Field(default_factory=list, default=None)

It doesn't happen if you only have a single repeated field b.

from protobuf_to_pydantic.