[-] IN : update_customer_default_group returning false

This commit is contained in:
gRoussac committed 2013-11-02 21:51:13 +01:00
1 parent a5bb4f0323
commit 8f72282594
1 file changed
+19 -15
@@ -29,16 +29,19 @@ function update_customer_default_group()
$filename = _PS_ROOT_DIR_.'/config/defines.inc.php'; $filename = _PS_ROOT_DIR_.'/config/defines.inc.php';
$filename_old = str_replace('.inc.', '.old.', $filename); $filename_old = str_replace('.inc.', '.old.', $filename);
copy($filename, $filename_old); copy($filename, $filename_old);
@chmod($filename_old, 0664); chmod($filename_old, 0664);
$content = file_get_contents($filename); $content = file_get_contents($filename);
$pattern = "/define\('_PS_DEFAULT_CUSTOMER_GROUP_', (\d)\);/"; $pattern = "/define\('_PS_DEFAULT_CUSTOMER_GROUP_', (\d)\);/";
preg_match($pattern, $content, $matches); preg_match($pattern, $content, $matches);
if (!defined('_PS_DEFAULT_CUSTOMER_GROUP_')) if (!defined('_PS_DEFAULT_CUSTOMER_GROUP_'))
define('_PS_DEFAULT_CUSTOMER_GROUP_', ((isset($matches[1]) AND is_numeric($matches[1]))? (int)$matches[1] : 3)); define('_PS_DEFAULT_CUSTOMER_GROUP_', ((isset($matches[1]) AND is_numeric($matches[1]))? $matches[1] : 3));
$ps_customer_group = DB::getInstance()->getValue('SELECT value FROM `'._DB_PREFIX_.'configuration` WHERE name LIKE "PS_CUSTOMER_GROUP"', false); $ps_customer_group = DB::getInstance()->getValue('SELECT value FROM `'._DB_PREFIX_.'configuration` WHERE name LIKE "PS_CUSTOMER_GROUP"', false);
$str_old = 'define(\'_PS_DEFAULT_CUSTOMER_GROUP_\', '.(int)_PS_DEFAULT_CUSTOMER_GROUP_.');'; if ($ps_customer_group)
$str_new = 'define(\'_PS_DEFAULT_CUSTOMER_GROUP_\', '.(int)$ps_customer_group.');'; {
$content = str_replace($str_old, $str_new, $content); $str_old = 'define(\'_PS_DEFAULT_CUSTOMER_GROUP_\', '.(int)_PS_DEFAULT_CUSTOMER_GROUP_.');';
$str_new = 'define(\'_PS_DEFAULT_CUSTOMER_GROUP_\', '.(int)$ps_customer_group.');';
$content = str_replace($str_old, $str_new, $content);
}
$carriers = Db::getInstance()->executeS(' $carriers = Db::getInstance()->executeS('
SELECT `id_carrier` SELECT `id_carrier`
@@ -50,21 +53,22 @@ function update_customer_default_group()
FROM `'._DB_PREFIX_.'configuration` FROM `'._DB_PREFIX_.'configuration`
WHERE `name` IN (\'PS_UNIDENTIFIED_GROUP\', \'PS_GUEST_GROUP\')'); WHERE `name` IN (\'PS_UNIDENTIFIED_GROUP\', \'PS_GUEST_GROUP\')');
$result = false; $result = true;
if (count($carriers) && is_array($carriers) && count($groups) && is_array($groups)) if (count($carriers) && is_array($carriers) && count($groups) && is_array($groups))
foreach ($carriers as $carrier) foreach ($carriers as $carrier)
foreach ($groups as $group) foreach ($groups as $group)
Db::getInstance()->execute(' $result &= (bool)Db::getInstance()->execute('
INSERT IGNORE INTO `'._DB_PREFIX_.'carrier_group` INSERT IGNORE INTO `'._DB_PREFIX_.'carrier_group`
VALUES ('.(int)$carrier['id_carrier'].', '.(int)$group['id_group'].')'); VALUES ('.(int)$carrier['id_carrier'].', '.(int)$group['id_group'].')');
if(file_exists($filename) && is_writable($filename)) if($result && file_exists($filename) && is_writable($filename))
$result &= (bool)@file_put_contents($filename, $content);
if($result === true && file_exists($filename) && file_exists($filename_old))
{ {
@unlink($filename_old); $result &= (bool)file_put_contents($filename, $content);
@chmod($filename, 0664); if($result && file_exists($filename_old))
return true; {
$result &= (bool)unlink($filename_old);
chmod($filename, 0664);
}
} }
return false; return $result;
} }