chrmorandi / yii2-ldap

ActiveRecord.php

Doc Comments +1 added lines, -1 removed lines

@@ -76,7 +76,7 @@
      * Returns the primary key name(s) for this AR class.
      * This method should be overridden by child classes to define the primary key.
      *
-     * @return string[] the primary keys of this record.
+     * @return string the primary keys of this record.
      */
     public static function primaryKey()
     {

Spacing +1 added lines, -1 removed lines

@@ -166,7 +166,7 @@
         }
         
         $values = $this->getDirtyAttributes($attributes);        
-        $params =[
+        $params = [
             'ou=teste,DC=adteste,DC=ifnmg,DC=edu,DC=br',
             $values
         ];

Command.php

Unused Use Statements -1 removed lines

@@ -9,7 +9,6 @@
 
 use Yii;
 use yii\base\Component;
-use yii\base\NotSupportedException;
 
 /**
  * Command represents a SQL statement to be executed against a database.

Spacing +4 added lines, -4 removed lines

@@ -169,7 +169,7 @@ 
         $params = [];
         foreach ($this->params as $name => $value) {
             if (is_string($name) && strncmp(':', $name, 1)) {
-                $name = ':' . $name;
+                $name = ':'.$name;
             }
             if (is_string($value)) {
                 $params[$name] = $this->db->quoteValue($value);
@@ -186,7 +186,7 @@ 
         }
         $sql = '';
         foreach (explode('?', $this->_sql) as $i => $part) {
-            $sql .= (isset($params[$i]) ? $params[$i] : '') . $part;
+            $sql .= (isset($params[$i]) ? $params[$i] : '').$part;
         }
 
         return $sql;
@@ -225,7 +225,7 @@ 
             $this->pdoStatement = $pdo->prepare($sql);
             $this->bindPendingParams();
         } catch (\Exception $e) {
-            $message = $e->getMessage() . "\nFailed to prepare SQL: $sql";
+            $message = $e->getMessage()."\nFailed to prepare SQL: $sql";
             $errorInfo = $e instanceof \PDOException ? $e->errorInfo : null;
             throw new Exception($message, $errorInfo, (int) $e->getCode(), $e);
         }
@@ -267,7 +267,7 @@ 
         } else {
             $this->pdoStatement->bindParam($name, $value, $dataType, $length, $driverOptions);
         }
-        $this->params[$name] =& $value;
+        $this->params[$name] = & $value;
 
         return $this;
     }

DataReader.php

Doc Comments +1 added lines, -1 removed lines

@@ -189,7 +189,7 @@
     /**
      * Returns the result of the current item.
      * This method is required by the interface [[\Iterator]].
-     * @return integer the index of the current row.
+     * @return string the index of the current row.
      */
     public function key()
     {

Spacing +3 added lines, -3 removed lines

@@ -120,7 +120,7 @@ 
      */
     public function toArray()
     {
-        if($this->_count <= 0){
+        if ($this->_count <= 0) {
             return [];
         }
         
@@ -144,7 +144,7 @@ 
             Yii::endProfile('ldap_free_result', 'chrmorandi\ldap\DataReader');
 
             $this->_result = null;
-            $this->_row  = null;
+            $this->_row = null;
         }
     }
 
@@ -216,7 +216,7 @@ 
 
         $resource = $this->_conn->resource;
         
-        Yii::beginProfile('current:' . $this->key(), 'chrmorandi\ldap\DataReader');
+        Yii::beginProfile('current:'.$this->key(), 'chrmorandi\ldap\DataReader');
         $name = ldap_first_attribute($resource, $this->_row);        
         
         while ($name) {            

LdapFunctionTrait.php

Doc Comments +9 added lines, -10 removed lines

@@ -111,7 +111,7 @@ 
      * @param string $dn
      * @param array  $entry
      *
-     * @return mixed
+     * @return boolean
      */
     public function modAdd($dn, array $entry)
     {
@@ -124,7 +124,7 @@ 
      * @param string $dn
      * @param array  $entry
      *
-     * @return mixed
+     * @return boolean
      */
     public function modReplace($dn, array $entry)
     {
@@ -137,7 +137,7 @@ 
      * @param string $dn
      * @param array  $entry
      *
-     * @return mixed
+     * @return boolean
      */
     public function modDelete($dn, array $entry)
     {
@@ -174,7 +174,7 @@ 
      *
      * @param $searchResult
      *
-     * @return mixed
+     * @return resource
      */
     public function getFirstEntry($searchResults)
     {
@@ -186,7 +186,7 @@ 
      *
      * @param $entry
      *
-     * @return mixed
+     * @return resource
      */
     public function getNextEntry($entry)
     {
@@ -220,6 +220,7 @@ 
 
     /**
      * {@inheritdoc}
+     * @param integer $option
      */
     public function setOption($option, $value)
     {
@@ -229,10 +230,8 @@ 
     /**
      * Sets an option on the current connection.
      *
-     * @param int   $option
-     * @param mixed $value
      *
-     * @return mixed
+     * @return boolean
      */
     public function setRebindCallback(callable $callback)
     {
@@ -242,7 +241,7 @@ 
     /**
      * Starts a connection using TLS.
      *
-     * @return mixed
+     * @return boolean
      */
     public function startTLS()
     {
@@ -253,7 +252,7 @@ 
      * Returns the error number of the last command
      * executed on the current connection.
      *
-     * @return mixed
+     * @return integer
      */
     public function getErrNo()
     {

Indentation +14 added lines, -14 removed lines

@@ -105,14 +105,14 @@ 
         return ldap_modify_batch($this->resource, $dn, $values);
     }
 
-     /**
-     * Add attribute values to current attributes.
-     *
-     * @param string $dn
-     * @param array  $entry
-     *
-     * @return mixed
-     */
+        /**
+         * Add attribute values to current attributes.
+         *
+         * @param string $dn
+         * @param array  $entry
+         *
+         * @return mixed
+         */
     public function modAdd($dn, array $entry)
     {
         return ldap_mod_add($this->resource, $dn, $entry);
@@ -249,12 +249,12 @@ 
         return ldap_start_tls($this->resource);
     }
     
-     /**
-     * Returns the error number of the last command
-     * executed on the current connection.
-     *
-     * @return mixed
-     */
+        /**
+         * Returns the error number of the last command
+         * executed on the current connection.
+         *
+         * @return mixed
+         */
     public function getErrNo()
     {
         return ldap_errno($this->resource);

PageControl.php

Doc Comments +1 added lines, -1 removed lines

@@ -39,7 +39,7 @@
     protected $isEnabled = true;
 
     /**
-     * @param LdapConnectionInterface $connection
+     * @param ConnectionInterface $connection
      */
     public function __construct(ConnectionInterface $connection)
     {

Query.php

Doc Comments +1 added lines, -1 removed lines

@@ -339,7 +339,7 @@
 
     /**
      * Sets the SELECT part of the query.
-     * @param string|array $columns the columns to be selected.
+     * @param string $columns the columns to be selected.
      * Columns can be specified in either a string (e.g. "id, name") or an array (e.g. ['id', 'name']).
      *
      * ```php

Spacing +2 added lines, -2 removed lines

@@ -108,7 +108,7 @@ 
         }
         
         $filter = (new FilterBuilder)->build($this->where);
-        $select = (is_array($this->select))?$this->select:[];
+        $select = (is_array($this->select)) ? $this->select : [];
 
         $params = [
             $db->baseDn,
@@ -302,7 +302,7 @@ 
         $params = $command->params;
         $command->setSql($command->db->getQueryBuilder()->selectExists($command->getSql()));
         $command->bindValues($params);
-        return (boolean)$command->queryScalar();
+        return (boolean) $command->queryScalar();
     }
 
     /**

schemas/Schema.php

Doc Comments +2 added lines, -2 removed lines

@@ -28,7 +28,7 @@ 
     /**
      * Sets the current LDAP attribute schema.
      *
-     * @param SchemaInterface $schema
+     * @param \chrmorandi\ldap\schemas\ActiveDirectory $schema
      */
     public static function set(SchemaInterface $schema)
     {
@@ -38,7 +38,7 @@ 
     /**
      * Returns a new instance of the default schema.
      *
-     * @return SchemaInterface
+     * @return \chrmorandi\ldap\schemas\ActiveDirectory
      */
     public static function getDefault()
     {

Utilities/LdapUtilities.php

Spacing +4 added lines, -4 removed lines

@@ -62,12 +62,12 @@ 
         $value = preg_match('/^(\\\[0-9a-fA-F]{2})+$/', (string) $value) ? $value : ldap_escape($value, $ignore, $flags);
 
         // Per RFC 4514, leading/trailing spaces should be encoded in DNs, as well as carriage returns.
-        if ((int)$flags & LDAP_ESCAPE_DN) {
+        if ((int) $flags & LDAP_ESCAPE_DN) {
             if (!empty($value) && $value[0] === ' ') {
-                $value = '\\20' . substr($value, 1);
+                $value = '\\20'.substr($value, 1);
             }
             if (!empty($value) && $value[strlen($value) - 1] === ' ') {
-                $value = substr($value, 0, -1) . '\\20';
+                $value = substr($value, 0, -1).'\\20';
             }
             // Only carriage returns seem to be valid, not line feeds (per testing of AD anyway).
             $value = str_replace("\r", '\0d', $value);
@@ -84,7 +84,7 @@ 
      */
     public static function unescapeValue($value)
     {
-        $callback = function ($matches) {
+        $callback = function($matches) {
             return chr(hexdec($matches[1]));
         };
 

index.php

Indentation +2 added lines, -2 removed lines

@@ -11,8 +11,8 @@
   </head>
   <body>
       <?php
-      // put your code here
+        // put your code here
 
-      ?>
+        ?>
   </body>
 </html>