Raising an error definitely seems a lot more intuitive.
On Thursday, January 19, 2017 at 1:17:49 PM UTC+1, Grzegorz Tężycki wrote: > > Hi everyone. > > Tim Graham told me that problem with ticet: #26911 should be discuss on > mailing list. > Details of ticket are here : https://code.djangoproject.com/ticket/26911 > > When developer set 'pattern_name' in RedirectView and this pattern is > incorrect, then method catch NoReverseMatch exception and return None. > > I think, that method get_redirect_url should raise ImproperlyConfigured > error with message "Reverse for 'xxxx' not found.", or NoReverseMatch > exception not should be catch. > > What you think about this > -- You received this message because you are subscribed to the Google Groups "Django developers (Contributions to Django itself)" group. To unsubscribe from this group and stop receiving emails from it, send an email to [email protected]. To post to this group, send email to [email protected]. Visit this group at https://groups.google.com/group/django-developers. To view this discussion on the web visit https://groups.google.com/d/msgid/django-developers/7462d34b-2f62-4393-8862-ec33ee34a311%40googlegroups.com. For more options, visit https://groups.google.com/d/optout.
