xyn Posted November 21, 2007 Share Posted November 21, 2007 Hey guys, I'm having a small problem with my foreach loop. Basically i have a pm system, and it sends the Carbon copies. however it only displays the FIRST letter of the username. my code is: function process_message(){ mysql_query("insert into `".$this->tbl."messenger` ( `pmid`,`pmrecipient`,`pmauthor`, `pmcc`, `pmsubject`, `pmmessage`, `pmsubmitted`, `pmfolderid`, `pmsaved`, `pmdeleted`, `pmstatus` ) values ( null, '".secure_vars($this->pm_recipient)."', '".secure_vars($_SESSION[$this->session])."', '".secure_vars($this->pm_carboncopy)."', '".secure_vars($this->pm_subject)."', '".secure_vars($this->pm_message)."', now(), '0', '0', '0', '0' )"); foreach($this->str2arr($this->pm_carboncopy) as $name){ if($this->recipient_confirm($name)){ mysql_query("insert into `".$this->tbl."messenger` (`pmid`,`pmrecipient`,`pmauthor`, `pmcc`, `pmsubject`, `pmmessage`, `pmsubmitted`, `pmfolderid`, `pmsaved`, `pmdeleted`, `pmstatus`) values( null, '".secure_vars($this->pm_carboncopy[$i])."', '".secure_vars($_SESSION[$this->session])."', '".secure_vars(trim($this->pm_carboncopy))."', '".secure_vars($this->pm_subject)."', '".secure_vars($this->pm_message)."', now(), '0', '0', '0', '0' )"); } } } The problem is within this code: foreach($this->str2arr($this->pm_carboncopy) as $name){ if($this->recipient_confirm($name)){ mysql_query("insert into `".$this->tbl."messenger` (`pmid`,`pmrecipient`,`pmauthor`, `pmcc`, `pmsubject`, `pmmessage`, `pmsubmitted`, `pmfolderid`, `pmsaved`, `pmdeleted`, `pmstatus`) values( null, '".secure_vars($this->pm_carboncopy[$i])."', '".secure_vars($_SESSION[$this->session])."', '".secure_vars(trim($this->pm_carboncopy))."', '".secure_vars($this->pm_subject)."', '".secure_vars($this->pm_message)."', now(), '0', '0', '0', '0' )"); } Quote Link to comment Share on other sites More sharing options...
xyn Posted November 21, 2007 Author Share Posted November 21, 2007 sorry, i didnt check my code before posting. it was this line: $this->pm_carboncopy[$i] shoule be $name. sorry. Quote Link to comment Share on other sites More sharing options...
Recommended Posts
Join the conversation
You can post now and register later. If you have an account, sign in now to post with your account.