silverstripe / silverstripe-activedirectory

code/model/LDAPUtil.php

Doc Comments +8 added lines, -1 removed lines

@@ -9,7 +9,7 @@ 
     /**
      * Checks if the string is a valid guid in the format of A98C5A1E-A742-4808-96FA-6F409E799937
      *
-     * @param $guid
+     * @param string $guid
      * @return bool
      */
     public static function validGuid($guid)
@@ -41,6 +41,9 @@ 
         return strtoupper($hex_guid_to_guid_str);
     }
 
+    /**
+     * @param string $str_guid
+     */
     public static function str_to_hex_guid($str_guid, $escape = false)
     {
         $str_guid = str_replace('-', '', $str_guid);
@@ -87,6 +90,10 @@ 
     }
 
     // Converts a little-endian hex-number to one, that 'hexdec' can convert
+
+    /**
+     * @param string $hex
+     */
     public static function little_endian($hex)
     {
         $result = '';

code/services/LDAPService.php

Doc Comments +3 added lines, -3 removed lines

@@ -192,7 +192,7 @@ 
      * Note that these get cached in-memory per-request for performance to avoid re-querying for the same results.
      *
      * @param string $dn
-     * @param array $attributes List of specific AD attributes to return. Empty array means return everything.
+     * @param string[] $attributes List of specific AD attributes to return. Empty array means return everything.
      * @return array
      */
     public function getNestedGroups($dn, $attributes = array())
@@ -376,7 +376,7 @@ 
      * @param Member
      * @param array|null $data If passed, this is pre-existing AD attribute data to update the Member with.
      *            If not given, the data will be looked up by the user's GUID.
-     * @return bool
+     * @return false|null
      */
     public function updateMemberFromLDAP(Member $member, $data = null)
     {
@@ -547,7 +547,7 @@ 
      * @param Group $group An existing Group or a new Group object
      * @param array $data LDAP group object data
      *
-     * @return bool
+     * @return false|null
      */
     public function updateGroupFromLDAP(Group $group, $data)
     {