[Note that I had problems on update (http://drupal.org/node/651988) so that may be affecting this.]

When I click the "Archive this chat" button on an existing chat (imported from v1 of the module), it sits and thinks about it for a while - then nothing happens (IE8) or the window freezes with incredibly high CPU activity (FF3.5.5).

In the log, I get hundreds of PHP errors until I kill the FF window:
array_key_exists() [<a href='function.array-key-exists'>function.array-key-exists</a>]: The second argument should be either an array or an object in /homepages/31/d292789962/htdocs/parentfocus/sites/all/modules/chatroom/chatroom.module on line 1538.

Browser: IE8, FF3.5.5
Drupal: 6.14
PHP: 5.2.11
MySQL: 5.0.81

Comments

Anonymous’s picture

Anonymous (not verified) CreditAttribution: Anonymous commented

i've committed a fix for the migration of banned user data, which had a bug that this issue has helped me find:

http://drupal.org/cvs?commit=301466

that commit also contained a fix to check that $node->chat->banned_list is an array, which should get rid of the warnings you are seeing.

however, the only way i can see that variable not being an array is if the chat fails to load entirely in chatroom_chat_load(), so i'm guessing there's another issue here.

i suspect the warnings will accumulate in this case for a migrated chat regardless of whether its archived or not.

adam_b’s picture

adam_b CreditAttribution: adam_b commented
Version: 6.x-2.7 » 6.x-2.x-dev

I've upgraded to the dev version, but no joy.
- am I right in thinking that commit is included in the 14 Dec dev version?
- if so, will it only work for new upgrades from v1 to v2 (I can't re-upgrade)?
- if so, is there any way for me to upgrade my problem migrated chats?