Simple problem I just can't seem to fix(mysql_query)

[nro]

I have a class called Calendar. In that class I have a method called Cal_AddEvent. I also have a class called CMS_DB(controls actions with database). The CMS_DB is installed properly so assume it works(I included the code anyway) . This is what Cal_AddEvent contains:

/// From CMS_Calendar class
function Cal_AddEvent($CalTitle,$CalEventInfo,$CalContactName,$CalContactEmail,$CalDate,$TimeHour,$TimeMinute,$TimeSecond,$TimeDuration) {

$query = "INSERT INTO `calendar_event` ( `event_id` , `type_id` , `title` , `description` , `contactname` , `contactemail` , `eventdate` , `duration` , `icon` ) 
	VALUES ('', '0', '" . $CalTitle . "', '" . $CalEventInfo . "', '" . $CalContactName . "', '" . $CalContactEmail . "', '" . $CalDate . " " . $TimeHour . ":" . $TimeMinute . ":" . $TimeSecond . "', '" . $TimeDuration . "', 'people.gif')";

$this->CMS_DB->DB_Query($query);

print '<font color="#009900">Event Added</font><br>';
}
/// From CMS_DB class
function DB_Query($query) {
	if( ! $this->DB_IsConnected() ) {
		print 'Not connected, can\'t query database';
		return false;
	}
	else
		return mysql_query($query,$this->DB_Link);
}

When I execute that portion of the code on a website it all seems to work correctly. I get the "Event Added" and everything. However when I look in the database for the newly added event, it is not there. For debugging purposes, I added a "print $query" just to make sure my query line was correct. This is what I got:

INSERT INTO calendar_event ( event_id , type_id , title , description , contactname , contactemail , eventdate , duration , icon ) VALUES ('', '0', '+Title+', '+Description+', '+Name', '+Email+', '2005-03-01 00:00:00', '00', 'people.gif')

I copied that line and used phpMyAdmin to execute it and it works properly. I am stumped. Please help!

Thanks.

[thorpe]

are event_id and type_id going into int type fields in your db?

if so, they do not need surrounding ' around them.

also, event_id is '', unless your db is setup to except nulls this will kill it.

[nro]

I have altered the code and made sure that the query was matching the database. I'm still having problems.

Here is my table setup:

CREATE TABLE `calendar_event` (
  `event_id` int(10) NOT NULL auto_increment,
  `type_id` int(10) default NULL,
  `title` varchar(30) default NULL,
  `description` text,
  `contactname` varchar(50) default NULL,
  `contactemail` varchar(50) default NULL,
  `eventdate` datetime NOT NULL default '0000-00-00 00:00:00',
  `duration` int(10) NOT NULL default '0',
  `icon` varchar(30) default NULL,
  PRIMARY KEY  (`event_id`)
) TYPE=MyISAM AUTO_INCREMENT=10 ;

Here is my new query line:

$query = "INSERT INTO `calendar_event` ( `title` , `description` , `contactname` , `contactemail` , `eventdate` , `duration` , `icon` ) VALUES ('" . $CalTitle . "', '" . $CalEventInfo . "', '" . $CalContactName . "', '" . $CalContactEmail . "', '" . $CalDate . " " . $TimeHour . ":" . $TimeMinute . ":" . $TimeSecond . "', " . $TimeDuration . ", 'people.gif')";

And this is my debug output:
INSERT INTO calendar_event ( title , description , contactname , contactemail , eventdate , duration , icon ) VALUES ('Title', 'info', 'admin', 'email', '2005-03-01 00:00:00', 0, 'people.gif')

Again, when I run it through phpMyAdmin it works perfectly fine.
😕

[TheDefender]

I get the "Event Added" and everything.

You aren't doing any type of comparison, so the "Event Added" is going to be printed whether the insert succeeds or fails.

also, event_id is '', unless your db is setup to except nulls this will kill it.

Thorpe, it won't kill it since the event_id is an auto increment value.

nro, in your DB_Query function, for debugging purposes, try adding a die to your query line:

return mysql_query($query,$this->DB_Link) or die(mysql_error()); 

It sounds like either your connection or your query is failing, since the query works when it's dumped right into your phpMyAdmin...

[Nostalgic/Historical EDIT - I got my Post Count Colt 1911 A1 Locked and Loaded]