[hide] BB-Code

[Vladson]

Было:

Код: Выделить всё

$valid = FALSE; 
if( $userdata['session_logged_in'] ) { 
$sql = "SELECT u.user_id, u.user_level 
FROM " . USERS_TABLE . " u 
WHERE u.user_id = " . $userdata['user_id'] . " 
AND u.user_posts >= 3"; 
$resultat = $db->sql_query($sql); 
$valid = $db->sql_numrows($resultat) ? TRUE : FALSE;}

Желательно:

Код: Выделить всё

if( $userdata['session_logged_in'] AND $userdata['user_posts'] > 3 )
{
	$valid = true;
}
else
{
	$valid = false;
}

(вы при написании кода придерживайтесь подчерка, или вовсе берите куски кода из самого движка форума а то трудно читаемый код получается, и как следствие трудность в отладке в случае ошибок)

[Егор Наклоняев]

Лично мне не очень нравится идея, что bbcode требует редактирования topic_review.php. Неужели нельзя ограничиться проверками прямо в bbcode.php?

[Alexgraf]

Парни, я ни в коем случае не претендую на "модописателя". Я же сказал, что нашел кусок, который мне показался рабочим, вставил его в мод, попробовал - он работает так, как я хотел.
В форуме этот кусок я привёл как раз для вашей критики (может из соображений безопасности его лучше не применять или еще по какой причине) и получил совет как этот кусок модифицировать.

Кстати вопрос открыт - можно ли сделать [hide=xx] ???

[Vladson]

Парни, я ни в коем случае не претендую на "модописателя". Я же сказал, что нашел кусок, который мне показался рабочим, вставил его в мод, попробовал - он работает так, как я хотел.

Я так аналогично сделал с модом счётчика, так мой код был квалифицирован как

шаманское заклинание

[Егор Наклоняев]

Alexgraf
$valid = ( $userdata['session_logged_in'] && ($userdata['user_id']!=ANONYMOUS) && ($userdata['user_posts']>3))?true:false;

Примерно так, только 3 я бы вынес в глобальные константы.

[Xpert]

Егор Наклоняев
Зарапортовались

Код: Выделить всё

 $userdata['session_logged_in'] && ($userdata['user_id']!=ANONYMOUS)

Будет достаточно только одной проверки...

[Егор Наклоняев]

(задумчиво чешет репу) И то верно...
$valid = ( ($userdata['user_id']!=ANONYMOUS) && ($userdata['user_posts']>3))?true:false;

P.S. Кстати, мне всё равно не нравится реализация этого мода. Какой-то он недобаюканный.

[Vladson]

(задумчиво чешет репу) И то верно...

(удивлённо глядя на это) А я думал что надо не верно, я думал надо то что автор точет (только более трёх постов)

P.S. Кстати, мне всё равно не нравится реализация этого мода. Какой-то он недобаюканный

Есть такое ощущение, но зачем судить так строго, можно просто вынести вердикт ("до выхода из стадии беты ещё далеко")

[[R: R@m$e$ :U]]

Vladson & Егор Наклоняев

до выхода из стадии беты ещё далеко

Какой-то он недобаюканный.

звиняйте... я считаю мод нормальный.... а то, что вы обусждаете это уже повышение функциональности...
Alexgraf

Кстати вопрос открыт - можно ли сделать [hide=xx] ???

можно.... как только появится время, сразу же сделаю...

[Егор Наклоняев]

звиняйте... я считаю мод нормальный.... а то, что вы обусждаете это уже повышение функциональности...

Ещё раз, "на бис":
"Лично мне не очень нравится идея, что bbcode требует редактирования topic_review.php. Неужели нельзя ограничиться проверками прямо в bbcode.php?"

Мне кажется, что разбор bbcode должен жить в специально отведенном для этого месте. Ваше решение хорошо и, может быть в чём-то оправдано, когда phpBB у нас девственный и на него поставили только этот мод, в противном случае, начинается лихорадка по переделке всех установленных "наворотов" под этот код.
Простой пример: Портал от того же Smartor'а. Сразу нужно править еще и portal.php, а что будет выводиться без модернизации - догадаться нетрудно.

[[R: R@m$e$ :U]]

Егор Наклоняев
"на бис" принято.... =))) будем исправлятся...

[[R: R@m$e$ :U]]

Егор Наклоняев
исправились оцени такой вариант....

[igorekS]

Тут такое дело: при установке мода обнаружил что в templates/subSilver/quick_reply.tpl нет кодов

Код: Выделить всё

bbtags = new Array('[b]','[/b]','[i]','[/i]','[u]','[/u]','[quote]','[/quote]','[code]','

','

• ','

','

• ','

','[img]','[/img]','','');[/code]

Код: Выделить всё

<input type="button" class="button" name="addbbcode16" value="URL" style="text-decoration: underline; width: 40px" onClick="bbstyle(16)" />

Так же не нашел там файла templates/subSilver/quick_topic.tpl
В следствие чего данный мод не работает, а он крайне необходим.

Версия форума 2.0.13. Скин стандартный.

[terve]

igorekS
это файлы быстрого ответа. Если у вас его нет - просто проигнорируйте этот пункт.

[igorekS]

Дык, проигнорировал, но мод не пашет. Мод быстрого ответа у меня стоит guickreply. Может накосячил с установкой?

Добавлено спустя 17 минут 47 секунд:

Тогда еще такой вопрос: устанавливать нужно тот мод что опубликован первым?