Вышел phpBB 3.1.10 [обсуждаем]

[rxu]

Обсуждаем новости о выходе phpBB 3.1.10 и phpBB 3.1.10-RC1.

[3.1] Изменения в стилях в версии 3.1.10

[Allweb]

Подскажите, в чем проблема.
Поставил чистый 3.1.9, все работало нормально. Обновил вчера на 3.1.10 - перестали работать настройки дополнительных полей в профиле. Т.е. при нажатии на кнопку настроек любого доп. поля, выскакивает ошибка:
Ошибка отправки формы. Попробуйте ещё раз.
Не работает только настройка, поле можно удалить и добавить.

Вернул обратно 3.1.9, там все нормально, никаких изменений не вносилось.

[nissin]

Проверил, баг подтверждается.
Временное исправление:
OPEN includes/acp/acp_profile.php
FIND

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

				if (!sizeof($error))
				{
					if (!check_form_key($form_key))
					{
						trigger_error($user->lang['FORM_INVALID'] . adm_back_link($this->u_action), E_USER_WARNING);
					}

					if (($step == 3 && (sizeof($this->lang_defs['iso']) == 1 || $save)) || ($action == 'edit' && $save))
					{
						$this->save_profile_field($cp, $field_type, $action);
					}
				}

REPLACE WITH

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

				if (!sizeof($error))
				{
					if (!check_form_key($form_key) && $submit)
					{
						trigger_error($user->lang['FORM_INVALID'] . adm_back_link($this->u_action), E_USER_WARNING);
					}

					if (($step == 3 && (sizeof($this->lang_defs['iso']) == 1 || $save)) || ($action == 'edit' && $save))
					{
						$this->save_profile_field($cp, $field_type, $action);
					}
				}

[southklad]

Смотрю сыпется новая версия вся

[Татьяна5]

Однако да, сыпется...
Установила сторонний стиль "по-умолчанию". "Заменять стиль пользователям" было установлено в "да", но стиль заменился только для текущего пользователя. Активировала и деактивировала prosilver - в результате для всех, кроме текущего пользователя и гостя, id стиля сбросилось в 0

[KimIV]

А я уж было тряпку красную на форуме вывесил. На субботу назначил обновление. Но теперь подожду...

[southklad]

Что то что в предпоследней версии были баги, а в новой их еще больше

[Татьяна5]

Одно чинят, с десяток при этом ломают. Обычное дело, причём для кого угодно
Будем ждать пока починят более-менее всё

[Татьяна5]

Быстрый fix для тех, у кого не устанавливаются расширения:

Вариант исправления ошибки "Обязательный модуль не существует" 

\phpbb\db\migration\tool\module.php
Найти

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

$parent = $this->get_parent_module_id($parent, $module

Добавить после (в строке)

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

, 'exists'

Найти

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

public function get_parent_module_id($parent_id, $data = ''

Добавить после (в строке)

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

, $mode = false

Найти

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

throw new \phpbb\db\migration\exception('MODULE_NOT_EXIST', $parent_id);

Заменить на

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

if ($mode === 'exists')
					{
						$sql = 'SELECT module_id
 							FROM ' . $this->modules_table . "
 							WHERE module_langname = '" . $this->db->sql_escape($parent);
 						$result = $this->db->sql_query($sql);
						$parent_id = $this->db->sql_fetchfield('module_id');
						$this->db->sql_freeresult($result);
					}
					else
					{
						throw new \phpbb\db\migration\exception('MODULE_NOT_EXIST', $parent_id);
					}

[Sheer]

Татьяна5, тикет уже в репо, только проще https://github.com/phpbb/phpbb/pull/4491/files

[Татьяна5]

Sheer, то от другой ошибки, видимо

Убрала свои правки, добавила те, попробовала установить liveSearch, и снова ошибка:

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

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

Обязательный модуль не существует: ACP_LIVE_SEARCH

Ошибка там тянется из метода exists, не из add

[Sheer]

Татьяна5, liveSearch не пробовал, вот Post Bookmarks и Spamer Finder с правками rxu устанавливаются и удаляются.

[Татьяна5]

Sheer, там другой тип ошибки, не добавлялись модули
А в liveSearch виновата проверка на существование модуля

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

			// Remove old ACP modules
			array('if', array(
				array('module.exists', array('acp', 'ACP_LIVE_SEARCH', array(
					'module_basename'	=> '\alg\liveSearch\acp\acp_live_search_module',
					'module_langname'	=> 'ACP_LIVE_SEARCH_SETTINGS',
					'module_mode'		=> 'live_search',
					'module_auth'		=> 'ext_alg/liveSearch && acl_a_board',
					),
				)),
				array('module.remove', array('acp', 'ACP_LIVE_SEARCH', array(
					'module_basename'	=> '\alg\liveSearch\acp\acp_live_search_module',
					'module_langname'	=> 'ACP_LIVE_SEARCH_SETTINGS',
					'module_mode'		=> 'live_search',
					'module_auth'		=> 'ext_alg/liveSearch && acl_a_board',
					),
				)),
			)),

module.php, метод exists

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

		if ($parent !== false)
		{
			$parent = $this->get_parent_module_id($parent, $module);
			$parent_sql = 'AND parent_id = ' . (int) $parent;
		}

Тут $parent = 'ACP_LIVE_SEARCH'

Метод get_parent_module_id

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

	public function get_parent_module_id($parent_id, $data = '')
	{
		// Allow '' to be sent as 0
		$parent_id = $parent_id ?: 0;

		// If automatic adding is in action, convert array back to string to simplify things
		if (is_array($data) && sizeof($data) == 1)
		{
			$data = $data['module_langname'];
		}

		if (!is_numeric($parent_id))
		{
			// Refresh the $module_categories array
			$this->get_categories_list();

			// Search for the parent module_langname
			$ids = array_keys($this->module_categories, $parent_id);

			switch (sizeof($ids))
			{
				// No parent with the given module_langname exist
				case 0:
					throw new \phpbb\db\migration\exception('MODULE_NOT_EXIST', $parent_id);
				break;

И на этом вылетает исключение, что модуля нет (но это проверка на существование, а не добавление дочернего модуля; исключения быть не должно)

[Sheer]

Да я уже понял. Руслан разберется.

[rxu]

Обновил пулл реквест, проверяйте.

[Татьяна5]

Работает
liveSearch устанавливается