Skip to content

Commit

Permalink
ARB: typing fix in data manager
Browse files Browse the repository at this point in the history
get_original_data always needs to return ARB values in the internal
string representation. They should only be transformed to/from arb_data
in the public get/put methods.
  • Loading branch information
stsnel committed Oct 19, 2023
1 parent fb7a9a5 commit 840c43c
Showing 1 changed file with 2 additions and 2 deletions.
4 changes: 2 additions & 2 deletions util/arb_data_manager.py
Original file line number Diff line number Diff line change
Expand Up @@ -62,12 +62,12 @@ def _get_original_data(self, ctx, keyname):

if len(arb_data) == 0:
# If we don't have an ARB value, ARB should ignore this resource
return constants.arb_status.IGNORE
return constants.arb_status.IGNORE.value
elif len(arb_data) == 1:
return arb_data[0][0]
else:
log.write(ctx, "WARNING: multiple ARB AVUs present for resource '{}'. ARB will ignore it.".format(keyname))
return constants.arb_status.IGNORE
return constants.arb_status.IGNORE.value

def _put_original_data(self, ctx, keyname, data):
"""This function is called when data needs to be updated in the original
Expand Down

0 comments on commit 840c43c

Please sign in to comment.