summaryrefslogtreecommitdiff
path: root/program/include/rcube_ldap.php
diff options
context:
space:
mode:
authoralecpl <alec@alec.pl>2009-09-11 08:15:10 +0000
committeralecpl <alec@alec.pl>2009-09-11 08:15:10 +0000
commit4aaecb661cd25d1b35ccc89ab64c1a1aff427087 (patch)
tree39d7073ad5884013add920ef0713caf8a9904e81 /program/include/rcube_ldap.php
parent71047326faf2feaac3f4017e71dad8d2fa5b2e42 (diff)
- small code improvements per discussion from #1485536
Diffstat (limited to 'program/include/rcube_ldap.php')
-rw-r--r--program/include/rcube_ldap.php15
1 files changed, 8 insertions, 7 deletions
diff --git a/program/include/rcube_ldap.php b/program/include/rcube_ldap.php
index 82e1df270..b72ef80aa 100644
--- a/program/include/rcube_ldap.php
+++ b/program/include/rcube_ldap.php
@@ -343,7 +343,7 @@ class rcube_ldap extends rcube_addressbook
*
* @return object rcube_result_set Resultset with values for 'count' and 'first'
*/
- function count()
+ function count($check)
{
$count = 0;
if ($this->conn && $this->ldap_result) {
@@ -388,8 +388,8 @@ class rcube_ldap extends rcube_addressbook
$res = null;
if ($this->conn && $dn)
{
- $this->ldap_result = @ldap_read($this->conn, base64_decode($dn), '(objectclass=*)', array_values($this->fieldmap));
- $entry = @ldap_first_entry($this->conn, $this->ldap_result);
+ if ($this->ldap_result = @ldap_read($this->conn, base64_decode($dn), '(objectclass=*)', array_values($this->fieldmap)))
+ $entry = ldap_first_entry($this->conn, $this->ldap_result);
if ($entry && ($rec = ldap_get_attributes($this->conn, $entry)))
{
@@ -565,11 +565,12 @@ class rcube_ldap extends rcube_addressbook
{
$filter = $this->filter ? $this->filter : '(objectclass=*)';
$function = $this->prop['scope'] == 'sub' ? 'ldap_search' : ($this->prop['scope'] == 'base' ? 'ldap_read' : 'ldap_list');
- $this->ldap_result = @$function($this->conn, $this->prop['base_dn'], $filter, array_values($this->fieldmap), 0, 0);
- return true;
+
+ if ($this->ldap_result = @$function($this->conn, $this->prop['base_dn'], $filter, array_values($this->fieldmap), 0, 0))
+ return true;
}
- else
- return false;
+
+ return false;
}