Foren / National / Russian / Бета -впечатления.

ErsteVorherige1234567...10NächsteLetzte

Ratibor
#46 30. August 2008, 19:03
бновился до ревизии 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
#47 30. August 2008, 19:11
Asmo:
Чего-то я начинаю снова впадать в пессимистическое уныние, чем дальше в лес, тем толще партизаны.
По-моему грех жаловаться, за неделю сделано больше чем за последние 1.5 года.
Trustmaster
#48 30. August 2008, 19:25
Asmo, хорошая мысль. Лучше это сделать таким образом:
if(!$cfg['parser_custom'])
{
sed_bbcode_load();
}
потому что если стандартный набор ббкодов не нужен, то другой парсер используется наверняка. А лишние опции конфига разводить незачем.

Ratibor, ага, субфорумы тут замешаны: новое поле в базе не учтено. Сейчас пофиксим ;)
May the Source be with you!
Asmo
#49 30. August 2008, 19:47
#735 Antar :
Asmo:
Чего-то я начинаю снова впадать в пессимистическое уныние, чем дальше в лес, тем толще партизаны.
По-моему грех жаловаться, за неделю сделано больше чем за последние 1.5 года.
Да вот я как раз к тому и веду что слишком уж много, я себе представлял так что это будет постепенно, первый релиз - фикс багов, а потому уже улучшения.
Какбы не потерятся во всем этом сразу.
Asmo
#50 30. August 2008, 19:53
#736 Trustmaster : Asmo, хорошая мысль. Лучше это сделать таким образом:
if(!$cfg['parser_custom'])
{
sed_bbcode_load();
}
потому что если стандартный набор ббкодов не нужен, то другой парсер используется наверняка. А лишние опции конфига разводить незачем.
Логично, есть конечно небольшая, но вероятность, что может вообще не использоваться bbcode, скажем чисто штмл в страницах, с отключенными комментами и форумами.
Оно то можно включить пустой customparser, чтобы зарезать этот запрос, но как-то не аккуратненько это выходит, имхо.
Trustmaster
#51 30. August 2008, 20:03
Не стоит бросаться в крайности в погоне за призраком производительности. Я, кажется, рассказывал, что интереса ради вообще вытравил все хуки и всю расширяемость из ядра и плагинов, сделав его единым прямоточным скриптом с несколькими инклюдами. И что же вы думаете я получил, невероятный прирост производительности? Ничего подобного, прирост был незначительный, а вот поддерживать этот сайт было практически невозможно (т.е. возможно, но некогда). Так что всегда стоит искать оптимум между производительностью и функциональностью/удобством.
May the Source be with you!
Ratibor
#52 31. August 2008, 06:45
Ну вот, сваял плагин на скорую руку: 10-wysiwyg_plugin.zip
Пока ничего не менял, все как в оригинале.
Нужно еще картинки лишние поудалять и почистить css.
Как я сказал, главное надо разобраться почему не правильно работают расширенные коды.
А так плагин работоспособный, проверил на последней ревизии.

Так что осталось немного доработать напильником, а главное дождаться когда Trustmaster зделает в парсере обработку [*]
Не задавай глупых вопросов, не услышишь вранья.

Dieser Beitrag wurde von Ratibor (am 31. August 2008, 07:06, vor 15 Jahre) bearbeitet
Trustmaster
#53 31. August 2008, 07:14
Я по-моему ясно сказал, обработку
  • я делать не буду
  • потому как либо некорректно работает с многострочными пунктами, либо сильно усложняет логику парсера.
    Если очень хочется, можете реализовать однострочный вариант в своем плагине, в 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
    #54 31. August 2008, 11:56
    Еще пара вопросов(уточнений):
    Плуг Recent polls с аяксом включен в стандартный набор и это хорошо (значит, постепенно эта технология будет включаться в набор)..... но есть одно но ....попробуйте включить в его конфиге отображение не одного последнего опроса, а к примеру двух (знаю что это практически никто не юзает но настройка то есть) и при таком конфиге поголосовать (в разном порядке сначала за предпоследний(и обновить стр.) потом за последний) думаю то что увидите, задумывалось не так....правда же?

    Да и еще небольшой вопрос, а в MarkItUp! тег спойлера не предусмотрен? или его нужно включать дополнительно? если да то думаю, стоит его сделать по умолчанию
    Pavlo Tkachenko aka Dayver
    Trustmaster
    #55 31. August 2008, 12:21
    Да, это баг в recentpolls, фикс в процессе.

    Пардон, про спойлер забыл совсем. Сделаем.
    May the Source be with you!
    Dayver
    #56 31. August 2008, 13:02
    И вот я посмотрел в /мой.сайт/users.php?m=details может сделать парсинг ббкодов в поле Подпись:, а?
    ну когда юзер в своем профиле вставляет в это поле текст то с помощью MarkItUp!'а то он же и может просмотреть как это будет выглядеть на форуме но вот когда смотришь это в юзердетайлс то видим что то вроде
    [hide]вот [b]это[/b] [color=#CC0000]скрыто[/color][/hide]
    хотя лучше было бы видеть так же как и в постах на форуме
    Ну эт больше как размышление.

    А вот не корректностью считаю вот такую ситуацию:
    в этом же поле подписи, если юзер вставит тег hide
    то на форуме в постах оно не обработается и так и будет висеть......возможно, его либо вырезать при парсинге поста или настроить, что б оно обрабатывалось так же как это делается в страницах
    Pavlo Tkachenko aka Dayver
    Trustmaster
    #57 31. August 2008, 13:06
    Недоделки. Спасибо за репорт.
    May the Source be with you!
    Dayver
    #58 31. August 2008, 15:40
    Да, а как насчет валидности хтмл кода?
    Одно дело что если в скине не валид но когда это выдается ядром(или базовыми плугами) то это уже оч не солидно

    Пример:
    функция 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

    Dieser Beitrag wurde von Dayver (am 31. August 2008, 15:46, vor 15 Jahre) bearbeitet
    Trustmaster
    #59 31. August 2008, 16:33
    Пофиксено, проверь новую ревизию валидатором плиз.
    May the Source be with you!
    Ratibor
    #60 31. August 2008, 19:38
    Закачал ревизию 51, установил вчистую
    Создаю пост, весь текст взят в тэг code.
    И что видим:

    А что тут миниатюры не показываются ?
    Во, сразу еще одна недоработка, при редактировании поста, нет Смайлы Мои файлы между редактором и кнопкой Готово.

    Ну да ладно, вот:


    Нет рамки для code, не отличишь где обычный текст, а где вставка code
    Ну и для кучи, внутри тэга code, парсится тэг hide, непорядок,
    там вообще ничего не должно парсится.
    Не задавай глупых вопросов, не услышишь вранья.

    ErsteVorherige1234567...10NächsteLetzte