[Отделено][dev] Thanks for posts

[LavIgor]

Хм... никаких изменений в ядре в отношении этого свойства не заметил.

Нашёл.
https://github.com/phpbb/phpbb/commit/b ... 65b8f2071a

Удалили $notifications_table, оставили $user_notifications_table.
Т.е. отделили внутрифорумные уведомления от уведомлений в целом.

Нужно думать, какую из двух таблиц всё же нужно использовать в данном случае.

Raisin, да, как оказалось, этот способ не поможет.
Тут нужно много чего переделывать, получается...
Разве что можно временный фикс сделать, использовав константу NOTIFICATIONS_TABLE.

Отправлено спустя 1 минуту 28 секунд:
Raisin, исправленный вариант: в тех строчках замените FROM ' . $this->notifications_table . ' на FROM ' . NOTIFICATIONS_TABLE . '.

[Slash]

Блин... Сейчас та все поля существуют, но ошибка все ровно есть:

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

You have an error in your SQL syntax; check the manual that corresponds to your MySQL server version for the right syntax to use near 'WHERE notification_type_id = 10 AND item_id = 2' at line 3 [1064]

SQL

UPDATE phpbb_notifications SET WHERE notification_type_id = 10 AND item_id = 2

BACKTRACE

Отправлено спустя 1 минуту 31 секунду:
Только сейчас заметил, что в сообщении ошибке запрос сменился с SELECT на UPDATE. Что то вообще сверх естественное...

[LavIgor]

BACKTRACE можно?

Отправлено спустя 2 минуты 35 секунд:
Сейчас тогда себе поставлю на 3.2.

Отправлено спустя 2 минуты 58 секунд:
У меня пока что и вовсе не ставится.
Потому что SQLite.

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

General Error

SQL ERROR [ sqlite3 ]

near "t": syntax error [1]

SQL

UPDATE phpbb_thanks t LEFT JOIN phpbb_posts p ON t.post_id = p.post_id SET t.forum_id = p.forum_id, t.topic_id = p.topic_id WHERE t.post_id = p.post_id

Пытаюсь исправить ситуацию.

[rxu]

По ходу пьесы, уведомление для совместимости с 3.2 придется переписывать чуть более, чем полностью.

Отправлено спустя 6 минут 40 секунд:
Хотя, похоже есть другой способ.

[Slash]

Хотя, похоже есть другой способ.

Что за способ?

[LavIgor]

Установилось только с такой правкой:

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

	public function update_thanks_table()
	{
		$thanks_table = $this->table_prefix . 'thanks';

		$sql = 'UPDATE '.$thanks_table.'
			SET forum_id = (
			SELECT p.forum_id FROM '.POSTS_TABLE.' p
			WHERE '.$thanks_table.'.post_id = p.post_id
			), topic_id = (
			SELECT p.topic_id FROM '.POSTS_TABLE.' p
			WHERE '.$thanks_table.'.post_id = p.post_id
			)';
		$this->db->sql_query($sql);
	}

Да, не так много возможностей у SQLite.

Отправлено спустя 4 минуты 39 секунд:
Хм... у меня без всяких правок работает.
Спасибка и ставится, и снимается.
phpBB 3.2.0-a3-dev.

[rxu]

Что за способ?

Попробуйте перекачать архив заново и обновить расширение.

[LavIgor]

Видимо, помог этот фикс.

[rxu]

Спасибка и ставится, и снимается.

Вопрос в том, что будет, если второй пользователь скажет спасибо.

[LavIgor]

rxu, да, сейчас уже не работает.
Обновил архив.

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

General Error

SQL ERROR [ sqlite3 ]

near "WHERE": syntax error [1]

[rxu]

А весь запрос как выглядит?

Отправлено спустя 32 секунды:
Надо стандартным способом обновляться, есть изменение в сервисах. Или чистить кэш.

[Slash]

Перезалил полностью расширение. Тестирую от админа все нормально, затем от пользователя (id ) получаю ошибку, а потом и от админа получил ошибку:

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

SQL ERROR [ mysql4 ]

You have an error in your SQL syntax; check the manual that corresponds to your MySQL server version for the right syntax to use near 'WHERE notification_type_id = 11 AND item_id = 2' at line 3 [1064]

SQL

UPDATE phpbb_notifications SET WHERE notification_type_id = 11 AND item_id = 2

Обновляю страницу, сообщение:

Некорректные параметры запрошенного действия

[LavIgor]

Он не показан.
Сделал var_dump().

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

string 'SELECT notification_data
			FROM phpbb_notifications
			WHERE notification_type_id = 19
				AND item_id = 2' (length=107)

Возможно, снова косяки SQLite, но на вид запрос корректный.

Надо стандартным способом обновляться

Обновлять расширение? Движок ставил с нуля.

Или чистить кэш.

Чистил и проверял.

[Slash]

Чистил и проверял.

Я тоже...

[LavIgor]

Похоже, что этот WHERE где-то в другом месте не работает.
Но никакого BACKTRACE нет.