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.
CommentFileSizeAuthor
#6 0001-1966046-by-sun-Removed-option-to-Redirect-local-mode.patch2.21 KBsun
#5 mollom.remove-cmp-redirect.5.patch1.72 KBsun
#2 mollom.remove-cmp-redirect.2.patch1.69 KBsun
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

sun’s picture

sun
Primary language German
Location Karlsruhe
CreditAttribution: sun commented

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.

sun’s picture

sun
Primary language German
Location Karlsruhe
CreditAttribution: sun commented
Assigned: Unassigned » sun
Status: Active » Needs review
FileSize
mollom.remove-cmp-redirect.2.patch1.69 KB

Attached patch removes the redirect option entirely.

Status: Needs review » Needs work

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

sun’s picture

sun
Primary language German
Location Karlsruhe
CreditAttribution: sun commented
Status: Needs work » Needs review

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

sun’s picture

sun
Primary language German
Location Karlsruhe
CreditAttribution: sun commented
FileSize
mollom.remove-cmp-redirect.5.patch1.72 KB

Re-rolled.

sun’s picture

sun
Primary language German
Location Karlsruhe
CreditAttribution: sun commented
Version: 7.x-2.x-dev » 6.x-2.x-dev
FileSize
0001-1966046-by-sun-Removed-option-to-Redirect-local-mode.patch2.21 KB
sun’s picture

sun
Primary language German
Location Karlsruhe
CreditAttribution: sun commented
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.

  • Commit 92a8fe6 on 7.x-2.x, 8.x-2.x, fbajs, actions by sun: 
    - #1966046 by sun: Removed option to 'Redirect local moderation pages to...

  • Commit 92a8fe6 on 7.x-2.x, 8.x-2.x, fbajs, actions by sun: 
    - #1966046 by sun: Removed option to 'Redirect local moderation pages to...