Edit forum page drupal_set_message warnings after installation

annez - April 15, 2008 - 15:46
Project:Forum Access
Version:6.x-1.x-dev
Component:Code
Category:bug report
Priority:normal
Assigned:Unassigned
Status:closed
Description

After installing D6 dev version of Forum access, I get these warnings on the Edit forum page for my forums:

* warning: Illegal offset type in isset or empty in /Applications/MAMP/htdocs/drupal/includes/bootstrap.inc on line 812.
* warning: Illegal offset type in /Applications/MAMP/htdocs/drupal/includes/bootstrap.inc on line 813.
* warning: Illegal offset type in /Applications/MAMP/htdocs/drupal/includes/bootstrap.inc on line 817.

It seems drupal_set_message is getting called with an incorrect set of parameters -- the replacement string for !Forum_access is being called as the second parameter to drupal_set_message rather than in a way that it would get substituted into the message string.

In my patch, I moved the translation call t() so it passes the right parameters to drupal_set_message and this means the entire message would get translated if necessary -- which seems like the right thing.

AttachmentSize
editForumMessage.patch993 bytes

#1

salvis - April 16, 2008 - 10:51
Status:needs review» fixed

Fixed, thank you!

#2

Anonymous (not verified) - April 30, 2008 - 10:52
Status:fixed» closed

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

 
 

Drupal is a registered trademark of Dries Buytaert.