Primary key field raises FieldDefinitionError with server_default

[rorik302]

Primary key field raises FieldDefinitionError with server_default. For example
id: UUID = UUIDField(primary_key=True, server_default=func.gen_random_uuid()) raises edgy.exceptions.FieldDefinitionError: Primary keys other then IntegerField and BigIntegerField, must provide a default or a server_default.

[tarsil]

@rorik302 could you provide some more details, please? Like what are you trying to achieve, what is your use case. Reason being it's because I'm doing now a patch and this could be part of the fix .

EDIT: there a fix coming for these situations. By the time this was released the server_default wasn't supported

[rorik302]

I want to give postgres to handle default values. It's not necessary, but would be great to have such opportunity

[tarsil]

@rorik302 to unblock you, try this:

id: UUIDField(primary_key=True, default=func.gen_random_uuid)

[tarsil]

@rorik302 it looks like a regression. I will have a look first thing tomorrow before releasing. Hopefully this gets done and released tomorrow

[tarsil]

Primary keys other then IntegerField and BigIntegerField

This is now being pushed to testing and fixed. Hopefully we will run a release today. Nice catch @rorik302 but before doing that, to test this, could you checkout the branch fix/primary_keys and see if this specific error still remains, please?

[rorik302]

I have set
id: UUID = UUIDField(primary_key=True, server_default=text("gen_random_uuid()"))
Get

 id: UUID = UUIDField(primary_key=True, server_default=text("gen_random_uuid()"))
               ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
  File "/lib/python3.12/site-packages/edgy/core/db/fields/core.py", line 442, in __new__
    return super().__new__(cls, **kwargs)
           ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
  File "/lib/python3.12/site-packages/edgy/core/db/fields/core.py", line 82, in __new__
    return Field(**namespace)  # type: ignore
           ^^^^^^^^^^^^^^^^^^
  File "/lib/python3.12/site-packages/edgy/core/db/fields/base.py", line 85, in __init__
    self.raise_for_non_default(default=default_value, server_default=self.server_default)
  File "/lib/python3.12/site-packages/edgy/core/db/fields/base.py", line 113, in raise_for_non_default
    if not self.field_type == int and not default and not server_default:
                                                          ^^^^^^^^^^^^^^
  File "lib/python3.12/site-packages/sqlalchemy/sql/elements.py", line 749, in __bool__
    raise TypeError("Boolean value of this clause is not defined")

[tarsil]

I have set id: UUID = UUIDField(primary_key=True, server_default=text("gen_random_uuid()")) Get

 id: UUID = UUIDField(primary_key=True, server_default=text("gen_random_uuid()"))
               ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
  File "/lib/python3.12/site-packages/edgy/core/db/fields/core.py", line 442, in __new__
    return super().__new__(cls, **kwargs)
           ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
  File "/lib/python3.12/site-packages/edgy/core/db/fields/core.py", line 82, in __new__
    return Field(**namespace)  # type: ignore
           ^^^^^^^^^^^^^^^^^^
  File "/lib/python3.12/site-packages/edgy/core/db/fields/base.py", line 85, in __init__
    self.raise_for_non_default(default=default_value, server_default=self.server_default)
  File "/lib/python3.12/site-packages/edgy/core/db/fields/base.py", line 113, in raise_for_non_default
    if not self.field_type == int and not default and not server_default:
                                                          ^^^^^^^^^^^^^^
  File "lib/python3.12/site-packages/sqlalchemy/sql/elements.py", line 749, in __bool__
    raise TypeError("Boolean value of this clause is not defined")

Sorry, had an issue there. Could you please try again (pull the latest branch)?

[rorik302]

Yes, now it's working

[tarsil]

Release 0.6.0 is now deployed 👍🏼 thank you for your help @rorik302