cvs: pear /DB_DataObject_FormBuilder FormBuilder.php

arnaud Mon Sep 8 05:52:18 2003 EDT Modified files: /pear/DB_DataObject_FormBuilder FormBuilder.php Log: * fix bug preventing the options array from being used * use raiseError instead of new PEAR_Error, use a variable to make php5 happy * cosmetic Index: pear/DB_DataObject_FormBuilder/FormBuilder.php diff -u pear/DB_DataObject_FormBuilder/FormBuilder.php:1.8 pear/DB_DataObject_FormBuilder/FormBuilder.php:1.9 --- pear/DB_DataObject_FormBuilder/FormBuilder.php:1.8 Sat Sep 6 12:39:43 2003 +++ pear/DB_DataObject_FormBuilder/FormBuilder.php Mon Sep 8 05:52:17 2003 @@ -95,7 +95,7 @@ * * @package DB_DataObject_FormBuilder * @author Markus Wolff <mw21st@php.net> - * @version $Id: FormBuilder.php,v 1.8 2003/09/06 16:39:43 mw21st Exp $ + * @version $Id: FormBuilder.php,v 1.9 2003/09/08 09:52:17 arnaud Exp $ */ @@ -182,15 +182,16 @@ * @access public * @returns object DB_DataObject_FormBuilder or PEAR_Error object */ - function &create(&$do, $options=false) + function &create(&$do, $options = false) { if (is_a($do, 'db_dataobject')) { $obj = &new DB_DataObject_FormBuilder($do, $options); return $obj; } - return (new PEAR_Error('DB_DataObject_FormBuilder::create(): Object does not extend DB_DataObject.', - DB_DATAOBJECT_FORMBUILDER_ERROR_WRONGCLASS) - ); + + $err =& PEAR::raiseError('DB_DataObject_FormBuilder::create(): Object does not extend DB_DataObject.', + DB_DATAOBJECT_FORMBUILDER_ERROR_WRONGCLASS); + return $err; } @@ -208,7 +209,7 @@ if (is_array($options)) { reset($options); while (list($key, $value) = each($options)) { - if (isset($this->key)) { + if (isset($this->$key)) { $this->$key = $value; } }