Asmo |
|
---|---|
Мда, хреново у меня с английским оказывается, простыми однослонжыми фразами кое как получается, а что либо серьезное связать - проблема :(
По теме. Работы выполнено много, за что Трастмастеру большой респект, но эту самодеятельность я буду сейчас очень критиковать. Первое, говорили, говорили по стилю кодописания и договорились. Вот эти вещи $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 |
|
---|---|
С оформлением строк не соглашусь, ибо разделяй и властвуй. Это все делается не от балды, а потому что каждый вид строк (одинарные, двойные кавычки и heredoc) должен использоваться по назначению, а не потому, что кому-то что-то привычнее.
Пустой $more пофиксим, спасибо. Возможность отключения jQuery в конфигурацию добавим. C markItUp мне не хочется повторять все подвиги текстбоксера и задействовать его с помощью хуков в php-коде каждый раз. Смысл именно в том, чтобы он подцеплялся там где встречается textarea редактора, автоматически без лишних телодвижений. Не нравится несколько лишних тегов в шапке - не используй. Поддержкой текстбоксера я больше заниматься не буду. Нужен? Делайте! Фиксы PFS пришлось отложить до следующей версии, слишком много проблем с поддержкой "folder mode" и обычного режима, там надо копать глубже, чем в твоем патче. May the Source be with you!
|
|
This post was edited by Trustmaster (2008-08-30 12:28, 16 years ago) |
Asmo |
|
---|---|
По оформлению строк спорить не буду, тебе виднее.
markItUp - согласен, кого не устраивает, использовать никто не заставляет, но все же это не просто пара тегов лишних, а еще и лишние KB в виде подгружаемых файлов и хттп запросы серверу. А что с PFS сложного именно в storage mode? Единственная проблема была которая напрягала с превью. Кстати ты про какой мой патч говоришь, про свежий или годичной давности на neocrome.ru? Старый он кривой и не решал всех вопросов. Свежий вот ищу, ищу, не могу найти куда я его запостил. |
Ratibor |
|
---|---|
Asmo : Опять же парсер, парсер новый, а текстбоксер с набором тегов старый, он там для красоты чтоли стоит?Так же не понятно за каким залиты в папке с плагинами следующие папки: contactus gallery managerpages overview toplastseen userrights Либо заливать полностью плагины, либо ничего. Это было во первых. Во вторых файлы русского языка зделаны абсолютно не правильно. Всмысле переведены из виндовой кодировки в utf-8. На данный момент эти файлы полностью не работоспособны. Если кто не знает, ту существует как минимум две кодировки utf-8, с ВОМ (EF BB BF) из без ВОМ. Отличия в том что в данных файлах присутствуют в начале каждого файла ВОМ символы, а их не должно быть. В третьих, нужно исправить search.ru.lang.php. Ну и в четвертых, раз уж перенесли папку img из систем, то тогда уже убрать оттуда и смайлы. Не задавай глупых вопросов, не услышишь вранья.
|
Asmo |
|
---|---|
Да языковые файлы нужно через Notepad++ перекодировать, там есть опция UTF-8 без BOM, а виндовый блокнот сохраняет только с BOM.
|
Asmo |
|
---|---|
Asmo : Опять же парсер, парсер новый, а текстбоксер с набором тегов старый, он там для красоты чтоли стоит?Тут е не против текстбоксера в целом, а что раз набор ббкодов ограничен, остальные, не поддерживаемые, по хорошему закомментить надо, чтоб не путались. А то я долго втыкал почему не парсится тег thumb, полез в парсеер ,а парсера нету, в привычном понимании :) |
Trustmaster |
|
---|---|
Asmo:А вот и наоборот. Внешние .js и .css файлы успешно кэшируются браузером, в отличие от кода, вставляемого в каждую страницу, как это делает текстбоксер. Ratibor:Просили русскую локализацию в UTF-8 от Antar'а? Так вот это ее часть. Ratibor:Странно, у меня все работает. Если пришлете правильно перекодированные файлы, включу их. Ratibor:Над смайлами будем работать в N-0.0.2. В частности, надо вместо жуткого редактора в админке сделать нормальную поддержку паков со смайлами + чтобы их можно было выбирать как и скины. May the Source be with you!
|
Asmo |
|
---|---|
Да в языковых файлах это не критично, но все же считается хорошим тоном кодировать без BOM, потому что будут проблемы, если так скодировать например function.php или другой исполняемый, будет вылазить ошибка header alredy sentRatibor:Странно, у меня все работает. Если пришлете правильно перекодированные файлы, включу их. И вообще BOM это чисто виндовая заморочка насколько я понял. |
Trustmaster |
|
---|---|
Тогда у меня большая просьба: перекодируйте, пожалуйста, языковый файлы как надо и выложите, чтобы я их включил в trunk.
May the Source be with you!
|
Ratibor |
|
---|---|
Trustmaster : Странно, у меня все работает.Работает, но как ? Сравни размер шрифта ? Когда переключаешься на русский шриф становится намного крупней. Не задавай глупых вопросов, не услышишь вранья.
|
Asmo |
|
---|---|
Ratibor, хз у меня нормально, но перекодировать надо.
Trustmaster, я точно постил патч для решения pfs storage mode или только хотел, но забыл? Во всяком случае меня он устроил и больше проблем я не заметил нигде там. |
Ratibor |
|
---|---|
Вот нормальная руссификация :10-seditio_utf8.zip
Файлы для плагинов: contactus gallery managerpages overview toplastseen userrights не переводил, т.к. это уже из другой оперы :) Их надо выкосить из основного дистрибутива. И на будующее, кто будет вносить изменения в русские файлы, как правильно сказал Asmo, пользуйтесь Notepad++ Там в закладке "Кодировки", вверуху должно быть выбрано "Преобразовать в UNIX-формат" и "Кодировать в UTF-8 (без ВОМ)". Не задавай глупых вопросов, не услышишь вранья.
|
Dayver |
|
---|---|
Да и у меня проблема с которую уже описали на английсков в соотв. топике
Скопировал на локалхост залил 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 |
|
---|---|
Dayver, У тебя другая проблема, нужно в php.ini включить поддержку mbstring
extension=php_mbstring.dll |
Dayver |
|
---|---|
Порадовало то что папка images теперь в корне, а не в системс.....ток вот смайлы чёт на старом месте оставили
Остальное пока не протестил из-за еррора(ручками в базе в конфиге сменил кодировку, не помогло) Pavlo Tkachenko aka Dayver
|