Бета-впечатления v.0.0.2

Тема обсуждения багов

Tags: Бета-Впечатления V.0.0.2

motor2hg	#6893 2009-02-05 20:48
Contributors Thanked: 1 time	Не правильно обошёл! Потому, что переменная $cfg['mysqlcharset'] должна заполнятся из значения Админ-Панель -> Главная -> Скины и кодировка -> HTML кодировка : ИМХО Или браться из config.php как предлагает Ratibor он её туда закоментированную сунул. А проблема у тебя возникла из-за того, что мы уже обсуждали выше в файле http://svn.cotonti.com/trunk/system/database.mysql.php function sed_sql_connect($host, $user, $pass, $db) { global $cfg; $connection = @mysql_connect($host, $user, $pass) or sed_diefatal('Could not connect to database !<br />Please check your settings in the file datas/config.php<br />'.'MySQL error : '.sed_sql_error()); if (!empty($cfg['mysqlcharset'])) { @mysql_query("SET NAMES {$cfg['mysqlcharset']}", $connection); } $select = @mysql_select_db($db, $connection) or sed_diefatal('Could not select the database !<br />Please check your settings in the file datas/config.php<br />'.'MySQL error : '.sed_sql_error()); return $connection; } Указано только $cfg['mysqlcharset'], а COLLATE в @mysql_query не указано, а брать значение этого параметра Ratibor предлагал внеся изменения в config.php а именно определить там переменную $cfg['mysqlcollate'] = 'utf8_unicode_ci'; И тогда получилось бы так, function sed_sql_connect($host, $user, $pass, $db) { global $cfg; $connection = @mysql_connect($host, $user, $pass) or sed_diefatal('Could not connect to database !<br />Please check your settings in the file datas/config.php<br />'.'MySQL error : '.sed_sql_error()); if (!empty($cfg['mysqlcharset'])) { @mysql_query("SET NAMES '{$cfg['mysqlcharset']}' COLLATE '{$cfg['mysqlcollate']}'", $connection); } $select = @mysql_select_db($db, $connection) or sed_diefatal('Could not select the database !<br />Please check your settings in the file datas/config.php<br />'.'MySQL error : '.sed_sql_error()); return $connection; } И не надо было-бы парится, а ещё лучшее предложение удалить этот код нафиг из этого файла из оператора @mysql_query, потому, что COLLATE можеть и не использоваться пользователем, а в коде переменная присутствует. А добавить код в конце запроса к базе в файле http://svn.cotonti.com/trunk/system/common.php Тоже предлагал Ratibor, правильно предлагал if (!empty($cfg['mysqlcharset'])) { sed_sql_query("SET NAMES '{$cfg['mysqlcharset']}' COLLATE '{$cfg['mysqlcollate']}'"); Тут проверочка на наличие переменной есть есть супер, если нет так и спросу нет. А твоё решение в переменную $cfg['mysqlcharset'] записать значение charset и COLLATE естественно сработало, только кто из юзеров догадается что мухи и котлеты вместе :)
This post was edited by motor2hg (2009-02-05 21:00, 16 years ago)

motor2hg

#6893 2009-02-05 20:48

Не правильно обошёл!
Потому, что переменная $cfg['mysqlcharset'] должна заполнятся из значения Админ-Панель -> Главная -> Скины и кодировка -> HTML кодировка : ИМХО
Или браться из config.php как предлагает Ratibor он её туда закоментированную сунул.

А проблема у тебя возникла из-за того, что мы уже обсуждали выше в файле

http://svn.cotonti.com/trunk/system/database.mysql.php

Указано только $cfg['mysqlcharset'], а COLLATE в @mysql_query не указано, а брать значение этого параметра Ratibor предлагал внеся изменения в config.php а именно определить там переменную $cfg['mysqlcollate'] = 'utf8_unicode_ci';
И тогда получилось бы так,

И не надо было-бы парится, а ещё лучшее предложение удалить этот код нафиг из этого файла из оператора @mysql_query, потому, что COLLATE можеть и не использоваться пользователем, а в коде переменная присутствует. А добавить код в конце запроса к базе в файле

http://svn.cotonti.com/trunk/system/common.php

Тоже предлагал Ratibor, правильно предлагал

if (!empty($cfg['mysqlcharset']))
	{
		sed_sql_query("SET NAMES '{$cfg['mysqlcharset']}' COLLATE '{$cfg['mysqlcollate']}'");

Тут проверочка на наличие переменной есть есть супер, если нет так и спросу нет.
А твоё решение в переменную $cfg['mysqlcharset'] записать значение charset и COLLATE естественно сработало, только кто из юзеров догадается что мухи и котлеты вместе :)

This post was edited by motor2hg (2009-02-05 21:00, 16 years ago)