I think this issue is similar to this one: http://drupal.org/node/327708

Attached to this issue there is two patch:
- the first one fixes the problem in the chatroom_archive_old_msgs() function
- the second one fixes the .install file (chatroom_archive_old_msgs table missing)

Regards.

CommentFileSizeAuthor
chatroom.install.patch2.32 KBantoniopironti
chatroom.module.patch1.36 KBantoniopironti

Comments

Anonymous’s picture

Anonymous (not verified) commented
Status: Patch (to be ported) » Postponed (maintainer needs more info)

thanks for the patches, i'd already made a fix in the branch but not rolled a release tag.

i've done that now, so please test this against beta8.

Anonymous’s picture

Anonymous (not verified) commented
Status: Postponed (maintainer needs more info) » Fixed
jamesialford’s picture

jamesialford commented

Has these patches been applies to the latest module?

Can someone tell me how to apply these patches?

james

antoniopironti’s picture

antoniopironti commented

You have just to upgrade your module to the beta8 release.

jamesialford’s picture

jamesialford commented

I have uninstalled the module, deleted the folder, downloaded Bata 8, added to sites/all/module, enabled the chatroom module, set the permissions, update.php.

After some research I found out that evertime I have "Automatically archive old messages" checked it will give me this "Fatal error: Call to undefined function chatroom_chat_get_latest_msgs() in /mysite/sites/all/modules/chatroom/chatroom.module on line 615

Do you know why it would give me this?

All I want is the old chat to go away and not stay there forever. Do you how to complete this?

Thanks for your help
James

antoniopironti’s picture

antoniopironti commented
Status: Fixed » Needs review

That function does not exist anymore. The patches attached here fix that error.

I think that the status of this issue must be changed to needs review.

Anonymous’s picture

Anonymous (not verified) commented
Version: 6.x-1.0-beta7 » 6.x-1.0-beta9
Status: Needs review » Postponed (maintainer needs more info)

i've committed a variation of the patches that gets rid of the old crufty message archive code.

please reopen if this bug still exists in 6.x-1.0-beta9.

Anonymous’s picture

Anonymous (not verified) commented
Status: Postponed (maintainer needs more info) » Fixed

Status: Fixed » Closed (fixed)

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