Баг при редактировании юзера в RUNCMS 1.4-1.6.1

[Linevo]

Наблюдаю косяк! При попытке редактировать юзера портала, получаю все его права! Т.е. редактируя его профайл становлюсь им же! В E-Xoops такого небыло! Как с этим бороться!???

[pan]

Я такое тоже замечал

[Jurist]

Подтверждаю... Имеет место быть... Что делать бум, славяне? Теперь же юзера не отредактировать! Жмем кнопку "сохранить", а учетная запись-то уже не админская! Выдает "Система не может обновить БД" и выкидывает на главную как редактируемого юзера..

[LARK]

Я пока могу тестировать 1.4 только на локалке. Создал нового юзера, при редактировании все нормально.

Но это на вновь установленном портале.

[Jurist]

у меня была конвертация БД... прыжок 1.2-1.4

[LARK]

В общем, это ошибка в /modules/system/admin/users/users.php

Причем сразу две ошибки.

Во-первых после обновления данных о юзере обновляются данные сессии. Но редиректы перепутаны и при обновлении данных выдается сообщение о невозможности обновить базу и, наоборот, при невозможности обновить данные выдается сообщение об успехе обновления.

Во-вторых, сама идея обновления сессии, честно говоря, не понятна и вызывает ошибку. Мы обновляем данные сессии админа и заносим туда данные обновленного юзера. Отсюда при редиректе мы становимся этим юзером.

Помимо этого обнаружил что начиная с версии 1.3 у таблицы runcms_session ключевым сделало поле time т.е. время начала сессии. Это, конечно не особо страшно, для малопосещаемых сайтов, но для сайтов с большим количеством посещений может привести к проблеме.

Лечение.

Вариант аварийный, естественно нужно делать более широкое исправление, но это пусть уже авторы делают.

Находим функцию updateUser

И начиная со строки 232 комментируем те строки, которые указаны во фрагменте ниже:

PHP код:

//$session = new XoopsUserSession();
    //$session->setUid($uid);
    //$session->setUname($uname);
    //$session->setPass($edituser->getVar('pass'));
    //$session->setSalt($edituser->getVar('pwdsalt'));
    //if (!$session->store())
    //{
    redirect_header("admin.php?fct=users",1,_UPDATED);
      exit();
    //}
      //redirect_header("../../index.php", 1, _NOTUPDATED);
  }
exit();
} 


Ниже прикреплен файлик с исправлением.

[Jurist]

Ларк, ты гений... Патчим, тестим.

[LARK]

К сожалению, с 2006 года этот баг так и не профиксили. Баг имеет место быть во всех версиях начиная с RUNCMS 1.3 и заканчивая RUNCMS 1.6.1. Лечение описано в предыдущем посте.

Так же всем пользователям RUNCMS 1.3 A - RUNCMS 1.6 (для 1.6.1 уже не надо), можно сделать следующее исправление:

Исправление ошибки "Система не может обновить БД" при авторизации