Prevent core pref corruption by old plugins that are using save_prefs();

Author: CaMer0n

class2.php

@@ -1717,10 +1717,31 @@ function save_prefs($table = 'core', $uid = USERID, $row_val = '')
 		e107::getMessage()->addDebug('Deprecated save_prefs() backtrace:<pre>'."\n".print_r($backtrace, true).'</pre>');
 	}
 
+
+
+
 	switch($table)
 	{
 		case 'core':
 			//brute load, force update
+
+			if(count($pref) < 100) // precaution for old plugins
+			{
+				$backtrace = debug_backtrace(false);
+
+				e107::getAdminLog()->add(
+				'Core pref corruption avoided',
+				"Call to deprecated function save_prefs() (class2.php) with too few prefs. Backtrace:\n".print_r($backtrace, true),
+				E_LOG_INFORMATIVE,
+				'DEPRECATED'
+				);
+
+
+				e107::getMessage()->addDebug('Core-pref corruption avoided. Too few prefs sent to save_prefs(). Backtrace:<pre>'."\n".print_r($backtrace, true).'</pre>');
+				return false;
+			}
+
+
 			return e107::getConfig()->loadData($pref, false)->save(false, true);
 			break;
 

e107_handlers/menu_class.php

@@ -451,6 +451,8 @@ public function renderArea($parm = '')
 	public function renderMenu($mpath, $mname='', $parm = '', $return = false)
 	{
 	//	global $sql; // required at the moment.
+
+
 		global $sc_style, $e107_debug;
 
 
@@ -540,7 +542,8 @@ public function renderMenu($mpath, $mname='', $parm = '', $return = false)
 			$id = e107::getForm()->name2id($mpath . $mname);
 			$ns->setUniqueId($id);
 
-			global $pref; // possibly used by plugin menu.
+
+			$pref = e107::getPref(); // possibly used by plugin menu.
 
 
 			$e107_debug ? include(e_PLUGIN.$mpath.$mname.'.php') : @include(e_PLUGIN.$mpath.$mname.'.php');