Hi there,

I think that once an announcement has expired, it should not be possible to view it by everyone but the author (and administrator).

I do not know if you will think that makes sense. I was thinking we could add a flag to the settings form so it could be decided by the administrator whether to go one way or the other.

I'm attaching a patch with the fix to the hook_access function that does the trick.

Let me know if you think the flag would be nice and I'll add it.

Thank you.
Alexis

CommentFileSizeAuthor
announcements-access-6.x-dev.patch439 bytesAlexisWilke

Comments

AlexisWilke’s picture

AlexisWilke commented

Hi Nancy,

I noticed that you applied a fix from another issue in link with hook_access()--see http://drupal.org/node/314118

So this patch may be irrelevant. And it is a good thing if it checks the full proper role access info because I use the same module.

Now, this patch may be slightly different from the other. I'll look into the other patch later.

Thank you.
Alexis

AlexisWilke’s picture

AlexisWilke commented
Status: Needs review » Closed (duplicate)

Nancy,

Okay, this is a duplicate of bug

http://drupal.org/node/314118

So we can close this entry. The code from the previous bug report is cleaner.

Thank you.
Alexis