Problem

  • Many users are getting confused by this option, and misunderstand it to be a "global toggle" for enabling CMP integration.

Proposed solution

  1. Remove the option without replacement.
Files: 
CommentFileSizeAuthor
#6 0001-1966046-by-sun-Removed-option-to-Redirect-local-mode.patch2.21 KBsun
PASSED: [[SimpleTest]]: [MySQL] 6,153 pass(es).
[ View ]
#5 mollom.remove-cmp-redirect.5.patch1.72 KBsun
PASSED: [[SimpleTest]]: [MySQL] 6,827 pass(es).
[ View ]
#2 mollom.remove-cmp-redirect.2.patch1.69 KBsun
PASSED: [[SimpleTest]]: [MySQL] 6,800 pass(es).
[ View ]

Comments

Alternatively, we could replace it with an help text + link on the corresponding pages — if the respective forms are protected by Mollom; e.g.:

- on /admin/content/comments/approve if any comment form is protected
- on /admin/content if any node form is protected
- on /admin/people if the user registration form is protected

But that will also require some non-trivial code... I'm really not sure whether this is worth the effort.

Assigned:Unassigned» sun
Status:Active» Needs review
StatusFileSize
new1.69 KB
PASSED: [[SimpleTest]]: [MySQL] 6,800 pass(es).
[ View ]

Attached patch removes the redirect option entirely.

Status:Needs review» Needs work

The last submitted patch, mollom.remove-cmp-redirect.2.patch, failed testing.

Status:Needs work» Needs review

#2: mollom.remove-cmp-redirect.2.patch queued for re-testing.

StatusFileSize
new1.72 KB
PASSED: [[SimpleTest]]: [MySQL] 6,827 pass(es).
[ View ]

Re-rolled.

Version:7.x-2.x-dev» 6.x-2.x-dev
StatusFileSize
new2.21 KB
PASSED: [[SimpleTest]]: [MySQL] 6,153 pass(es).
[ View ]

Version:6.x-2.x-dev» 7.x-2.x-dev
Status:Needs review» Fixed

Committed to all branches.

Status:Fixed» Closed (fixed)

Automatically closed - issue fixed for 2 weeks with no activity.