Skip to content

Commit 72d7c08

Browse files
committed
Removed one use of sqlalchemy lower function in marshables
1 parent 8a0c1ac commit 72d7c08

File tree

1 file changed

+3
-5
lines changed

1 file changed

+3
-5
lines changed

source/app/schema/marshables.py

Lines changed: 3 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -97,6 +97,7 @@
9797
from app.schema.utils import str_to_bool
9898
from app.business.users import get_primary_organisation
9999
from app.business.users import get_organisations
100+
from app.datamgmt.case.assets_type import get_asset_type_by_name_case_insensitive
100101

101102

102103
ALLOWED_EXTENSIONS = {'png', 'svg'}
@@ -621,11 +622,8 @@ def verify_unique(self, data: Dict[str, Any], **kwargs: Any) -> Dict[str, Any]:
621622
type=int,
622623
allow_none=True)
623624

624-
client = AssetsType.query.filter(
625-
func.lower(AssetsType.asset_name) == func.lower(data.asset_name),
626-
AssetsType.asset_id != data.asset_id
627-
).first()
628-
if client:
625+
asset_type = get_asset_type_by_name_case_insensitive(data.asset_name)
626+
if asset_type and asset_type.asset_id != data.asset_id:
629627
raise ValidationError('Asset type name already exists', field_name='asset_name')
630628

631629
return data

0 commit comments

Comments
 (0)