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.

CommentFileSizeAuthor
#3 homebox-removejsforanonusers-1473316-3.patch3.75 KBmikebell_
#1 homebox-removejsforanonusers-1473316-1.patch6.52 KBmikebell_

Comments

mikebell_’s picture

mikebell_ commented
Status: Active » Needs review
StatusFileSize
new homebox-removejsforanonusers-1473316-1.patch6.52 KB

Patch!

drumm’s picture

drumm
he/him
Location NY, US
commented
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

mikebell_ commented
StatusFileSize
new homebox-removejsforanonusers-1473316-3.patch3.75 KB

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

mikebell_ commented
Status: Needs work » Needs review

setting status

drumm’s picture

drumm
he/him
Location NY, US
commented
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.