The ability to disable the javascript for anonymous users so they cannot modify the content of a specific homebox. Odd request I know. Patch in next post.

Support from Acquia helps fund testing for Drupal Acquia logo

Comments

mikebell_’s picture

Status: Active » Needs review
FileSize
6.52 KB

Patch!

drumm’s picture

Version: 7.x-2.0-beta6 » 7.x-2.x-dev
Issue summary: View changes
Status: Needs review » Needs work

Amazingly, this patch applies nearly cleanly, except for the two whitespace-related chunks.

There isn't a way to store a customized homebox layout at all for anonymous users. Rather than being a setting, I think we should always disable it.

mikebell_’s picture

Wow this took a bit of remembering.

I've attached a patch that checks to see if the user is logged in and then attaches the correct js and css.

mikebell_’s picture

Status: Needs work » Needs review

setting status

drumm’s picture

Status: Needs review » Fixed

Committed with fixing some merge conflicts and pulling the common code out of the if statement.

  • drumm committed 8479b32 on 7.x-2.x
    Issue #1473316: Merge branch '7.x-2.x' into 1473316-anonymous
    
  • drumm committed bc8acb1 on 7.x-2.x
    Issue #1473316 by mikebell_: Disable homebox functionality for Anonymous...
  • drumm committed feee6f9 on 7.x-2.x authored by mikebell_
    Issue #1473316 by mikebell_: Disable homebox functionality for Anonymous...

Status: Fixed » Closed (fixed)

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