I keep on getting ths PHP notice:
Notice: Undefined property: stdClass::$name in fillpdf_merge_pdf() (line 431 /modules/fillpdf/fillpdf.module)
Thanks in advance.

CommentFileSizeAuthor
#1 fillpdf_watchdog.patch558 bytesliam morland

Comments

liam morland’s picture

liam morland
Primary language English
Location Ontario, CA 🇨🇦
commented
Status: Active » Needs review
StatusFileSize
new fillpdf_watchdog.patch558 bytes

This happens with unauthenticated users, since they do not have a user name.

The attached patch solves the problem by removing the username from the watchdog message. It is not needed since the current user is automatically part of the log entry.

wizonesolutions’s picture

wizonesolutions
he/him
Primary language English
Location Norway
commented
Version: 7.x-1.4 » 6.x-1.x-dev
Status: Needs review » Patch (to be ported)

Good catch. Committed in 29898a0.

  • Commit 29898a0 on 7.x-1.x, 7.x-2.x, 7.x-2.x-tests1, 7.x-1.x-ubercartbackporttest authored by Liam Morland, committed by wizonesolutions: 
    Issue #1442924: Remove username from watchdog notice.

wizonesolutions’s picture

wizonesolutions
he/him
Primary language English
Location Norway
commented
Issue summary: View changes
Status: Patch (to be ported) » Fixed

Drupal 8 is around the corner. I'm not going to leave issues that are waiting on a D6 backport open. I don't have time to maintain it anymore. New issues can be opened by people interested in contributing patches.

Status: Fixed » Closed (fixed)

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