Improved BC layer for the Ldap component

Author: csarrazi

src/Symfony/Component/Ldap/BaseLdapInterface.php

@@ -44,6 +44,22 @@ public function bind($dn = null, $password = null)
         $this->ldap->bind($dn, $password);
     }
 
+    /**
+     * {@inheritdoc}
+     */
+    public function query($dn, $query, array $options = array())
+    {
+        return $this->ldap->query($dn, $query, $options);
+    }
+
+    /**
+     * {@inheritdoc}
+     */
+    public function getEntryManager()
+    {
+        return $this->ldap->getEntryManager();
+    }
+
     /**
      * {@inheritdoc}
      */

src/Symfony/Component/Ldap/LdapClient.php

@@ -21,9 +21,9 @@
  *
  * @deprecated You should use LdapInterface instead
  */
-interface LdapClientInterface extends BaseLdapInterface
+interface LdapClientInterface extends LdapInterface
 {
-    /*
+    /**
      * Find a username into ldap connection.
      *
      * @param string $dn

src/Symfony/Component/Ldap/LdapClientInterface.php

@@ -13,17 +13,28 @@
 
 use Symfony\Component\Ldap\Adapter\EntryManagerInterface;
 use Symfony\Component\Ldap\Adapter\QueryInterface;
+use Symfony\Component\Ldap\Exception\ConnectionException;
 
 /**
  * Ldap interface.
  *
  * @author Charles Sarrazin <charles@sarraz.in>
  */
-interface LdapInterface extends BaseLdapInterface
+interface LdapInterface
 {
     const ESCAPE_FILTER = 0x01;
     const ESCAPE_DN = 0x02;
 
+    /**
+     * Return a connection bound to the ldap.
+     *
+     * @param string $dn       A LDAP dn
+     * @param string $password A password
+     *
+     * @throws ConnectionException If dn / password could not be bound.
+     */
+    public function bind($dn = null, $password = null);
+
     /**
      * Queries a ldap server for entries matching the given criteria.
      *
@@ -39,4 +50,15 @@ public function query($dn, $query, array $options = array());
      * @return EntryManagerInterface
      */
     public function getEntryManager();
+
+    /**
+     * Escape a string for use in an LDAP filter or DN.
+     *
+     * @param string $subject
+     * @param string $ignore
+     * @param int    $flags
+     *
+     * @return string
+     */
+    public function escape($subject, $ignore = '', $flags = 0);
 }

src/Symfony/Component/Ldap/LdapInterface.php

@@ -54,6 +54,16 @@ public function testLdapEscape()
         $this->client->escape('foo', 'bar', 'baz');
     }
 
+    public function testLdapQuery()
+    {
+        $this->ldap
+            ->expects($this->once())
+            ->method('query')
+            ->with('foo', 'bar', array('baz'))
+        ;
+        $this->client->query('foo', 'bar', array('baz'));
+    }
+
     public function testLdapFind()
     {
         $collection = $this->getMock(CollectionInterface::class);