Fatal error: Call to undefined function chatroom_chat_get_from_id()

ozchris - November 1, 2008 - 08:43
Project:Chat Room
Version:6.x-2.x-dev
Component:Code
Category:bug report
Priority:critical
Assigned:justinrandell
Status:fixed
Description

got this message when trying to delete a chat.

#1

netgenius - November 9, 2008 - 20:34

Me too - checked source to see if it was a simple mis-spelling - seems like the function is just not there .. is there a file missing from the distribution maybe?

#2

adshill - December 12, 2008 - 03:16

I'm also getting this.

#3

adam42 - December 12, 2008 - 12:43

I get the same error message when trying to access a chat which I created my self... (?)

#4

gording - January 3, 2009 - 15:40

I get the same Message, when trying to delete a Chatroom.

#5

janusman - January 9, 2009 - 22:46

same. subscribing

#6

janusman - January 9, 2009 - 23:30
Priority:normal» critical
Status:active» needs review

Decided to do something about it =)

Please test attached patch... NOTE that you need to reinstall the module in order for it to work completely (as there was also a missing DB table!)

AttachmentSize
328811-6_D6--1-0-BETA2.patch 4.01 KB

#7

adetoyan - January 16, 2009 - 14:46

Please could you upload your patched version of this module I'd like to test it out and I don't understand the concept of patching and all the stuff I'm reading about it seems like greek to me
Thanks

#8

britne - January 17, 2009 - 22:47

subscribing

#9

Mekk - February 23, 2009 - 10:34

I patched it but after pressing the delete button, the chatroom is still there. After deleting the chat, i find myself on the front page and the chat is still available. can anybody help?
Thanks.

#10

migas - March 2, 2009 - 11:21

The patch looks strange to me. Usually you have one patch a file. This patch contains a patch for 2 files: chatroom.install ans chatroom.module. With patch from Cygwin you have to enter: "patch [origfile] [patchfile].

So please guys make this chatroom work. Mainly its a good module. I intended to use it on a bigger site, but with this critical bug, this isn't possible.

#11

dani matielo - March 26, 2009 - 10:35

Would also be interested in seeing it fixed. Thank you.

#12

Paul Kishimoto - March 27, 2009 - 17:50
Title: Fatal error: Call to undefined function chatroom_chat_get_from_id() » Fatal error: Call to undefined function chatroom_chat_get_from_id()
Status:needs review» needs work

@janusman: the patch is wrong for two reasons:

  1. chatroom_load_chat (chatroom.module, l.1025) replaced the older chatroom_chat_get_by_id. Instead of recreating the old function, calls to the latter function need to be updated to use the new one.
  2. Older code did archiving by transferring messages between two tables, {chatroom_msg} and {chatroom_msg_archive}. 6.x-1.0-beta2 has a boolean 'archived' column in {chatroom_msg} (chatroom.install, l.159).

A quick grep shows there are still three references to chatroom_load_chat. Maybe you can create a new patch according to #1, and start a new issue for #2.

@migas: it is perfectly normal to modify multiple files in a single patch.

#13

Paul Kishimoto - March 27, 2009 - 18:16

The issue regarding archiving is #327708: Archive feature is broken. This one should focus only on the calls to chatroom_chat_get_from_id.

#14

neofutur - April 5, 2009 - 23:36

same here, subscribing

#15

jefflane - April 8, 2009 - 17:02

This happens when trying to delete the chat room, for me.

Fatal error: Call to undefined function chatroom_chat_get_from_id() in /sites/all/modules/chatroom/chatroom.forms.inc on line 452

#16

juanito - April 26, 2009 - 16:32

Something new about this, cause I have the same problem!!

#17

DickSwart - May 21, 2009 - 12:40

Have anyone found a solution, or am anyone looking into this issue. I need this module, Please help us.

#18

jsimonis - May 28, 2009 - 13:35

Same thing is happening on our chat.

#19

dork-maneuver - May 31, 2009 - 22:49

subscribing

#20

blure007 - June 23, 2009 - 19:56

subscribing

#21

Donngal - June 30, 2009 - 10:03

subscribing

#22

NaPa - July 1, 2009 - 08:54

same for me

#23

diego.paroni - July 8, 2009 - 16:20

This module is unusable...
I surrender.

#24

yountod - July 9, 2009 - 18:09

Pardon my ignorance, how do I subscribe to this? I read a post about the subscribe link disappearing... I don't see one *anywhere*

#25

RWWood - July 20, 2009 - 12:09

subscribe

#26

npobase - July 29, 2009 - 06:01

subscribe

#27

chadhester - August 26, 2009 - 16:58

subscribe

#28

Bastlynn - August 27, 2009 - 17:47

Is this module still being actively supported?

#29

Gökhan DAĞ - September 6, 2009 - 08:32

:((((((

#30

justinrandell - September 12, 2009 - 13:48
Assigned to:Anonymous» justinrandell
Status:needs work» needs review

i've committed some fixes that should get rid of the WSOD. still lots of broken code in the module in general, but could you try checking out the latest DRUPAL-6--1 version and seeing if it is fixed?

#31

justinrandell - November 3, 2009 - 13:55
Version:6.x-1.0-beta2» 6.x-2.x-dev
Status:needs review» fixed
 
 

Drupal is a registered trademark of Dries Buytaert.