Posted by sbatman on October 17, 2009 at 8:24pm
Jump to:
| Project: | Chat Room |
| Version: | 6.x-1.0-beta9 |
| Component: | Code |
| Category: | bug report |
| Priority: | critical |
| Assigned: | sbatman |
| Status: | closed (fixed) |
Issue Summary
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.
| Attachment | Size |
|---|---|
| chatroom.module.patch | 1.36 KB |
| chatroom.install.patch | 2.32 KB |
Comments
#1
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.
#2
#3
Has these patches been applies to the latest module?
Can someone tell me how to apply these patches?
james
#4
You have just to upgrade your module to the beta8 release.
#5
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
#6
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.
#7
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.
#8
#9
Automatically closed -- issue fixed for 2 weeks with no activity.