Ezio Melotti added the comment:
The attached patch fixes the issue by changing action="#" to
action="name_of_current_page".
This loses the information about the query string (e.g. in case of search).
Since Roundup uses different pages for different issues, logging in from an
issue page brings you the same page, however this doesn't work for pages that
depend on the query string (e.g. searches).
Adding the query string in the action would be ineffective since this form uses
POST. If we really wanted to keep them we could try to convert the query
string to hidden fields in the form, but then I'm not sure Roundup would
recognize them if they are sent via POST.
_______________________________________________________
PSF Meta Tracker <metatrac...@psf.upfronthosting.co.za>
<http://psf.upfronthosting.co.za/roundup/meta/issue448>
_______________________________________________________
diff --git a/html/page.html b/html/page.html
--- a/html/page.html
+++ b/html/page.html
@@ -180,7 +180,7 @@
<li tal:condition="python:request.user.username=='anonymous'">
<strong i18n:translate="">User</strong>
- <form method="post" action="#">
+ <form method="post" tal:attributes="action
string:${request/env/PATH_INFO}">
<ul class="level-three">
<li>
<tal:span
i18n:translate="">Login</tal:span> (OpenID possible)<br />
_______________________________________________
Tracker-discuss mailing list
Tracker-discuss@python.org
https://mail.python.org/mailman/listinfo/tracker-discuss
