"new so_sql2 class: adding magic __get() and __set() methods, to be able to work more object oriented with so_sql (just a first step)

you can now use $so_sql_obj->my_col = 15 instead of $so_sql_obj->data['my_col']=15; (be aware, that does NOT work with var-parameters of functions and probably references)
$so_sql_obj->id alwas references the auto-id, independent of it's name (id is a bad choice of a column-name, because it's a reserved word in some dbms)"

etemplate/inc/class.so_sql2.inc.php

@@ -0,0 +1,114 @@
+<?php
+/**
+ * eGroupWare generalized SQL Storage Object Version 2 - requires PHP5.1.2+!!!
+ *
+ * @license http://opensource.org/licenses/gpl-license.php GPL - GNU General Public License
+ * @package etemplate
+ * @link http://www.egroupware.org
+ * @author Ralf Becker <RalfBecker@outdoor-training.de>
+ * @version $Id$
+ */
+
+require_once(EGW_INCLUDE_ROOT.'/etemplate/inc/class.so_sql.inc.php');
+
+/**
+ * generalized SQL Storage Object
+ *
+ * the class can be used in following ways:
+ * 1) by calling the constructor with an app and table-name or
+ * 2) by setting the following documented class-vars in a class derifed from this one
+ * Of cause can you derife the class and call the constructor with params.
+ * 
+ * The so_sql2 class uses a privat $data array and __get and __set methods to set its data.
+ * Please note: 
+ * You have to explicitly declare other object-properties of derived classes, which should NOT
+ * be handled by that mechanism!
+ *
+ * @package etemplate
+ * @subpackage api
+ * @author RalfBecker-AT-outdoor-training.de
+ * @license http://opensource.org/licenses/gpl-license.php GPL - GNU General Public License
+ */
+class so_sql2 extends so_sql
+{
+	/**
+	 * Private array containing all the object-data
+	 * 
+	 * Colides with the original definition in so_sql and I dont want to change it there at the moment.
+	 *
+	 * @var array
+	 */
+	//private $data;
+
+	/**
+	 * constructor of the class
+	 *
+	 * NEED to be called from the constructor of the derived class !!!
+	 *
+	 * @param string $app should be set if table-defs to be read from <app>/setup/tables_current.inc.php
+	 * @param string $table should be set if table-defs to be read from <app>/setup/tables_current.inc.php
+	 * @param object/db $db database object, if not the one in $GLOBALS['egw']->db should be used, eg. for an other database
+	 * @param string $colum_prefix='' column prefix to automatic remove from the column-name, if the column name starts with it
+	 * 
+	 * @return so_sql2
+	 */
+	function so_sql2($app='',$table='',$db=null,$column_prefix='')
+	{
+		$this->so_sql($app,$table,$db,$column_prefix);
+	}
+
+	/**
+	 * magic method to read a property from $this->data
+	 * 
+	 * The special property 'id' always refers to the auto-increment id of the object, independent of its name.
+	 *
+	 * @param string $property
+	 * @return mixed
+	 */
+	function __get($property)
+	{
+		switch($property)
+		{
+			case 'id':
+				$property = $this->autoinc_id;
+				break;
+		}
+		if (in_array($property,$this->db_cols))
+		{
+			return $this->data[$property];
+		}
+	}
+	
+	/**
+	 * magic method to set a property in $this->data
+	 *
+	 * The special property 'id' always refers to the auto-increment id of the object, independent of its name.
+	 *
+	 * @param string $property
+	 * @param mixed $value
+	 * @return mixed
+	 */
+	function __set($property,$value)
+	{
+		switch($property)
+		{
+			case 'id':
+				$property = $this->autoinc_id;
+				break;
+		}
+		if (in_array($property,$this->db_cols))
+		{
+			$this->data[$property] = $value;
+		}
+	}
+	
+	/**
+	 * Return the whole object-data as array, it's a cast of the object to an array
+	 *
+	 * @return array
+	 */
+	function as_array()
+	{
+		return $this->data;
+	}
+}