Ticket #1382 (closed Bug: fixed)

Opened 9 years ago

Last modified 3 years ago

Logging in and replying to a discussion at the same time is broken

Reported by: limi Owned by:
Priority: critical Milestone: 2.1
Component: Templates/CSS Version:
Severity: Keywords:
Cc:

Description

When upgrading from Plone 1.0.1 to Plone 1.0.3, the ability for anonymous users to comment disappears.

This will need to be put back until we have a solution that allows you to:

a) allow users to enter their name without signing up for an account when commenting (this should be togglable on site-wide basis by site admin)

b) allow users to register as part of the commenting

Change History

comment:1 Changed 8 years ago by limi

Fix this in 1.0.6.

comment:2 Changed 8 years ago by tiran

Still a bug - or a feature? :) Rescheduling for 2.1

comment:3 Changed 7 years ago by dn

When I enable "Reply to item" for Anonymous, I am able to comment. The code is still in place and working. However, there's a problem with logging in and replying at the same time: While it does log you in, your comment is not submitted and you are presented an empty discussion reply form. I've changed this issue's title to reflect this.

comment:4 Changed 7 years ago by briang

Redirection after login has been changed in logged_in.py.

Original behavior is to strip any query string passed before redirecting. Now, if query string exists (such as when coming from discussion_reply_form.pt), don't strip it off.

Details: Problem was that "logged_in.py" strips out any query strings passed to it before it redirects. So, after filling in login info, subject and comment the system would successfully login, but then not pass subject and comment to discussion_reply_form.

Unit Tests:

Tested manually and existing unittests work, but this really needs its own unittest.

fyi for someone tackling this: There is an existing unittest for discussion replies in CMFPlone/tests/testContentTypeScripts.py that could serve as a start.

comment:5 Changed 7 years ago by briang

Resolved. (see note about unit test below)

comment:6 Changed 7 years ago by hannosch

  • Status changed from new to closed
  • Resolution set to fixed

Marking this resolved as it is really.

comment:7 Changed 3 years ago by hannosch

  • Component changed from Visual and templates to Templates/CSS
Note: See TracTickets for help on using tickets.