search

ProjectMoon / matrix-dicebot

This repository is mirrored from: https://git.agnos.is/projectmoon/matrix-dicebot
https://git.agnos.is/projectmoon/matrix-dicebot
Other
4 stars 1 forks source link

Do not expose variable_count to the user #27

Closed ProjectMoon closed 4 years ago

ProjectMoon commented 4 years ago

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 4 years ago

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 4 years ago

Fixed in https://github.com/ProjectMoon/matrix-dicebot/pull/32/commits/794bf0b85420019d2aac2627bc88a21f7190f311.

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