Here's the code from before:
``` if values['group'] and values['group'] != "" and !(values['group'] =~ /^[a-zA-Z][a-zA-Z0-9._:-]+$/)```
This was doing something special for empty value, which your code is not doing. And maybe you're right. But that's not the same as before.