Fatal error when displaying send message to author links on nodes [#832708]

Hey Berdir and all,

I've been the testing process on the latest 7.x-dev snapshot. I noticed that when a user who has a new private message logs in there is a White Screen of Death. Checking the Apache error logs, here is the message I'm getting:

PHP Fatal error: Cannot use string offset as an array in /home/sandboxes/sandbox10/public_html/sites/all/modules/contrib/privatemsg/privatemsg.module on line 1956

I tried unchecking "Inform the user about new messages on login," but that didn't fix the error.

Navigating to the home page or a node page causes the error (even after initial login). But navigating to /user or an admin page still works fine.

Thoughts?

--Ben

Comment	File	Size	Author
#9	author_links_with_tests7.patch	5.86 KB	berdir
#8	author_links_with_tests6.patch	5.83 KB	berdir
#7	author_links_with_tests-D6.patch	5.83 KB	berdir
#4	author_links_with_tests.patch	7.73 KB	berdir

Comments

Comment #1

BenK commented 20 June 2010 at 05:58

Title:

Fatal Error When a User Logs In Who Has a New Message

» D7: Fatal Error When a User Logs In Who Has a New Message

Comment #2

BenK commented 20 June 2010 at 06:04

Note that the user can navigate to /messages even though he can't visit the homepage or a node page.

I also tried deleting the message thread altogether, but that didn't fix the White Screen of Death.

--Ben

Comment #3

BenK commented 20 June 2010 at 07:38

Title:

D7: Fatal Error When a User Logs In Who Has a New Message

» Fatal Error When a User Logs In Who Has a New Message

Comment #4

berdir

German

Switzerland

commented 20 June 2010 at 08:10

Title:

Fatal Error When a User Logs In Who Has a New Message

» Fatal error when displaying send message to author links on nodes

Status	File	Size
new	author_links_with_tests.patch	7.73 KB

See title, this happens when you have the setting to display privatemsg links enabled because Privatemsg still assumes that node titles are fields.

The patch fixes that and also two separate bugs for blocks and comes with tests (Took them together so that I can write a single test class for both).