mirror of
https://gitea.osmocom.org/sim-card/pysim.git
synced 2026-07-09 11:56:33 +03:00
Neels Hofmeyr
5e543f5566
EnumParam is for labels shown in the UI, not an enum to be used in python code (what the python enum module is written for). So it is semantically wrong to use enum.IntEnum. - the label should be allowed to be any string, not just valid python identifiers. For example, a label like "SUCI-in-USIM" should be possible. - we should not "leak" UI labels into the python namespace. - the value should be allowed to be any type, so that each ConfigurableParameter implementation can use whichever is its internal native type. History: this patch is the original version of EnumParam, which was modified during CR to use enum.IntEnum. However, newer ConfigurableParameters coming up don't match well: - MncLen (labels "2" and "3") - EuiccMandatoryServiceParam (values True and False) - EfUstServiceParam like SuciInUsim (labels "SUCI-in-UE" and "SUCI-in-USIM") So this brings back the original capability for any label string, and any type of value. Change-Id: I690ceccf0ec7ef7067bcaa5cec1303cdaf0f78a4