LoginView not using success_url about django-bootstrap-modal-forms HOT 4 CLOSED

Using this SO answer I was able to work around the issue by including a hidden form element with name="next" to provide the redirect url to the LoginView:

{% load widget_tweaks %}

<form method="post" action="">
  {% csrf_token %}
  <input type="hidden" name="next" value="{% url 'appname:index' %}">
  <div class="modal-header">
    <h3 class="modal-title">Log in</h3>
   [...]

from django-bootstrap-modal-forms.

The redirection really happens based on the LOGIN_REDIRECT_URL = '/' in the settings.py and success_url doesn't have an effect. Good catch. And at the same time good hack with the hidden input field and name="next". Regarding the successURL from .js, it was deprecated during implementation of Django messages.

I think that hidden input field and name="next" could provide some extra functionality for the package. Maybe I give it a shoot in the future.

from django-bootstrap-modal-forms.

To give my background story here: the reason I ran into this issue is that I want a redirect specifically to the index of the app the user was logging in at, so not the root of the project, meaning a global setting won't help me.

On a more general note I would expect it to default to the current page the user clicked the Log in button, like how most websites behave. Although most of them use an intermediate page, like Github here or Amazon for example, they then redirect to the previous page, not the root index.

Checking the Django docs on LoginView it seems that the class property extra_context is targeted to forward data to the template:

extra_context: A dictionary of context data that will be added to the default context data passed to the template.

And indeed providing success_url via that dict in the LoginView

class CustomLoginView(LoginAjaxMixin, SuccessMessageMixin, LoginView):
    authentication_form = CustomAuthenticationForm
    template_name = 'myappname/login.html'
    success_message = 'Success: You were successfully logged in.'
    extra_context = dict(success_url=reverse_lazy('myappname:index'))

and using success_url manually as the hidden form element's value

<input type="hidden" name="next" value="{{ success_url }}">

Works fine, so at least in that way the redirection value can be statically defined in the view again.

from django-bootstrap-modal-forms.

@bastiaan85 That's great way for custom success_url. I'll update the README.rst. Thanks for great contribution.

from django-bootstrap-modal-forms.