From aa218fce3b98ef091acfa202addd64e7f18f169a Mon Sep 17 00:00:00 2001 From: Benjamin Renard Date: Tue, 16 Nov 2010 19:38:55 +0100 Subject: [PATCH] LSsearch :: setParams() : Fixed notice PHP --- public_html/includes/class/class.LSsearch.php | 40 ++++++++++--------- 1 file changed, 22 insertions(+), 18 deletions(-) diff --git a/public_html/includes/class/class.LSsearch.php b/public_html/includes/class/class.LSsearch.php index 3c39109d..b9544006 100644 --- a/public_html/includes/class/class.LSsearch.php +++ b/public_html/includes/class/class.LSsearch.php @@ -240,19 +240,21 @@ class LSsearch { $OK=true; // Filter - if (isset($params['filter']) && is_string($params['filter'])) { - $filter = Net_LDAP2_Filter::parse($params['filter']); - if (!LSerror::isLdapError($filter)) { - $this -> params['filter'] = $filter; + if (isset($params['filter'])) { + if (is_string($params['filter'])) { + $filter = Net_LDAP2_Filter::parse($params['filter']); + if (!LSerror::isLdapError($filter)) { + $this -> params['filter'] = $filter; + } + else { + LSerror :: addErrorCode('LSsearch_01',$params['filter']); + $OK=false; + } } - else { - LSerror :: addErrorCode('LSsearch_01',$params['filter']); - $OK=false; + elseif($params['filter'] instanceof Net_LDAP2_Filter) { + $this -> params['filter'] =& $params['filter']; } } - elseif($params['filter'] instanceof Net_LDAP2_Filter) { - $this -> params['filter'] =& $params['filter']; - } // Approx if (isset($params['approx'])) { @@ -277,14 +279,16 @@ class LSsearch { } // Patterm - if (isset($params['pattern']) && $params['pattern']=="") { - $this -> params['pattern'] = NULL; - $this -> params['filter'] = NULL; - } - elseif (isset($params['pattern']) && self :: isValidPattern($params['pattern'])) { - $this -> params['pattern'] = $params['pattern']; - if (!is_string($params['filter'])) { - $this -> params['filter']=NULL; + if (isset($params['pattern'])) { + if ($params['pattern']=="") { + $this -> params['pattern'] = NULL; + $this -> params['filter'] = NULL; + } + elseif (self :: isValidPattern($params['pattern'])) { + $this -> params['pattern'] = $params['pattern']; + if (!is_string($params['filter'])) { + $this -> params['filter']=NULL; + } } }