[BETA] myAJAX 0.6.6

xa0c · Сообщение **xa0c** » 21.04.2006 17:18

edgar писал(а):Не, 2.0.19 тогда была.

А... Просто у меня та же проблема есть, я думал из-за форума(он 2.0.15 был вчера)

[Fix] correctly preview signature if editing other users posts (Bug #101)

Сегодня обновил до 2.0.19 - то же самое...

edgar писал(а):Вы убрали, если админ редактирует свой собственный пост? Подскажете мне, как убрать?

Эээ... Не знаю насчет глючности... Я убрал надпись о редактировании админом любого(не только своего) поста:

		$sql = "UPDATE " . POSTS_TABLE . " SET
			post_edit_time = $current_time, post_edit_count = post_edit_count + 1 
			WHERE post_id = $post_id";
		if ( !$db->sql_query($sql) )
		{
			message_die(GENERAL_ERROR, 'Could not update post text', '', __LINE__, __FILE__, $sql);
		}

Перед этим поставить:

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

if ($userdata['user_level'] != ADMIN) {

, а после этого "}".

Если добавить дополнительный запрос к базе, то можно сделать как и при обычном редактировании постов: если пост юзера и пост последний, то сообщение о редактировании не пишется, если пост не последний, то пишется...

P.S. 2Автор. Зачем идёт куча if'ов на $mode?..
Сначала объявляем, что $mode='delete', а потОм идёт "if $mode != 'newtopic'", "if $mode == 'poll_delete'"... o.O

xa0c · Сообщение **xa0c** » 24.04.2006 2:44

Небольшая переделка того, что я предложил выше.
Если заменить в оригинальном коде это:

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

      $sql = "UPDATE " . POSTS_TABLE . " SET 
         post_edit_time = $current_time, post_edit_count = post_edit_count + 1 
         WHERE post_id = $post_id"; 
      if ( !$db->sql_query($sql) ) 
      { 
         message_die(GENERAL_ERROR, 'Could not update post text', '', __LINE__, __FILE__, $sql); 
      }

На это:

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

                $sql = "SELECT p.post_id, p.poster_id, p.topic_id, t.topic_id, t.topic_last_post_id
			FROM " . POSTS_TABLE . " p, " . TOPICS_TABLE . " t
			WHERE p.post_id = $post_id
				AND t.topic_id = p.topic_id";
                 if ( !$result = $db->sql_query($sql) )
		        {
                             message_die(GENERAL_ERROR, 'Could not anything :(', '', __LINE__, __FILE__, $sql);
		        }
		        
		$post_info = $db->sql_fetchrow($result);
		$db->sql_freeresult($result);
                $post_data['last_post'] = ( $post_info['topic_last_post_id'] == $post_id ) ? true : false;
                $post_data['poster_post'] = ( $post_info['poster_id'] == $userdata['user_id'] ) ? true : false;

                if ( $userdata['user_level'] != ADMIN && !$post_data['last_post'] && $post_data['poster_post'] ) {

                $sql = "UPDATE " . POSTS_TABLE . " SET
			post_edit_time = $current_time, post_edit_count = post_edit_count + 1 
			WHERE post_id = $post_id";

                        if ( !$db->sql_query($sql) )
		        {
                             message_die(GENERAL_ERROR, 'Could not update post text', '', __LINE__, __FILE__, $sql);
		        }
		        
                }

То сообщение о редактировании юзером своего же поста(если пост последний в теме) не показывается и сообщение о редактировании админом любого поста не показывается.

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

edgar писал(а):1. Если делаешь быстрое редактирование поста юзера, а у него подпись с тегами, то теги url перестают работать, а остальные становятся такими:

Я тут подумал... Если в файле viewtopic_body.tpl убрать код

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

<div id="p_{postrow.U_POST_ID}_sig" style="display: inline;"></div>

(То, что между этими тегами оставить

) - то всё ок будет. Долго думал - зачем это сделано, но разумного объяснения не нашел.

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

edgar · Сообщение **edgar** » 24.04.2006 9:54

xa0c писал(а): сообщение о редактировании юзером своего же поста(если пост последний в теме) не показывается

Так оно если ничего и не менять не показывается. Так в phpBB изначально. Мне нужно было только одно сделать - админ или модер правят свой пост - сообщение о редактировании не показывается.

xa0c

Xpert писал(а):К сожалению, в нынешнем состоянии мод может навредить форуму.

Поэтому я решил подождать когда мод будет доработан

xa0c · Сообщение **xa0c** » 24.04.2006 14:55

edgar писал(а):Так оно если ничего и не менять не показывается. Так в phpBB изначально

Угу. Только если использовать данный мод(без предолженного исправления), то сообщение о редактировании будет показываться всегда.

Alf · Сообщение **Alf** » 27.04.2006 14:05

вот несколько напрягает, еще то что при удалении в Осле пост не исчезает, хотя в Лисе всё работает хорошо.

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

Хотя может у вас всё нормально - подскажите пути решения

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

И кстати, довольно быстро привыкаешь к удобным функциям - начинаешь дважды жать во всех форумах теперь. хихи

Lanior · Сообщение **Lanior** » 30.04.2006 11:05

Выпустил весию 0.3.4.
Все дружно качаем файл из первого поста

Alf писал(а):вот несколько напрягает, еще то что при удалении в Осле пост не исчезает

Эта ошибка исправлена.

NCom · Сообщение **NCom** » 30.04.2006 12:27

Lanior
это ненормально.. может ты все таки файлики для обновления сделаешь.

xa0c · Сообщение **xa0c** » 30.04.2006 15:25

Поддерживаю NCom'a. Я до сих пор с первой версией сижу, только добавил некоторые изменения от себя, ибо влом вручную диффать

Lanior · Сообщение **Lanior** » 01.05.2006 4:40

C 0.3.1 до 0.3.4 обновление есть, лежит в корне архива.

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

Добавил в архив update с версии 0.2

NCom · Сообщение **NCom** » 01.05.2006 18:37

Lanior спасибо. Вот это отлично

А что нового? С удаление темы ничего не решено?

Lanior · Сообщение **Lanior** » 02.05.2006 1:01

Дв вроде все решено, я тестил, багу больше повторить не удалось.

Alf · Сообщение **Alf** » 08.05.2006 21:34

Lanior
О дело, спасибо за коррективы и файлы обновлений.
Респект за очень удобный мод.

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

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

Хы, коррективы не помогли лично мне решить проблему в ИЕ
В Осле - тэги с идентификатор id также не определяются функцией getElementsByName, хотя поидее должны может у меня эксплорер какой-то не такой? IE 6.0.2900
Ни в какую не хочет.

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

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

	req.onreadystatechange = function() {
		if (req.readyState == 4) {
			if (req.responseJS) {
				var elements = document.getElementsByName('post_' + post);
				for (var i = 0; i < elements.length; i++) {
					elements[i].innerHTML = '';
				}
			}
	        }
	}

У меня в эксплорере почему-то вот эта часть скрипта вообще не работала. Если выносить из условия if (req.responseJS) тело за пределы определения функции, что в принципе не правильно, т.к. не факт, что пост будет удален (ведь возможно у пользователя не окажется прав для этого, ну тогда и кнопки быть видно не должно с удалением, а если он догадался напрямую вызвать этот жаба скрипт, то пусть не жалуется что какая-то фигня происходит). То эксплорер обламывает выполнение из-за вот этого elements.innerHTML ну не хочет он присваивать значение, если свойство определяется таким способом, зато сработало - elements.style.display = 'none'; и даже выглядит по симпатичнее результат, т.к. при использовании innerHTML - тег хоть и пустой но остается оставляя однопиксельную белую полосу (лично у меня так было).
Итого конечный вариант функции удаления поста:

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

function dp(post) { // Delete Post
	var answer = confirm("Вы действительно хотите удалить это сообщение?");

	if (answer == 0) {
		return;
	}

    var req = new Subsys_JsHttpRequest_Js();

    var elements = document.getElementsByName('post_' + post);
    for (var i = 0; i < elements.length; i++) {
    	elements[i].style.display = 'none';
    }

	req.caching = false;
	req.open('POST', 'ajax.php?mode=deletepost', true);
	req.send({ p: post });
}

п.с. Обломно тока что чередование цветов строк постов не учитывается, хотя если поизвращаться - можно и это сделать через ЖабуСкрипт. хихи

NCom · Сообщение **NCom** » 08.05.2006 22:33

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

ОБЕИМИ РУКАМИ ЗА.
я об этом не подумал!!! Великолепный способ экономии трафа.
Есть возможность сделать это?

Lanior · Сообщение **Lanior** » 09.05.2006 6:13

Alf писал(а):В Осле - тэги с идентификатор id также не определяются функцией getElementsByName, хотя поидее должны может у меня эксплорер какой-то не такой? IE 6.0.2900

Как ни странно, Осел возвражает не теги с нужным name, а теги id. Проверите, обновили ли вы шаблон.

Alf писал(а):elements.style.display = 'none';

Спасибо, приму к сведению.

Alf писал(а):Обломно тока что чередование цветов строк постов не учитывается

Сделаю.

Alf писал(а):Как насчет модификации QuickReply с помощью Аджакс

Попробую сделать.

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

Новое в версии 0.3.5:
Теперь класс ячейки ответа обновляется после быстрого удаления.
Строки из JavaScript вынесены в языковые переменные.

Alf · Сообщение **Alf** » 09.05.2006 15:40

2 Lanior, конечно я обновил шаблон

. Да и функция работает в Осле как надо, но в чем был трабл я так и не понял, хотя по спецификации идентификатор id должен быть уникальным, а соответственно применение к нему getElementsByName точно не должно работать. Но это уже другая тема.

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