As discovered in #259488: Webform support the menu path information for reporting contact e-mails is incorrect and leads to an access denied.

CommentFileSizeAuthor
#1 486798.patch751 bytesdave reid

Comments

dave reid’s picture

dave reid
he/him
Primary language English
Location Nebraska USA
commented
Status: Active » Needs review
StatusFileSize
new 486798.patch751 bytes

Attached patch fixes the bug by changing 'access arguments' => TRUE to 'access callback' => TRUE.

dave reid’s picture

dave reid
he/him
Primary language English
Location Nebraska USA
commented

I suppose it would be nice to add a test for this link once the e-mail testing capability is backported to SimpleTest 6.x-2.x. I don't think this can hold it up though.

dries’s picture

dries commented
Status: Needs review » Fixed

Committed to CVS HEAD. Thanks Dave.

Maybe we should create a place holder issue for writing e-mail tests when the changes are backported? I think it is pretty important to have tests for those.

Status: Fixed » Closed (fixed)

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