Форуми / National / Russian / Бета -впечатления.

12345...10>>>

Asmo
#1 30.08.2008 11:48
Мда, хреново у меня с английским оказывается, простыми однослонжыми фразами кое как получается, а что либо серьезное связать - проблема :(
По теме. Работы выполнено много, за что Трастмастеру большой респект, но эту самодеятельность я буду сейчас очень критиковать.

Первое, говорили, говорили по стилю кодописания и договорились.
Вот эти вещи
$result = <<<END <scrip ... 
например в функции sed_javascript() и в других местах зачем?
Может я чего не понимаю и это несет какую то пользу? Я вижу только вред, после этой функции у меня в редакторе код не подсвечиается :) , а вываливается просто как текст, да и вообще нарушает общий стиль.
Чем хуже, как это уже делается при генерации доктайпа и прочих метатегах?
$result = '<script type="text/javascript" src="js/jquery.js"></script>';

Вот эти фигурные скобки в sql это модно?
@mysql_query("SET NAMES {$cfg['mysqlcharset']}", $connection);
Почему не сделать как везде в движке по людски, сохраняя общий стиль
@mysql_query("SET NAMES ".$cfg['mysqlcharset'], $connection);

Далее, по яваскриптам:
Раз уж jquery интегрировали, то хотябы сделать возможность отключать его если не нужен.
И зачем sed_javascript() пустой контейнер вставляет в шапку если $more пустой
<script type="text/javascript">
<!--

//-->
</script>

Гораздо элегантнее имхо:
function sed_javascript($more='')
{
	$result = "<script type=\"text/javascript\" src=\"js/jquery.js\"></script>
<script type=\"text/javascript\" src=\"js/base.js\"></script>\n";
	if (!empty($more)) {
	$result .= "<script type=\"text/javascript\">
<!--
".$more."
//-->
</script>\n";
	}
	return $result;
}

Далее, по парсеру, мудрено все очень как-то это, позже когда разберусь с ним оптишусь.

Markitup свои яваскрипты и цсс вставляет тупо в шапку везде подряд - не порядок, надо что-то приудмать чтобы только там где он используется вставлялось.
Опять же парсер, парсер новый, а текстбоксер с набором тегов старый, он там для красоты чтоли стоит?
Полез проверить пофиксен ли баг с предпросмотром картинки из общего файлового архива, при методе хранения в папках, тег превью вообще не работает - круто. Но судя по коду pfs не пофиксен, а я где-то выкладывал решение или в трекере или тут на форуме ,не найду чего-то только где.

Пока все, но еще буду ругатсья дальше :) чтобы не расслаблялся никто :)
Trustmaster
#2 30.08.2008 12:20
С оформлением строк не соглашусь, ибо разделяй и властвуй. Это все делается не от балды, а потому что каждый вид строк (одинарные, двойные кавычки и heredoc) должен использоваться по назначению, а не потому, что кому-то что-то привычнее.

Пустой $more пофиксим, спасибо. Возможность отключения jQuery в конфигурацию добавим.

C markItUp мне не хочется повторять все подвиги текстбоксера и задействовать его с помощью хуков в php-коде каждый раз. Смысл именно в том, чтобы он подцеплялся там где встречается textarea редактора, автоматически без лишних телодвижений. Не нравится несколько лишних тегов в шапке - не используй.

Поддержкой текстбоксера я больше заниматься не буду. Нужен? Делайте!

Фиксы PFS пришлось отложить до следующей версии, слишком много проблем с поддержкой "folder mode" и обычного режима, там надо копать глубже, чем в твоем патче.
May the Source be with you!

Відредаговано: Trustmaster (30.08.2008 12:28, 15 років тому)
Asmo
#3 30.08.2008 12:41
По оформлению строк спорить не буду, тебе виднее.
markItUp - согласен, кого не устраивает, использовать никто не заставляет, но все же это не просто пара тегов лишних, а еще и лишние KB в виде подгружаемых файлов и хттп запросы серверу.

А что с PFS сложного именно в storage mode?
Единственная проблема была которая напрягала с превью. Кстати ты про какой мой патч говоришь, про свежий или годичной давности на neocrome.ru? Старый он кривой и не решал всех вопросов.
Свежий вот ищу, ищу, не могу найти куда я его запостил.
Ratibor
#4 30.08.2008 12:44
Asmo : Опять же парсер, парсер новый, а текстбоксер с набором тегов старый, он там для красоты чтоли стоит?
Так же не понятно за каким залиты в папке с плагинами следующие папки:
contactus
gallery
managerpages
overview
toplastseen
userrights

Либо заливать полностью плагины, либо ничего.
Это было во первых.
Во вторых файлы русского языка зделаны абсолютно не правильно.
Всмысле переведены из виндовой кодировки в utf-8.
На данный момент эти файлы полностью не работоспособны.
Если кто не знает, ту существует как минимум две кодировки utf-8, с ВОМ (EF BB BF) из без ВОМ.
Отличия в том что в данных файлах присутствуют в начале каждого файла ВОМ символы, а их не должно быть.
В третьих, нужно исправить search.ru.lang.php.
Ну и в четвертых, раз уж перенесли папку img из систем, то тогда уже убрать оттуда и смайлы.
Не задавай глупых вопросов, не услышишь вранья.
Asmo
#5 30.08.2008 12:51
Да языковые файлы нужно через Notepad++ перекодировать, там есть опция UTF-8 без BOM, а виндовый блокнот сохраняет только с BOM.
Asmo
#6 30.08.2008 12:57
Asmo :
Опять же парсер, парсер новый, а текстбоксер с набором тегов старый, он там для красоты чтоли стоит?
Тут е не против текстбоксера в целом, а что раз набор ббкодов ограничен, остальные, не поддерживаемые, по хорошему закомментить надо, чтоб не путались.
А то я долго втыкал почему не парсится тег thumb, полез в парсеер ,а парсера нету, в привычном понимании :)
Trustmaster
#7 30.08.2008 13:05
Asmo:
markItUp - согласен, кого не устраивает, использовать никто не заставляет, но все же это не просто пара тегов лишних, а еще и лишние KB в виде подгружаемых файлов и хттп запросы серверу.
А вот и наоборот. Внешние .js и .css файлы успешно кэшируются браузером, в отличие от кода, вставляемого в каждую страницу, как это делает текстбоксер.

Ratibor:
Так же не понятно за каким залиты в папке с плагинами следующие папки:
contactus
gallery
managerpages
overview
toplastseen
userrights
Просили русскую локализацию в UTF-8 от Antar'а? Так вот это ее часть.

Ratibor:
Во вторых файлы русского языка зделаны абсолютно не правильно.
Всмысле переведены из виндовой кодировки в utf-8.
На данный момент эти файлы полностью не работоспособны.
Странно, у меня все работает. Если пришлете правильно перекодированные файлы, включу их.

Ratibor:
Ну и в четвертых, раз уж перенесли папку img из систем, то тогда уже убрать оттуда и смайлы.
Над смайлами будем работать в N-0.0.2. В частности, надо вместо жуткого редактора в админке сделать нормальную поддержку паков со смайлами + чтобы их можно было выбирать как и скины.
May the Source be with you!
Asmo
#8 30.08.2008 13:11
Ratibor:
Во вторых файлы русского языка зделаны абсолютно не правильно.
Всмысле переведены из виндовой кодировки в utf-8.
На данный момент эти файлы полностью не работоспособны.
Странно, у меня все работает. Если пришлете правильно перекодированные файлы, включу их.
Да в языковых файлах это не критично, но все же считается хорошим тоном кодировать без BOM, потому что будут проблемы, если так скодировать например function.php или другой исполняемый, будет вылазить ошибка header alredy sent
И вообще BOM это чисто виндовая заморочка насколько я понял.
Trustmaster
#9 30.08.2008 13:21
Тогда у меня большая просьба: перекодируйте, пожалуйста, языковый файлы как надо и выложите, чтобы я их включил в trunk.
May the Source be with you!
Ratibor
#10 30.08.2008 13:21
Trustmaster : Странно, у меня все работает.
Работает, но как ?
Сравни размер шрифта ?
Когда переключаешься на русский шриф становится намного крупней.
Не задавай глупых вопросов, не услышишь вранья.
Asmo
#11 30.08.2008 13:30
Ratibor, хз у меня нормально, но перекодировать надо.
Trustmaster, я точно постил патч для решения pfs storage mode или только хотел, но забыл? Во всяком случае меня он устроил и больше проблем я не заметил нигде там.
Ratibor
#12 30.08.2008 13:40
Вот нормальная руссификация :10-seditio_utf8.zip
Файлы для плагинов:
contactus
gallery
managerpages
overview
toplastseen
userrights

не переводил, т.к. это уже из другой оперы :)
Их надо выкосить из основного дистрибутива.

И на будующее, кто будет вносить изменения в русские файлы, как правильно сказал Asmo, пользуйтесь Notepad++

Там в закладке "Кодировки", вверуху должно быть выбрано "Преобразовать в UNIX-формат" и "Кодировать в UTF-8 (без ВОМ)".
Не задавай глупых вопросов, не услышишь вранья.
Dayver
#13 30.08.2008 13:56
Да и у меня проблема с которую уже описали на английсков в соотв. топике
Скопировал на локалхост залил seditio-n-0.0.1.sql настроил config.php запускаю, а вываливает ошибку
Fatal error: Call to undefined function mb_internal_encoding() in C:\home\cotonti\www\system\common.php on line 49)
Pavlo Tkachenko aka Dayver
Asmo
#14 30.08.2008 14:01
Dayver, У тебя другая проблема, нужно в php.ini включить поддержку mbstring
extension=php_mbstring.dll
Dayver
#15 30.08.2008 14:03
Порадовало то что папка images теперь в корне, а не в системс.....ток вот смайлы чёт на старом месте оставили

Остальное пока не протестил из-за еррора(ручками в базе в конфиге сменил кодировку, не помогло)
Pavlo Tkachenko aka Dayver

12345...10>>>