djaenicke Posted July 12, 2013 Share Posted July 12, 2013 Hi, Everytime I try and send a reply to a customer service message I get this: Warning: preg_match() expects parameter 2 to be string, array given in /home/joovuuco/public_html/store/classes/Validate.php on line 177 Warning: Cannot modify header information - headers already sent by (output started at /home/joovuuco/public_html/store/classes/Validate.php:177) in /home/joovuuco/public_html/store/classes/Tools.php on line 141 When I go back to customer service it says that the message has sent however it has not (I've tested extensively). I have tried deleting and reuploading a new validate.php and tools.php but to no avail. This is what is written at 177 on validate.php (the bit in red is line 177) /** * Check for sender name validity * * @param string $mail_name Sender name to validate * @return boolean Validity is ok or not */ public static function isMailName($mail_name) { return preg_match(Tools::cleanNonUnicodeSupport('/^[^<>;=#{}]*$/u'), $mail_name); } And from tools.php (line 141 in red) * Redirect user to another admin page * * @param string $url Desired URL */ public static function redirectAdmin($url) { header('Location: '.$url); exit; } Any help? thank you! Link to comment Share on other sites More sharing options...
joseantgv Posted July 23, 2013 Share Posted July 23, 2013 Check this: https://github.com/PrestaShop/PrestaShop/pull/588/files Link to comment Share on other sites More sharing options...
djaenicke Posted July 23, 2013 Author Share Posted July 23, 2013 Hi Joseantgv, Do I just copy everything: $contact = new Contact((int)$ct->id_contact); + $contact = new Contact((int)$ct->id_contact, (int)$ct->id_lang); and put it into: AdminCustomerThreadsController.php? Link to comment Share on other sites More sharing options...
joseantgv Posted July 23, 2013 Share Posted July 23, 2013 Yes, you have to replace only the affected line. It's based on last PS version, so find the original line in your file. Link to comment Share on other sites More sharing options...
requ Posted April 1, 2015 Share Posted April 1, 2015 Thank You! 1 Link to comment Share on other sites More sharing options...
Recommended Posts
Create an account or sign in to comment
You need to be a member in order to leave a comment
Create an account
Sign up for a new account in our community. It's easy!
Register a new accountSign in
Already have an account? Sign in here.
Sign In Now