From 417bbd178bf5904e2ece7a66c97ac34272892f3f Mon Sep 17 00:00:00 2001 From: FrederikBaerentsen Date: Thu, 18 Dec 2025 22:16:14 -0500 Subject: [PATCH] fix(meta): fixed an issue where owner, status and tag didn't save on sets detail page --- bricktracker/metadata.py | 11 +++++++---- 1 file changed, 7 insertions(+), 4 deletions(-) diff --git a/bricktracker/metadata.py b/bricktracker/metadata.py index a285bd1..50132ec 100644 --- a/bricktracker/metadata.py +++ b/bricktracker/metadata.py @@ -191,15 +191,18 @@ class BrickMetadata(BrickRecord): parameters['set_id'] = brickset.fields.id parameters['state'] = state - rows, _ = BrickSQL().execute( + rows, _ = BrickSQL().execute_and_commit( self.update_set_state_query, parameters=parameters, - defer=True, name=self.as_column(), ) - # Note: rows will be -1 when deferred, so we can't validate here - # Validation will happen at final commit in set.py + if rows != 1: + raise DatabaseException('Could not update the {kind} state for set {set} ({id})'.format( + kind=self.kind, + set=brickset.fields.set, + id=brickset.fields.id, + )) # Info logger.info('{kind} "{name}" state changed to "{state}" for set {set} ({id})'.format( # noqa: E501