Skip to content

Commit a25fe6b

Browse files
committed
fix: Tenant toverrides that add to a list failed on meeting create
1 parent b243a5c commit a25fe6b

1 file changed

Lines changed: 6 additions & 6 deletions

File tree

bbblb/model.py

Lines changed: 6 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -279,7 +279,7 @@ def validate_param(self, key, value: str):
279279
raise ValueError("TenantOverride.param must be a non-empty string")
280280
return value
281281

282-
def apply(self, params):
282+
def apply(self, params: dict[str, str]):
283283
if self.op == OPERATOR_FORCE:
284284
if self.value:
285285
params[self.param] = self.value
@@ -295,11 +295,11 @@ def apply(self, params):
295295
if orig <= 0 or orig > int(self.value):
296296
params[self.param] = self.value
297297
elif self.op == OPERATOR_ADD:
298-
params = params.get(self.param, "").split(",")
299-
for add in self.value.split():
300-
if add not in params:
301-
params.append(add)
302-
params[self.param] = ",".join(filter(None, params))
298+
values = params.get(self.param, "").split(",")
299+
for add in self.value.split(","):
300+
if add not in values:
301+
values.append(add)
302+
params[self.param] = ",".join(filter(None, values))
303303

304304
def __str__(self):
305305
return f"TenantOverride({self.tenant}, {self.type}, {self.param}{self.op}{self.value})"

0 commit comments

Comments
 (0)