Use static_format over format if present in Asset.replace

This commit is contained in:
Soheab 2022-06-10 07:50:19 +02:00 committed by GitHub
parent 66a97c6655
commit dd52f856f3
No known key found for this signature in database
GPG Key ID: 4AEE18F83AFDEB23

View File

@ -353,6 +353,11 @@ class Asset(AssetMixin):
) -> Self:
"""Returns a new asset with the passed components replaced.
.. versionchanged:: 2.0
``static_format`` is now preferred over ``format``
if both are present and the asset is not animated.
.. versionchanged:: 2.0
This function will now raise :exc:`ValueError` instead of
``InvalidArgument``.
@ -386,7 +391,7 @@ class Asset(AssetMixin):
if format not in VALID_ASSET_FORMATS:
raise ValueError(f'format must be one of {VALID_ASSET_FORMATS}')
else:
if format not in VALID_STATIC_FORMATS:
if static_format is MISSING and format not in VALID_STATIC_FORMATS:
raise ValueError(f'format must be one of {VALID_STATIC_FORMATS}')
url = url.with_path(f'{path}.{format}')