#333 TaurusGUI listerer - Inconsistency with MntGrp with capital letters

[Daniel Roldan]

The listener Raise a Exception on use MntGrps with Capital letters.
"ActiveMntGrp XX is not defined"

[Daniel Roldan]

The PATCH has been sent

[Daniel Roldan]

PATCH

[Zbigniew Reszela]

See the improved patch sent to the develop mailing list.

[Carlos Pascual]

I have some issues with the approach. I think we should not use lower() here. See some motivation in https://sourceforge.net/p/sardana/tickets/498/#e443/678e

I think we should consider an alternative approach, maybe based in using case-insensitive comparisons from CaselessDict. See this snippet:

from taurus.core.util import CaselessDict

d=CaselessDict({'BAR':1})

print 'bar' in d.keys() #--> True
print 'BAR' in d.keys() #--> False
print 'bar' in d        #--> True
print 'BAR' in d        #--> True

So I think we should not use the .keys() (and not use lower()), and also make sure that we are really using a CaselessDict (i.e., that we did not cast it to a normal dict at some point).

BTW, even for regular dicts, the PEP8 also recommends using foo in mydict over foo in mydict.keys()

[Zbigniew Reszela]

The new patch was sent to the develop mailing list.

[Carlos Pascual]

Patch applied to develop and support-3.x branches

[Daniel Roldan]

Great solution, thanks!