Hello,

I would like to ask your help.

I installed the Simplenews 7.x-1.0-beta2 version.

If I send the newsletter or test after the system sends back an error message:

---
Notice: Undefined variable: from SimplenewsSourceNode->getHeaders() (/home/****/sites/all/modules/simplenews/includes/simplenews.source.inc 461 line).
Notice: Undefined variable: from SimplenewsSourceNode->getHeaders() (/home/****/sites/all/modules/simplenews/includes/simplenews.source.inc 462 line).
---

The newsmail is arrived correct.

What is the problem this error?

I think this problem about "receipt".

How can I fix the problem?

I am waiting your idea.

THX

KALMI

CommentFileSizeAuthor
#1 receipt_bug_tests_only.patch1.04 KBberdir
#1 fix_receipt_bug.patch1.66 KBberdir

Comments

berdir’s picture

berdir
Primary language German
Location Switzerland
commented
Status: Active » Needs review
StatusFileSize
new fix_receipt_bug.patch1.66 KB
new receipt_bug_tests_only.patch1.04 KB

Nice catch, looks like nobody has been trying to use the receipt confirmation since december ;)

The attached patch fixes the bug and also adds test coverage for this feature. The tests only patch should fail.

berdir’s picture

berdir
Primary language German
Location Switzerland
commented
Version: 7.x-1.0-beta2 » 7.x-1.x-dev
berdir’s picture

berdir
Primary language German
Location Switzerland
commented

Edit: Wrong issue.

berdir’s picture

berdir
Primary language German
Location Switzerland
commented
Status: Needs review » Fixed

Commited.

Status: Fixed » Closed (fixed)

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