Fix pass-by-reference errors

symphonycms · Jun 15, 2011 · 9903bea · 9903bea
1 parent e63b479
commit 9903bea
Show file tree

Hide file tree

Showing 5 changed files with 7 additions and 7 deletions.
diff --git a/events/event.members_reset_password.php b/events/event.members_reset_password.php
@@ -210,7 +210,7 @@ protected function __trigger(){
 				// Create new password using the auth field so simulate the checkPostFieldData
 				// and processRawFieldData functions.
 				$message = '';
-				$status = $auth->checkPostFieldData($fields[$auth->get('element_name')], &$message, $member_id);
+				$status = $auth->checkPostFieldData($fields[$auth->get('element_name')], $message, $member_id);
 				if(Field::__OK__ != $status) {
 					$result->setAttribute('result', 'error');
 					$result->appendChild(

diff --git a/extension.driver.php b/extension.driver.php
@@ -1054,17 +1054,17 @@ public function checkEventPermissions(array &$context){
 		private function __processEventFilters(array &$context) {
 			// Process the Member Lock Role
 			if (in_array('member-lock-role', $context['event']->eParamFILTERS)) {
-				$this->getMemberDriver()->filter_LockRole(&$context);
+				$this->getMemberDriver()->filter_LockRole($context);
 			}
 
 			// Process the Member Lock Activation
 			if (in_array('member-lock-activation', $context['event']->eParamFILTERS)) {
-				$this->getMemberDriver()->filter_LockActivation(&$context);
+				$this->getMemberDriver()->filter_LockActivation($context);
 			}
 
 			// Process updating a Member's Password
 			if (in_array('member-update-password', $context['event']->eParamFILTERS)) {
-				$this->getMemberDriver()->filter_UpdatePassword(&$context);
+				$this->getMemberDriver()->filter_UpdatePassword($context);
 			}
 		}
 

diff --git a/fields/field.memberactivation.php b/fields/field.memberactivation.php
@@ -262,7 +262,7 @@ public function displaySettingsPanel(&$wrapper, $errors=NULL){
 		}
 
 		public function checkFields(&$errors, $checkForDuplicates=true) {
-			Field::checkFields(&$errors, $checkForDuplicates);
+			Field::checkFields($errors, $checkForDuplicates);
 
 			if (trim($this->get('code_expiry')) == '') {
 				$errors['code_expiry'] = __('This is a required field.');

diff --git a/fields/field.memberrole.php b/fields/field.memberrole.php
@@ -118,7 +118,7 @@ public function displaySettingsPanel(&$wrapper, $errors=NULL){
 		}
 
 		public function checkFields(&$errors, $checkForDuplicates=true) {
-			Field::checkFields(&$errors, $checkForDuplicates);
+			Field::checkFields($errors, $checkForDuplicates);
 		}
 
 		public function commit(){

diff --git a/fields/field.membertimezone.php b/fields/field.membertimezone.php
@@ -219,7 +219,7 @@ public function displaySettingsPanel(&$wrapper, $errors=NULL){
 		}
 
 		public function checkFields(&$errors, $checkForDuplicates=true) {
-			Field::checkFields(&$errors, $checkForDuplicates);
+			Field::checkFields($errors, $checkForDuplicates);
 		}
 
 		public function commit(){