Do not expose variable_count to the user #27

New Issue

Closed

opened 2020-10-22 20:34:46 +00:00 by projectmoon · 2 comments

projectmoon commented

2020-10-22 20:34:46 +00:00

Owner

Variable count is stored as a variable for the user. Do not allow it to be deleted. It also should not show up in lists, or count towards the limit.

projectmoon commented

2020-10-23 20:20:16 +00:00

Author

Owner

Solution is to add a metadata key space in the variables tree, where we keep track of the variable count with a different prefix.

I suppose we could also put it in an entirely different metadata tree, but maybe that's not so useful.

Solution is to add a metadata key space in the variables tree, where we keep track of the variable count with a different prefix. I suppose we could also put it in an entirely different metadata tree, but maybe that's not so useful.

projectmoon commented

2020-10-23 20:30:51 +00:00

Author

Owner

Fixed in 794bf0b854.

We need #29 to actually fix the count of existing variables though.

Fixed in https://github.com/ProjectMoon/matrix-dicebot/pull/32/commits/794bf0b85420019d2aac2627bc88a21f7190f311. We need #29 to actually fix the count of existing variables though.