Ratibor |
|
---|---|
бновился до ревизии 47, стал создавать базу, вылетела ошибка:
SQL-запрос: INSERT INTO sed_forum_sections VALUES ( '1', '0', '100', 'General discussion', 'pub', 'General chat.', 'system/img/admin/forums.gif', 0, '', 0, 0, '', 365, 1, 1, 1, 1, 1, 0, 0, 0, 0, 0 ) Ответ MySQL: #1136 - Column count doesn't match value count at row 1 Потом попробовал запустить сайт: Warning: mb_internal_encoding() [function.mb-internal-encoding]: Unknown encoding "" in Z:\home\test7.ru\www\system\common.php on line 49 2008-08-31 01:05 Fatal error : No groups found. Не задавай глупых вопросов, не услышишь вранья.
|
Antar |
|
---|---|
Asmo:По-моему грех жаловаться, за неделю сделано больше чем за последние 1.5 года. |
Trustmaster |
|
---|---|
Asmo, хорошая мысль. Лучше это сделать таким образом:
if(!$cfg['parser_custom'])
{
sed_bbcode_load();
} Ratibor, ага, субфорумы тут замешаны: новое поле в базе не учтено. Сейчас пофиксим ;) May the Source be with you!
|
Asmo |
|
---|---|
#735 Antar :Да вот я как раз к тому и веду что слишком уж много, я себе представлял так что это будет постепенно, первый релиз - фикс багов, а потому уже улучшения.Asmo:По-моему грех жаловаться, за неделю сделано больше чем за последние 1.5 года. Какбы не потерятся во всем этом сразу. |
Asmo |
|
---|---|
#736 Trustmaster : Asmo, хорошая мысль. Лучше это сделать таким образом:Логично, есть конечно небольшая, но вероятность, что может вообще не использоваться bbcode, скажем чисто штмл в страницах, с отключенными комментами и форумами. Оно то можно включить пустой customparser, чтобы зарезать этот запрос, но как-то не аккуратненько это выходит, имхо. |
Trustmaster |
|
---|---|
Не стоит бросаться в крайности в погоне за призраком производительности. Я, кажется, рассказывал, что интереса ради вообще вытравил все хуки и всю расширяемость из ядра и плагинов, сделав его единым прямоточным скриптом с несколькими инклюдами. И что же вы думаете я получил, невероятный прирост производительности? Ничего подобного, прирост был незначительный, а вот поддерживать этот сайт было практически невозможно (т.е. возможно, но некогда). Так что всегда стоит искать оптимум между производительностью и функциональностью/удобством.
May the Source be with you!
|
Ratibor |
|
---|---|
Ну вот, сваял плагин на скорую руку: 10-wysiwyg_plugin.zip
Пока ничего не менял, все как в оригинале. Нужно еще картинки лишние поудалять и почистить css. Как я сказал, главное надо разобраться почему не правильно работают расширенные коды. А так плагин работоспособный, проверил на последней ревизии. Так что осталось немного доработать напильником, а главное дождаться когда Trustmaster зделает в парсере обработку [*] Не задавай глупых вопросов, не услышишь вранья.
|
|
Bu gönderi Ratibor tarafından düzenlendi (2008-08-31 07:06, 16 yıllar önce) |
Trustmaster |
|
---|---|
Я по-моему ясно сказал, обработку
потому как либо некорректно работает с многострочными пунктами, либо сильно усложняет логику парсера.Если очень хочется, можете реализовать однострочный вариант в своем плагине, в install.php это делается очень легко:
sed_bbcode_add('li', 'pcre', '\[\*\](.*?)\n', '<li>$1</li>', true, 128, 'your_plugin_name');
Подробные примеры того, как легко расширяется функционал парсера при установке плагинов, можно посмотреть в markitup и chili. EDIT: хотя нет, можно зацеплять и многострочные одной регуляркой, чуть позже напишу. Но в May the Source be with you!
|
Dayver |
|
---|---|
Еще пара вопросов(уточнений):
Плуг Recent polls с аяксом включен в стандартный набор и это хорошо (значит, постепенно эта технология будет включаться в набор)..... но есть одно но ....попробуйте включить в его конфиге отображение не одного последнего опроса, а к примеру двух (знаю что это практически никто не юзает но настройка то есть) и при таком конфиге поголосовать (в разном порядке сначала за предпоследний(и обновить стр.) потом за последний) думаю то что увидите, задумывалось не так....правда же? Да и еще небольшой вопрос, а в MarkItUp! тег спойлера не предусмотрен? или его нужно включать дополнительно? если да то думаю, стоит его сделать по умолчанию Pavlo Tkachenko aka Dayver
|
Trustmaster |
|
---|---|
Да, это баг в recentpolls, фикс в процессе.
Пардон, про спойлер забыл совсем. Сделаем. May the Source be with you!
|
Dayver |
|
---|---|
И вот я посмотрел в /мой.сайт/users.php?m=details может сделать парсинг ббкодов в поле Подпись:, а?
ну когда юзер в своем профиле вставляет в это поле текст то с помощью MarkItUp!'а то он же и может просмотреть как это будет выглядеть на форуме но вот когда смотришь это в юзердетайлс то видим что то вроде [hide]вот [b]это[/b] [color=#CC0000]скрыто[/color][/hide]хотя лучше было бы видеть так же как и в постах на форуме Ну эт больше как размышление. А вот не корректностью считаю вот такую ситуацию: в этом же поле подписи, если юзер вставит тег hide то на форуме в постах оно не обработается и так и будет висеть......возможно, его либо вырезать при парсинге поста или настроить, что б оно обрабатывалось так же как это делается в страницах Pavlo Tkachenko aka Dayver
|
Trustmaster |
|
---|---|
Недоделки. Спасибо за репорт.
May the Source be with you!
|
Dayver |
|
---|---|
Да, а как насчет валидности хтмл кода?
Одно дело что если в скине не валид но когда это выдается ядром(или базовыми плугами) то это уже оч не солидно Пример: функция sed_build_user строка return (!empty($user)) ? '<a href="users.php?m=details&id='.$id.'">'.$user.'</a>' : '?';sed_build_forums три строки ... $tmp[] = '<a href="forums.php?m=topics&s='.$master[0].'">'.sed_cc($master[1]).'</a>'; ... $tmp[] = '<a href="forums.php?m=topics&s='.$master.'">'.sed_cc( $rowa['fs_title']).'</a>'; ... $tmp[] = '<a href="forums.php?m=topics&s='.$sectionid.'">'.sed_cc($title).'</a>'; ...используется символ амперсанда, а не его код это вроде не валидно (солидно :) каламбурчик) вообщем много чего в этом плане можно найти... если то о чем я говорю актуально, то могу, составит список ошибок по крупнее[/][/][/] Pavlo Tkachenko aka Dayver
|
|
Bu gönderi Dayver tarafından düzenlendi (2008-08-31 15:46, 16 yıllar önce) |
Trustmaster |
|
---|---|
Пофиксено, проверь новую ревизию валидатором плиз.
May the Source be with you!
|
Ratibor |
|
---|---|
Закачал ревизию 51, установил вчистую
Создаю пост, весь текст взят в тэг code. И что видим: ![]() А что тут миниатюры не показываются ? Во, сразу еще одна недоработка, при редактировании поста, нет Смайлы Мои файлы между редактором и кнопкой Готово. Ну да ладно, вот: ![]() Нет рамки для code, не отличишь где обычный текст, а где вставка code Ну и для кучи, внутри тэга code, парсится тэг hide, непорядок, там вообще ничего не должно парсится. Не задавай глупых вопросов, не услышишь вранья.
|