Needs work
Project:
Disable breadcrumbs
Version:
7.x-1.3
Component:
Code
Priority:
Normal
Category:
Bug report
Assigned:
Unassigned
Reporter:
Created:
15 Dec 2011 at 18:20 UTC
Updated:
17 May 2017 at 09:57 UTC
Jump to comment: Most recent, Most recent file
Comments
Comment #1
damiankloip commentedThis turned out to be a permissions problem, you need the 'administer nodes' permission to do node operations so this access check is now in place. Committed to dev branch.
Comment #3
nonzero commentedTo work around the error, I changed line 339 to:
There's probably a more semantically correct approach that looks at permissions instead.
Comment #4
nikolay shapovalov commentedWorks for me. Thanks a lot.
Comment #5
elvismdev commentedWorks for me too, many thanks!
Comment #6
rooby commentedAre you guys saying that you still need this patch with the latest dev version? Or is this just for the 1.3 version? Because a fix was apparently commited to the dev version.
Comment #7
rooby commentedAlso, instead of array_key_exists() it is preferrable to use isset(), like this:
if (isset($form['admin']['nodes']['#option'])) {This isset also makes the first one redundant because for $form['admin']['nodes']['#option'] to exist $form['admin'] must also exist.
Comment #8
ailgm commentedChanging it to this worked for me:
if (isset($form['admin']['nodes']['#options'])) {The singular word "option" didn't.
Comment #9
sheldonkreger commentedThis is still and issue and none of the solutions posted are working for me. I am using 7.x-1.3
Comment #10
sheldonkreger commentedComment #11
sheldonkreger commentedAlso not fixed in dev branch.
Comment #12
narquam commentedI see this is still an issue. This seems like a very minor change, I'm surprised it still isn't in a release.
Comment #13
bart lambert commentedI'm also still struggeling with this problem!
Comment #14
fonant commentedFix in #8 works for me.