i have reviewed the current cvs version and found some errors:

  • you forgot to delete $title .= variable_get("feedback_pageheader", "");

    (are you doing the patch by hand?)
    so you get the title printed twice
  • $title .= "<br><p>" . variable_get("feedback_pageheader", "") . "</p><br>";

    the <br><p> is not needed. this breaks some templates
    should be $title .= variable_get("feedback_pageheader", "");
    • cu

      Tobias Maier

CommentFileSizeAuthor
feedback.module_2.patch776 bytesTobias Maier

Comments

kbahey’s picture

Assigned: Unassigned » kbahey
Status: Active » Fixed

Yes, i patched by hand because the patch did not want to apply.

Anyway, thanks for informing me. It is now fixed.

Anonymous’s picture

Anonymous’s picture

Anonymous’s picture

Anonymous’s picture

Anonymous’s picture

Status: Fixed » Closed (fixed)