sorry. it breaks backward compatibility. Massimo
On Dec 19, 4:53 am, Iceberg <[email protected]> wrote: > Hi Massimo, > > According to document, source code and test, currently IS_IN_SET > accepts a list OR A SET as its first parameter "theset", and an > optional list as labels. Actually, something is inconsistent here. How > can an order-less set works with an ordered label? It can't. > > I am writing to suggest an improvement to IS_IN_SET, to better deal > with order-less set. > > Let's say I have following field in model.py of a online survey > application: > > # I like to group the order-less values and their labels in a dict > DO_YOU_LIKE_SUSHI = { > 'yes': 'Like it? I love it!', > 'no': 'Come on, Sushi sucks!', > 'unknown': 'What is Sushi?' } > > # Currently I have to use it in this style. It is too verbose to > mention DO_YOU_LIKE_SUSHI three times. > SQLField('do_you_like_sushi','text',requires=IS_IN_SET > (DO_YOU_LIKE_SUSHI, DO_YOU_LIKE_SUSHI.values())) > > # So I suggest this modification in __init__() of class IS_IN_SET, in > validators.py: > if isinstance(theset,dict): # Iceberg's hack > self.theset=theset > self.labels=theset.values() > else: > self.theset = [str(item) for item in theset] > self.labels = labels > > # Then, I can rewrite the model in this style. Notice the use of > anonymous dict. I think it is clearer. > SQLField('do_you_like_sushi','text',requires=IS_IN_SET( > {'yes': 'Like it? I love it!', > 'no': 'Come on, Sushi sucks!', > 'unknown': 'What is Sushi?' } > )) > > # In case you want to ask, why a hack inside validators.py is the only > way to do this? I tried to define a subclass of IS_IN_SET in my own > model.py, it almost works, but fails when a record is going to be > saved in session (due to some boring pickling problem of __builtin__ > naming space issue). > > Regards, > Iceberg --~--~---------~--~----~------------~-------~--~----~ You received this message because you are subscribed to the Google Groups "web2py Web Framework" group. To post to this group, send email to [email protected] To unsubscribe from this group, send email to [email protected] For more options, visit this group at http://groups.google.com/group/web2py?hl=en -~----------~----~----~----~------~----~------~--~---
