I noticed what I believe is a bug where the title specified on the settings page for the "Forward this page" page view isn't actually used in the page callback (actually, the form builder function).

Along with fixing that issue, I pulled calls to drupal_set_title() out of the form builder function and put them in the page callback, added a theme function for theming the page, and renamed "theme_forward_page" to "theme_forward_email".

On submitting this ticket, I also noticed that the version I have isn't even listed above... I'm working from DRUPAL-6--1-9. Perhaps that was part of problem all along. In any case, hope this is helpful.

CommentFileSizeAuthor
fix-title-issue.patch2.35 KBjustin2pin

Comments

stella’s picture

Project: Frequently Asked Questions » Forward
Version: master » 7.x-1.x-dev

Re-assigning to the correct project

seanr’s picture

Status: Needs review » Fixed

Fixed. Should be in the next release (later today).

Status: Fixed » Closed (fixed)

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