CMS DataLife Engine - Система управления сайтами

Sign in to follow this  
webair

Кодировка в DLE

Recommended Posts

Как я понимаю, с 13.0 осуществлен полный переход на utf-8 без каких либо других вариантов?

Но в коде DLE везде присутствует бесполезный легаси код. Который, как бы и не мешает, но смысла копить этот код нет (параметр charset).

К примеру:

function dle_strlen($value, $charset ) {

	if( function_exists( 'mb_strlen' ) ) {
		return mb_strlen( $value, $charset );
	} elseif( function_exists( 'iconv_strlen' ) ) {
		return iconv_strlen($value, $charset);
	}

	return strlen($value);
}

function dle_substr($str, $start, $length, $charset ) {

	if( function_exists( 'mb_substr' ) ) {
		return mb_substr( $str, $start, $length, $charset );
	
	} elseif( function_exists( 'iconv_substr' ) ) {
		return iconv_substr($str, $start, $length, $charset);
	}

	return substr($str, $start, $length);

}

function dle_strrpos($str, $needle, $charset ) {

	if( function_exists( 'mb_strrpos' ) ) {
		return mb_strrpos( $str, $needle, null, $charset );
	
	} elseif( function_exists( 'iconv_strrpos' ) ) {
		return iconv_strrpos($str, $needle, $charset);
	}

	return strrpos($str, $needle);

}

function dle_strpos($str, $needle, $charset ) {

	if( function_exists( 'mb_strpos' ) ) {
		return mb_strpos( $str, $needle, null, $charset );
	} elseif( function_exists( 'iconv_strrpos' ) ) {
		return iconv_strpos($str, $needle, null, $charset);
	}

	return strpos($str, $needle);

}

 

Может я не прав, конечно.

Edited by webair

Share this post


Link to post
Share on other sites

Это раздел приема багов. Баг в чем? Ничего бесполезного в этом коде нет. Универсальный код, работающий с любой кодировкой не значит бесполезный. Сохранен он ради совместимости с кодом написанным под ранние версии, например для сторонних модулей чтобы они работали с минимальными изменениями при необходимости или вообще без изменений.

Share this post


Link to post
Share on other sites

Join the conversation

You can post now and register later. If you have an account, sign in now to post with your account.

Guest
Reply to this topic...

×   Pasted as rich text.   Paste as plain text instead

  Only 75 emoji are allowed.

×   Your link has been automatically embedded.   Display as a link instead

×   Your previous content has been restored.   Clear editor

×   You cannot paste images directly. Upload or insert images from URL.

Sign in to follow this