[xoops-svn] SF.net SVN: xoops:[9339] XoopsCore/branches/2.5.x/2.5.5

[<be...@us...>]

Revision: 9339
          http://xoops.svn.sourceforge.net/xoops/?rev=9339&view=rev
Author:   beckmi
Date:     2012-04-16 00:15:04 +0000 (Mon, 16 Apr 2012)
Log Message:
-----------
ID: 3518291 Fix of variables assigned by reference in formelementtray.php

Modified Paths:
--------------
    XoopsCore/branches/2.5.x/2.5.5/docs/changelog.250.txt
    XoopsCore/branches/2.5.x/2.5.5/htdocs/class/xoopsform/formelementtray.php

Modified: XoopsCore/branches/2.5.x/2.5.5/docs/changelog.250.txt
===================================================================
--- XoopsCore/branches/2.5.x/2.5.5/docs/changelog.250.txt	2012-04-15 22:59:21 UTC (rev 9338)
+++ XoopsCore/branches/2.5.x/2.5.5/docs/changelog.250.txt	2012-04-16 00:15:04 UTC (rev 9339)
@@ -7,6 +7,7 @@
  - preventing division by zero in pagenav.php (timgno)
  - ID: 3466534 tooltip "$ not defined" fix (culex)
  - ID: 3513787 wrong cookie_domain (arion92fr) 
+ - ID: 3518291 Fix of variables assigned by reference in formelementtray
  
 Updated:
  - phpThumb to 1.7.11 (mamba)

Modified: XoopsCore/branches/2.5.x/2.5.5/htdocs/class/xoopsform/formelementtray.php
===================================================================
--- XoopsCore/branches/2.5.x/2.5.5/htdocs/class/xoopsform/formelementtray.php	2012-04-15 22:59:21 UTC (rev 9338)
+++ XoopsCore/branches/2.5.x/2.5.5/htdocs/class/xoopsform/formelementtray.php	2012-04-16 00:15:04 UTC (rev 9339)
@@ -83,14 +83,14 @@
 	 * @param object $ &$element    {@link XoopsFormElement} to add
 	 */
 	function addElement( &$formElement, $required = false ) {
-		$this->_elements[] = &$formElement;
+		$this->_elements[] = $formElement;
 		if ( !$formElement->isContainer() ) {
 			if ( $required ) {
 				$formElement->_required = true;
-				$this->_required[] = &$formElement;
+				$this->_required[] = $formElement;
 			}
 		} else {
-			$required_elements = &$formElement->getRequired();
+			$required_elements = $formElement->getRequired();
 			$count = count( $required_elements );
 			for ( $i = 0 ; $i < $count; $i++ ) {
 				$this->_required[] = &$required_elements[$i];

This was sent by the SourceForge.net collaborative development platform, the world's largest Open Source development site.