[final] Limit code and quote bbCode bodytext

[rxu]

Мод позволяет ограничить размеры "портянок", выкладываемых в тегах [сode] [/сode] и [quоte] [/quоte].

[quazi]

только имхо: лучше - ограничивать количеством не символов, но строк
еще более имхо: лучше - сделать "прокручиваемые портянки" средствами CSS

[rxu]

только имхо: лучше - ограничивать количеством не символов, но строк

Спасибо, подумаю над этим.

еще более имхо: лучше - сделать "прокручиваемые портянки" средствами CSS

Неплохо, но целью было сделать максимально простое и максимально эффективное (хоть и грубоватое) средство

[Поручик]

rxu, можно добавить тоже самое для цитат.

[rxu]

Благодаря идеям quazi и Поручик-а, переписал мод до версии 1.3.0.
Здесь:
- работа кода сделана более универсальной;
- добавлена возможность обрезки цитат;
- добавлена возможность отключения обрезки путём установки нуля в строках с define (пока только вручную правкой кода).

Первая версия с обрезкой по количеству символов удалена за ненадобностью.

Добавлено спустя 2 часа 1 минуту 9 секунд:

Обнаружена опечатка в моде, исправлена, архив обновлён без смены версии.

[quazi]

rxu, предлагаю маленький тест на совместимость с 4 и 5 версиями - написал навскидку
не проверял, но должно работать

разрыв в строковых константах в строках 1 и 2 - умышленный, чтобы форматирование сработало (проявился недостаток обработки bb-тегов форума)

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

$limit_code  = $open_tag[0] == '[' . 'code]'  && CODE_MAX_LENGTH  > 0;
$limit_quote = $open_tag[0] == '[' . 'quote]' && QUOTE_MAX_LENGTH > 0;

if ( $limit_code || $limit_quote )
{
	$limit_lines = 1 + ( $limit_code ? CODE_MAX_LENGTH : QUOTE_MAX_LENGTH );
	$between_tags = implode("\n", 
			array_slice(
			explode("\n", $between_tags, $limit_lines), 0, -1));
}

Добавлено спустя 16 минут 4 секунды:

еще одно имхо

наверно XXX_MAX_LENGTH > 0 не самый лучший вариант как сигнал к обрезке
лучше наверно некоторое минимальное положительное число, оптимальное для большинства случаев
например 20-30 строк и делать проверки на превышение с этим числом

в качестве примера

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

$limit_code  = ...  && CODE_MAX_LENGTH  > 30;
$limit_quote = ... && QUOTE_MAX_LENGTH > 30;

[rxu]

наверно XXX_MAX_LENGTH > 0 не самый лучший вариант как сигнал к обрезке

Сделано просто для того, чтобы получить возможность отключать обрезку. Т.е. если предел равен false, обрезка не производится.

В любом случае, всё проверю - отпишусь, спасибо.

[quazi]

rxu
я вот тут на досуге подумал
допустим, у меня есть пост
[ code]
много
много
много
...
тут двадцатая строка
...
опять много
много
много
строк
[ /code]

именно так - открывающий и закрывающий теги, каждый на отдельной строке (хотя это совсем не принципиально)
и так - всего этих строк до полусотни, а у нас ограничение - не более 20 строк

и таким образом мы отсекаем все строки после 20-ой и получаем
[ code]
много
много
много
...
тут двадцатая строка

в результате мы теряем закрывающий тег

[rxu]

Вобщем код понятен, но в нём не вижу оснований для несовместимости с версиями 4 и 5
Видимо, поэтому сработал и там, и там.

Добавлено спустя 2 минуты 27 секунд:

в результате мы теряем закрывающий тег

Как мы его потеряем, если обрезаем только текст, находящийся между тегами?

[quazi]

Как мы его потеряем, если

сообразил.
вопрос снят

тем не менее, я склоняюсь к стилевому управлению.
это более мягкий и не на столько грубый способ