slug validator added to GroupForm

[Onitopheles]

Hello, I was testing your demo tendenci groups, and I found an issues with the "URL PATH", when you try to use '/' in the text field and it displays an "Oops admin error".

so, I try to make a fix for that using forms.py in user_groups app folder adding these lines of code

from tendenci.apps.base.fields import SlugField to import tendenci SlugField from tendenci

and added this line to class GroupForm(TendenciBaseForm)

slug = forms.SlugField(label=_("URL Path"))

that works for me but, I'd like to know if you have some advices to fix it properly.

[jennyq]

Hi @GfBot,
I cannot replicate the issue https://demo.tendenci.com/groups/add/. The demo site gets recreated every 30 minutes, it's likely the error was due to the site was being removed and recreated. Have you tried again?

Although the slug field is not specified in the GroupForm, it is specified as SlugField for the Group class in the models.py. Thus it does use the validators for tendenci.apps.base.fields.SlugField. But I see you changed it to use django's forms.SlugField which I think is better. Thanks!

[Onitopheles]

@jennyq Hello, thanks for the feedback, if you want to reproduce this error you have to create a group, and then when you're in the "URL Path" field, you have to use a slash for that for ex: "testing-group/error-slug" or something like this using a slash and that will reproduce that error.