[mxbb-commits] core/modules/mx_navmenu/admin mx_module_defs.php, 1.31, 1.32

SourceForge Headquarters 1320 Columbia Street Suite 310 San Diego, CA 92101 +1 (858) 422-6466

Update of /cvsroot/mxbb/core/modules/mx_navmenu/admin
In directory sc8-pr-cvs16.sourceforge.net:/tmp/cvs-serv28768/modules/mx_navmenu/admin

Modified Files:
	mx_module_defs.php 
Log Message:
part of project wide change of using request_vars wrapper class instead of explicite and unfiltered $HTML_*_VARS

Index: mx_module_defs.php
===================================================================
RCS file: /cvsroot/mxbb/core/modules/mx_navmenu/admin/mx_module_defs.php,v
retrieving revision 1.31
retrieving revision 1.32
diff -C2 -d -r1.31 -r1.32
*** mx_module_defs.php	6 Feb 2008 23:44:02 -0000	1.31
--- mx_module_defs.php	9 Feb 2008 12:49:19 -0000	1.32
***************
*** 82,86 ****
  	function display_panel_nav_menu( $parameter_data, $block_id )
  	{
! 		global $template, $board_config, $db, $theme, $lang, $images, $mx_blockcp, $mx_root_path, $HTTP_POST_VARS, $HTTP_COOKIE_VARS, $userdata, $mx_request_vars, $dynamic_block_id, $portalpage, $mx_cache, $phpEx;

  		//
--- 82,86 ----
  	function display_panel_nav_menu( $parameter_data, $block_id )
  	{
! 		global $template, $board_config, $db, $theme, $lang, $images, $mx_blockcp, $mx_root_path, $HTTP_COOKIE_VARS, $userdata, $mx_request_vars, $dynamic_block_id, $portalpage, $mx_cache, $phpEx;

  		//
***************
*** 128,134 ****
  		// Parameters
  		//
! 		$submit = ( isset($HTTP_POST_VARS['submit']) ) ? true : false;
! 		$cancel = ( isset($HTTP_POST_VARS['cancel']) ) ? true : false;
! 		$preview = ( isset($HTTP_POST_VARS['preview']) ) ? true : false;
  		$refresh = $preview || $submit_search;

--- 128,134 ----
  		// Parameters
  		//
! 		$submit = $mx_request_vars->is_post('submit');
! 		$cancel = $mx_request_vars->is_post('cancel');
! 		$preview = $mx_request_vars->is_post('preview');
  		$refresh = $preview || $submit_search;

***************
*** 701,705 ****
  	function display_panel_site_menu( $parameter_data, $block_id )
  	{
! 		global $template, $board_config, $db, $theme, $lang, $images, $mx_blockcp, $mx_root_path, $HTTP_POST_VARS, $HTTP_COOKIE_VARS, $userdata, $mx_request_vars, $dynamic_block_id, $portalpage, $mx_cache, $phpEx;

  		$mx_page = new mx_page();
--- 701,705 ----
  	function display_panel_site_menu( $parameter_data, $block_id )
  	{
! 		global $template, $board_config, $db, $theme, $lang, $images, $mx_blockcp, $mx_root_path, $HTTP_COOKIE_VARS, $userdata, $mx_request_vars, $dynamic_block_id, $portalpage, $mx_cache, $phpEx;

  		$mx_page = new mx_page();
***************
*** 744,750 ****
  		// Parameters
  		//
! 		$submit = ( isset($HTTP_POST_VARS['submit']) ) ? true : false;
! 		$cancel = ( isset($HTTP_POST_VARS['cancel']) ) ? true : false;
! 		$preview = ( isset($HTTP_POST_VARS['preview']) ) ? true : false;
  		$refresh = $preview || $submit_search;

--- 744,750 ----
  		// Parameters
  		//
! 		$submit = $mx_request_vars->is_post('submit');
! 		$cancel = $mx_request_vars->is_post('cancel');
! 		$preview = $mx_request_vars->is_post('preview');
  		$refresh = $preview || $submit_search;

***************
*** 1153,1157 ****
  	function _do_insert($type, $id )
  	{
! 		global $mx_root_path, $phpbb_root_path, $template, $lang, $db, $board_config, $theme, $phpEx, $HTTP_GET_VARS, $HTTP_POST_VARS, $userdata, $mx_request_vars;

  		switch ( $type )
--- 1153,1157 ----
  	function _do_insert($type, $id )
  	{
! 		global $mx_root_path, $phpbb_root_path, $template, $lang, $db, $board_config, $theme, $phpEx, $userdata, $mx_request_vars;

  		switch ( $type )
***************
*** 1161,1165 ****
  				if ( !MX_PANEL_DEBUG )
  				{
! 					if ( empty( $HTTP_POST_VARS['cat_title'] ) )
  					{
  						mx_message_die( GENERAL_ERROR, $lang['error_no_field'] );
--- 1161,1165 ----
  				if ( !MX_PANEL_DEBUG )
  				{
! 					if ($mx_request_vars->is_empty_post('cat_title'))
  					{
  						mx_message_die( GENERAL_ERROR, $lang['error_no_field'] );
***************
*** 1236,1240 ****
  				if ( !MX_PANEL_DEBUG )
  				{
! 					if ( empty( $HTTP_POST_VARS['menuname'] ) )
  					{
  						mx_message_die( GENERAL_ERROR, $lang['error_no_field'] );
--- 1236,1240 ----
  				if ( !MX_PANEL_DEBUG )
  				{
! 					if ($mx_request_vars->is_empty_post('menuname'))
  					{
  						mx_message_die( GENERAL_ERROR, $lang['error_no_field'] );
***************
*** 1301,1306 ****
  						'" . $mx_request_vars->post('menu_alt_icon_hot', MX_TYPE_NO_TAGS | MX_TYPE_SQL_QUOTED, '') . "',
  						'" . $mx_request_vars->post('auth_view', MX_TYPE_INT, '') . "',
! 						'" . $HTTP_POST_VARS['auth_view_group'] . "',
! 						'" . $HTTP_POST_VARS['link_target'] . "' )";

  					if( !($result = $db->sql_query($sql)) )
--- 1301,1306 ----
  						'" . $mx_request_vars->post('menu_alt_icon_hot', MX_TYPE_NO_TAGS | MX_TYPE_SQL_QUOTED, '') . "',
  						'" . $mx_request_vars->post('auth_view', MX_TYPE_INT, '') . "',
! 						'" . $mx_request_vars->post('auth_view_group', MX_TYPE_NO_TAGS | MX_TYPE_SQL_QUOTED, '') . "',
! 						'" . $mx_request_vars->post('link_target', MX_TYPE_NO_TAGS | MX_TYPE_SQL_QUOTED, '') . "' )";

  					if( !($result = $db->sql_query($sql)) )
***************
*** 1328,1332 ****
  	function _do_update($type, $id )
  	{
! 		global $mx_root_path, $phpbb_root_path, $template, $lang, $db, $board_config, $theme, $phpEx, $HTTP_GET_VARS, $HTTP_POST_VARS, $userdata, $mx_request_vars, $mx_cache;

  		switch ( $type )
--- 1328,1332 ----
  	function _do_update($type, $id )
  	{
! 		global $mx_root_path, $phpbb_root_path, $template, $lang, $db, $board_config, $theme, $phpEx, $userdata, $mx_request_vars, $mx_cache;

  		switch ( $type )
***************
*** 1336,1340 ****
  				if ( !MX_PANEL_DEBUG )
  				{
! 					if ( empty( $HTTP_POST_VARS['cat_title'] ) )
  					{
  						mx_message_die( GENERAL_ERROR, $lang['error_no_field'] );
--- 1336,1340 ----
  				if ( !MX_PANEL_DEBUG )
  				{
! 					if ($mx_request_vars->is_empty_post('cat_title'))
  					{
  						mx_message_die( GENERAL_ERROR, $lang['error_no_field'] );
***************
*** 1388,1392 ****
  				if ( !MX_PANEL_DEBUG )
  				{
! 					if ( empty( $HTTP_POST_VARS['menuname'] ) )
  					{
  						mx_message_die( GENERAL_ERROR, $lang['error_no_field'] );
--- 1388,1392 ----
  				if ( !MX_PANEL_DEBUG )
  				{
! 					if ($mx_request_vars->is_empty_post('menuname'))
  					{
  						mx_message_die( GENERAL_ERROR, $lang['error_no_field'] );
***************
*** 1442,1446 ****
  				if ( !MX_PANEL_DEBUG )
  				{
! 					if ( empty( $HTTP_POST_VARS['menuname'] ) && empty( $HTTP_POST_VARS['cat_title'] ))
  					{
  						mx_message_die( GENERAL_ERROR, $lang['error_no_field'] );
--- 1442,1446 ----
  				if ( !MX_PANEL_DEBUG )
  				{
! 					if ($mx_request_vars->is_empty_post('menuname') && $mx_request_vars->is_empty_post('cat_title'))
  					{
  						mx_message_die( GENERAL_ERROR, $lang['error_no_field'] );
***************
*** 1464,1468 ****
  					// Encode for db storage
  					//
! 					if ( empty( $HTTP_POST_VARS['cat_title'] ))
  					{
  						$menu_title = $mx_text->encode_simple($mx_request_vars->post('menuname', MX_TYPE_NO_STRIP | MX_TYPE_SQL_QUOTED, ''));
--- 1464,1468 ----
  					// Encode for db storage
  					//
! 					if ($mx_request_vars->is_empty_post('cat_title'))
  					{
  						$menu_title = $mx_text->encode_simple($mx_request_vars->post('menuname', MX_TYPE_NO_STRIP | MX_TYPE_SQL_QUOTED, ''));
***************
*** 1510,1514 ****
  	function _do_delete($type, $id, $parent, $recache )
  	{
! 		global $template, $lang, $db, $board_config, $theme, $phpEx, $HTTP_GET_VARS, $HTTP_POST_VARS, $mx_request_vars;

  		switch ( $type )
--- 1510,1514 ----
  	function _do_delete($type, $id, $parent, $recache )
  	{
! 		global $template, $lang, $db, $board_config, $theme, $phpEx, $mx_request_vars;

  		switch ( $type )
***************
*** 1599,1603 ****
  	function _do_move($type, $id)
  	{
! 		global $template, $lang, $db, $board_config, $theme, $phpEx, $HTTP_GET_VARS, $HTTP_POST_VARS, $mx_request_vars;

  		switch ( $type )
--- 1599,1603 ----
  	function _do_move($type, $id)
  	{
! 		global $template, $lang, $db, $board_config, $theme, $phpEx, $mx_request_vars;

  		switch ( $type )