Minor issue in arguments for AttributeImpl
#10474
-
|
Hi In if impl_class:
# TODO: this appears to be the WriteOnlyAttributeImpl /
# DynamicAttributeImpl constructor which is hardcoded
impl = cast("Type[WriteOnlyAttributeImpl]", impl_class)(
class_, key, dispatch, **kw
)but in
def __init__(
self,
class_: _ExternalEntityType[_O],
key: str,
callable_: Optional[_LoaderCallable], # < - third argument.
dispatch: _Dispatch[QueryableAttribute[Any]],
trackparent: bool = False,
...
)Not sure, but if downgrade my sqlalchemy to 2.0.21 it works but with sqlalchemy 2.0.22 it fails. |
Beta Was this translation helpful? Give feedback.
Replies: 2 comments 10 replies
-
|
Hi, With what you have posted it's impossible to provide any help. How are you getting that error? That code path is used for write only, that doesn't have callable as argument sqlalchemy/lib/sqlalchemy/orm/writeonly.py Line 164 in d2ce313 Please provide an example of what you are trying to do, thanks. |
Beta Was this translation helpful? Give feedback.
-
|
to ...make their own kinds of mapped attributes ? I haven't looked at sqlalchemy_utils. is that what they're doing ? |
Beta Was this translation helpful? Give feedback.
-
|
apparently they define a so called |
Beta Was this translation helpful? Give feedback.
-
|
it's probably what we have here: https://docs.sqlalchemy.org/en/20/_modules/examples/generic_associations/generic_fk.html but I really would have preferred they didnt make their own attribute impl |
Beta Was this translation helpful? Give feedback.
-
|
Hi @CaselIT ,
|
Beta Was this translation helpful? Give feedback.
-
|
The api is used internally by sqlalchemy and there was no expectation of it being stable. The code that it calls impl has changed following the addition of types to some of the modules: 0a12d22 |
Beta Was this translation helpful? Give feedback.
-
|
Fixed by kvesteri/sqlalchemy-utils#725 |
Beta Was this translation helpful? Give feedback.
Fixed by kvesteri/sqlalchemy-utils#725