From: Eloi G. <ada...@us...> - 2004-10-03 03:31:24
|
Update of /cvsroot/phpwsbb/phpwsbb/class In directory sc8-pr-cvs1.sourceforge.net:/tmp/cvs-serv2553/class Modified Files: Manager.php Log Message: Fixed Bug Report [1038110] "1.0.0 CVS: Edit other user's settings" Index: Manager.php =================================================================== RCS file: /cvsroot/phpwsbb/phpwsbb/class/Manager.php,v retrieving revision 1.53 retrieving revision 1.54 diff -C2 -d -r1.53 -r1.54 *** Manager.php 3 Oct 2004 02:47:48 -0000 1.53 --- Manager.php 3 Oct 2004 03:31:13 -0000 1.54 *************** *** 666,669 **** --- 666,670 ---- $form->add('module', 'hidden', 'phpwsbb'); + $form->add('user', 'hidden', $id); $form->add('PHPWSBB_MAN_OP', 'hidden', 'saveusersettings'); *************** *** 760,770 **** } } ! ! if($GLOBALS['core']->sqlUpdate($data, 'mod_phpwsbb_user_info', 'user_id', $_SESSION['OBJ_user']->user_id)) { ! // This had to be moved to the user settings table. ! // Use of setUserVar creates too many SELECTs to retrieve the values ! // ex: #queries = #users * #variables ! // Using a summary table instead. ! // if( $_SESSION['OBJ_user']->setUserVar('monitordefault', $monitordefault, NULL, 'phpwsbb') && $_SESSION['OBJ_user']->setUserVar('suspendmonitors', $suspendmonitors, NULL, 'phpwsbb')) { $title = $_SESSION['translate']->it('Settings Saved'); $content = $this->_menu() . $_SESSION['translate']->it("Your settings have been successfully saved."); --- 761,766 ---- } } ! ! if($GLOBALS['core']->sqlUpdate($data, 'mod_phpwsbb_user_info', 'user_id', $_POST['user'])) { $title = $_SESSION['translate']->it('Settings Saved'); $content = $this->_menu() . $_SESSION['translate']->it("Your settings have been successfully saved."); |