Commit: 34644268b20666bcbe0276ad3ea8028d353b4e4d - xoopscubelegacy (git)

XOOPS Cube Legacy base repository

Commit MetaInfo

• Revision: 34644268b20666bcbe0276ad3ea8028d353b4e4d (tree)
• Time: 2012-08-16 10:49:42
• Author: HIKAWA Kilica <kilica.hikawa@gmai...>
• Commiter: HIKAWA Kilica

Log Message

#8 AbstractClientObjectHandler fixation is wrong.

Change Summary

• modified: html/modules/legacy/class/interface/AbstractClientObjectHandler.class.php

Incremental Difference

--- a/html/modules/legacy/class/interface/AbstractClientObjectHandler.class.php
+++ b/html/modules/legacy/class/interface/AbstractClientObjectHandler.class.php
@@ -345,7 +345,14 @@ abstract class Legacy_AbstractClientObjectHandler extends XoopsObjectGenericHand
 	 */
 	protected function _isActivityClient(/*** mixed[] ***/ $conf)
 	{
-		return $conf[$this->_mClientConfig['activity']]==1 ? true : false;
+		$key ='activity';
+		if(! isset($this->_mClientConfig[$key])){
+			return false;
+		}
+		if(! isset($conf[$this->_mClientConfig[$key]])){
+			return false;
+		}
+		return $conf[$this->_mClientConfig[$key]]==1 ? true : false;
 	}
 
 	/**
@@ -357,7 +364,14 @@ abstract class Legacy_AbstractClientObjectHandler extends XoopsObjectGenericHand
 	 */
 	protected function _isTagClient(/*** mixed[] ***/ $conf)
 	{
-		return isset($conf[$this->_mClientConfig['tag']]) ?  true : false;
+		$key ='tag';
+		if(! isset($this->_mClientConfig[$key])){
+			return false;
+		}
+		if(! isset($conf[$this->_mClientConfig[$key]])){
+			return false;
+		}
+		return $conf[$this->_mClientConfig[$key]] ? true : false;
 	}
 
 	/**
@@ -369,7 +383,14 @@ abstract class Legacy_AbstractClientObjectHandler extends XoopsObjectGenericHand
 	 */
 	protected function _isWorkflowClient(/*** mixed[] ***/ $conf)
 	{
-		return isset($conf[$this->_mClientConfig['workflow']]) ? true : false;
+		$key ='workflow';
+		if(! isset($this->_mClientConfig[$key])){
+			return false;
+		}
+		if(! isset($conf[$this->_mClientConfig[$key]])){
+			return false;
+		}
+		return $conf[$this->_mClientConfig[$key]] ? true : false;
 	}
 
 	/**
@@ -381,7 +402,14 @@ abstract class Legacy_AbstractClientObjectHandler extends XoopsObjectGenericHand
 	 */
 	protected function _isImageClient(/*** mixed[] ***/ $conf)
 	{
-		return isset($conf[$this->_mClientConfig['image']]) ? true : false;
+		$key ='image';
+		if(! isset($this->_mClientConfig[$key])){
+			return false;
+		}
+		if(! isset($conf[$this->_mClientConfig[$key]])){
+			return false;
+		}
+		return $conf[$this->_mClientConfig[$key]] ? true : false;
 	}
 
     /**
@@ -393,8 +421,15 @@ abstract class Legacy_AbstractClientObjectHandler extends XoopsObjectGenericHand
      */
     protected function _isMapClient(/*** mixed[] ***/ $conf)
     {
-		return isset($conf[$this->_mClientConfig['map']]) ? true : false;
-    }
+		$key ='map';
+		if(! isset($this->_mClientConfig[$key])){
+			return false;
+		}
+		if(! isset($conf[$this->_mClientConfig[$key]])){
+			return false;
+		}
+		return $conf[$this->_mClientConfig[$key]]==1 ? true : false;
+	}
 
 	/**
 	 * get client field name