Devel cannot run image_access() correctly as another user (or any content-type module that implements _access incorrectly), so node access information is wrong about Image nodes.
Fix is to use the $account param, and use that if? provided. User in this following thread said $account is required in D6, so maybe use it no matter what instead of using $user or $account as i did?
Here is thread that led to this conclusion:
http://drupal.org/node/309765#comment-1017973
Thanks,
Arian
| Comment | File | Size | Author |
|---|---|---|---|
| #5 | image.310693.5.patch | 591 bytes | salvis |
| #3 | image-HEAD.access_.patch | 1.03 KB | sun |
| #2 | image-HEAD.access.patch | 1.08 KB | sun |
Comments
Comment #1
salvisPlease run Coder on Image — it will reveal this as well as a number of other obvious D6 porting issues that need to be fixed before it makes sense to start testing.
Comment #2
sunPlease test attached patch.
Comment #3
sunRevised patch after reading the docs properly.
Comment #4
sunCommitted.
Comment #5
salvisSince you're accessing $account->uid, $account must not be NULL.
EDIT: patch relative to current HEAD.
Comment #6
sunI do not think so - the API docs do not contain this enhanced validation either, so $account is always passed in. All I mistakenly left over from my first patch was the wrong default, i.e. "= NULL" in the signature.
Comment #7
salvisAh, right, I didn't check the API docs, only looked at your code.
The docs say that $account is not optional, so just remove the
= NULLand that's it.Comment #8
sunRemoved.