I noticed a lot of watchdog posts:

Notice: Trying to get property of non-object in _forum_access_comment_access_callback()

And tracked it down to a situation in which the comment had been deleted and cleared from the system. This particular client expects to be doing that a lot, so there's a pretty high occurrence of this sort of thing. It's not a terribly critical need, but it would make my watchdog tracking cleaner if it was handled better. Thankfully it's a really easy fix too. I'd hate to have to reapply this patch on every update - so I'm passing it up the chain. Lemme know if you have any questions.

CommentFileSizeAuthor
forum_access.patch394 bytesBastlynn
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

Bastlynn’s picture

Status: Active » Needs review

  • Commit 4f279ca on 7.x-1.x by salvis:
    #2276445 by Bastlynn: Avoid a notice when a comment pending for...
salvis’s picture

Version: 7.x-1.2 » 7.x-1.x-dev
Status: Needs review » Fixed

Thank you for the analysis and patch!

Bastlynn’s picture

And thank you for getting to the patch quickly! :)

Status: Fixed » Closed (fixed)

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