[BETA] NOEdit MOD 0.4

[Никто]

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

[Xpert]

Порка, сэр!

## MOD Author: Идиятов Сергей

Фамилие говорящее, я вам скажу А вообще, там ник пользователя указывается, но никак не ФИО. Шаблон модов изучил бы.

message_die(GENERAL_ERROR, 'Вы не можете убирать пометку о редактировании');

Этой проверке стоит быть в posting.php, кроме того, почему не языковая переменная?
Кстати, в случае соблюдения вышесказанного достаточно будет просто расширить условие формирования $edited_sql

$noedit = NULL;

1. Зачем, этой переменной вообще не нужно исходное значение.
2. Почему NULL?

if ( $userdata['user_level'] == ADMIN )
{
$template->assign_block_vars('switch_noedit', array());
}

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

Мод не ставил, просто по коду говорю...
Английский и описание мода не впечатлили.

[Никто]

Фамилие говорящее, я вам скажу А вообще, там ник пользователя указывается, но никак не ФИО. Шаблон модов изучил бы.

Это не столь важно.

Этой проверке стоит быть в posting.php, кроме того, почему не языковая переменная?
Кстати, в случае соблюдения вышесказанного достаточно будет просто расширить условие формирования $edited_sql

Ок, в следующей версии учту.

1. Зачем, этой переменной вообще не нужно исходное значение.

Дабы определить переменную

2. Почему NULL?

NULL - пусто все равно что определить ее как ''

Как насчет дополнительной проверки на то, является ли именно данный пользователь автором сообщения

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

редактирует ли он вообще сообщение

Если бы ты глянул куда мы добавляем кусок, ты бы понял что проверка уже есть

Мод не ставил, просто по коду говорю...

Поставь

Английский и описание мода не впечатлили.

Английский у меня нулевой.
Дальше?

[Xpert]

Дабы определить переменную

А зачем? В posting.php просто проверяем, есть ли такая переменная. Если есть и админ - то используем ее значение, иначе нет.

все равно что определить ее как

Шикарный смайлик. Вот только NULL как-раз таки означает неопределенность переменной.

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

Ответ неверный. Мод позволяет админу не оставлять пометки о редактировании или не увеличивать счетчик. Естественно, только для своих сообщений, поскольку во всех остальных случаях он и так не появится/не увеличится.

Если бы ты глянул куда

А я и глянул... Как думаешь, зачем чуть выше твоего кода вот такие проверки?

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

if ( $mode == 'newtopic' || ( $mode == 'editpost' && $post_data['first_post'] ) )
{
	$template->assign_block_vars('switch_type_toggle', array());

Поставь

Ну если еще и поставлю, то тебе точно несдобровать

[Никто]

А зачем? В posting.php просто проверяем, есть ли такая переменная. Если есть и админ - то используем ее значение, иначе нет.

Спасибо за подсказку, в след версии учту

Вот только NULL как-раз таки означает неопределенность переменной.

Тем лучше

Ответ неверный. Мод позволяет админу не оставлять пометки о редактировании или не увеличивать счетчик. Естественно, только для своих сообщений, поскольку во всех остальных случаях он и так не появится/не увеличится.

Это я и имел ввиду

А я и глянул... Как думаешь, зачем чуть выше твоего кода вот такие проверки?

Ой незаметил... Скоро будет новая версия.

Ну если еще и поставлю, то тебе точно несдобровать

Ничего, зато мод нормальный будет.

[Mr. Anderson]

мод нормальный будет

Написанный под диктовку Xpert'а

[Никто]

Mr. Anderson
К этому моду эксперт не имеет никакого отношения, а рекомендованные изменения ещё не внесены.

[Никто]

Обновляемся =)
Изменения:

• Удалено определение переменной $noedit
• Добавлена проверка на режим правки
• Исравлена шапка мода
• В ошибке теперь языковая переменная.

[Xpert]

Проверку админ или нет надо выносить в posting.php сразу после получения переменной. Тогда и вот этот код и соответствующая языковая переменная становятся ненужными.

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

	elseif ( ($userdata['user_level'] != ADMIN) && ($noedit == true) )
	{
		message_die(GENERAL_ERROR, $lang['noedit_cant']);
	}

This hack allow check noedit flag for admin to not atach the "edited by" message.

Я кажется объяснил что мод делает. Или как в том анекдоте "сейчас запустим написанное и узнаем что это"?

[Никто]

Проверку админ или нет надо выносить в posting.php сразу после получения переменной. Тогда и вот этот код и соответствующая языковая переменная становятся ненужными.

Имхо - это излишне

Я кажется объяснил что мод делает. Или как в том анекдоте "сейчас запустим написанное и узнаем что это"?

Ну скажи как это будет по английски правильно...

[Xpert]

Имхо - это излишне

Хм, в phpBB большая часть проверок сделана именно так, посмотри к примеру topic_type...

[Никто]

Xpert
Мне кажется, что здесь более уместно так. Это все-таки не топик тайп
2) Как все-таки это по англ будет?

[Никто]

Мод отправлен на phpbb.com
Обещали зарелизить скоро. MOD-TEAM обещали валидейтнуть

[vlad77]

Что бы модераторы тоже могли оставлять/неоставлять информацию о правке.

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

открыть 
includes/functions_post.php

найти
	if ( ($userdata['user_level'] == ADMIN) && ($noedit == true) )
	{
		$edited_sql = '';
	}
	elseif ( ($userdata['user_level'] != ADMIN) && ($noedit == true) )
	{
		message_die(GENERAL_ERROR, $lang['noedit_cant']);
	}
	else
	{
		$edited_sql = ($mode == 'editpost' && !$post_data['last_post'] && $post_data['poster_post']) ? ", post_edit_time = $current_time, post_edit_count = post_edit_count + 1 " : "";
	}

заменить
	if ( ( in_array($userdata['user_level'], array (MOD, ADMIN) ) ) && ($noedit == true) )
	{
		$edited_sql = '';
	}
	elseif ( ($userdata['user_level'] != ADMIN && $userdata['user_level'] != MOD) && ($noedit == true) )
	{
		message_die(GENERAL_ERROR, $lang['noedit_cant']);
	}
	else
	{
		$edited_sql = ($mode == 'editpost' && !$post_data['last_post'] && $post_data['poster_post']) ? ", post_edit_time = $current_time, post_edit_count = post_edit_count + 1 " : "";
	}

открыть
posting.php

найти
//
// NOEDIT
//
if ( $mode == 'editpost' )
{
	if ( $userdata['user_level'] == ADMIN )
	{
		$template->assign_block_vars('switch_noedit', array());
	}
}

заменить
//
// NOEDIT
//
if ( $mode == 'editpost' )
{
	if ( in_array($userdata['user_level'], array (MOD, ADMIN) ) )
	{
		$template->assign_block_vars('switch_noedit', array());
	}
}

Можно удалить информацию о правке пользователями всех своих сообщений, выполнив SQL запрос

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

update phpbb_posts
set post_edit_time = NULL
, post_edit_count = 0
where post_edit_count > 0;