Enalean / tuleap

plugins/ldap/include/LDAP_ProjectGroupManager.class.php

Doc Comments +1 added lines, -1 removed lines

@@ -62,7 +62,7 @@
      *
      * @param Integer $groupId Id of project
      *
-     * @return Array
+     * @return integer
      */
     protected function getDbGroupMembersIds($groupId)
     {

plugins/ldap/include/LDAP_SearchPeople.class.php

Doc Comments +3 added lines

@@ -75,6 +75,9 @@
         return new LDAP_SearchPeopleResultPresenter($ldap_result->getCommonName(), PFUser::DEFAULT_AVATAR_URL, $directory_uri);
     }
 
+    /**
+     * @param string $value
+     */
     private function buildLinkToDirectory(LDAPResult $lr, $value) {
         include_once($GLOBALS['Language']->getContent('directory_redirect', 'en_US', 'ldap'));
         if(function_exists('custom_build_link_to_directory')) {

plugins/ldap/include/LDAP_SyncMail.class.php

Doc Comments +1 added lines, -1 removed lines

@@ -34,7 +34,7 @@
      *
      * @param PFUser $user Suspended user after LDAP daily synchro
      *
-     * @return Array
+     * @return Project[]
      */
     private function getProjectsForUser(PFUser $user) {
         return $this->projectManager->getActiveProjectsForUser($user);

plugins/ldap/include/LDAP_UserGroupManager.class.php

Doc Comments +2 added lines, -2 removed lines

@@ -52,7 +52,7 @@ 
      * @param Integer $ugroupId Codendi Group ID
      * @param Integer $userId   User ID
      *
-     * @return Boolean
+     * @return boolean|null
      */
     protected function addUserToGroup($ugroupId, $userId)
     {
@@ -65,7 +65,7 @@ 
      * @param Integer $ugroupId Codendi Group ID
      * @param Integer $userId   User ID
      *
-     * @return Boolean
+     * @return boolean|null
      */
     protected function removeUserFromGroup($ugroupId, $userId)
     {

plugins/ldap/include/LDAP_UserManager.class.php

Doc Comments +2 added lines, -2 removed lines

@@ -86,7 +86,7 @@ 
     /**
      * Get LDAPResult object corresponding to an LDAP ID
      *
-     * @param  $ldapId    The LDAP identifier
+     * @param  string $ldapId    The LDAP identifier
      * @return LDAPResult
      */
     function getLdapFromLdapId($ldapId) {
@@ -202,7 +202,7 @@ 
      * Return LDAP logins stored in DB corresponding to given userIds.
      * 
      * @param Array $userIds Array of user ids
-     * @return Array ldap logins
+     * @return DataAccessResult ldap logins
      */
     function getLdapLoginFromUserIds(array $userIds) {
         $dao = $this->getDao();

plugins/ldap/include/LDAP_UserSync.class.php

Doc Comments +1 added lines

@@ -58,6 +58,7 @@
     /**
      * Return the sync attributes
      * 
+     * @param LDAP $ldap
      * @return array
      */
     public function getSyncAttributes($ldap) {

plugins/ldap/include/ldapPlugin.class.php

Doc Comments +4 added lines, -1 removed lines

@@ -510,6 +510,9 @@ 
         print '</TR>';
     }
 
+    /**
+     * @param LDAPResult $lr
+     */
     function buildLinkToDirectory(&$lr, $value='') {
         if($value === '') {
             $value = $lr->getLogin();
@@ -849,7 +852,7 @@ 
      *
      * @param Array $params
      *
-     * @return void
+     * @return boolean|null
      */
     function codendi_daily_start($params) {
         if ($GLOBALS['sys_auth_type'] == 'ldap' && $this->isDailySyncEnabled()) {

plugins/ldap/include/LDAPResult.class.php

Doc Comments +4 added lines, -1 removed lines

@@ -66,7 +66,7 @@ 
     /**
      * Return true if there is no entries in the result set.
      *
-     * @return int
+     * @return boolean
      */
     function isEmpty() {
         return empty($this->list);
@@ -215,6 +215,9 @@ 
         return $this->get($this->ldapParams['eduid']);
     }
 
+    /**
+     * @return string|null
+     */
     function getDn() {
         return $this->info['dn'];
     }

plugins/ldap/tests/bootstrap.php

Doc Comments +3 added lines

@@ -54,6 +54,9 @@
 
     private $expected;
 
+    /**
+     * @param string $expected_cn
+     */
     public function __construct($expected_cn) {
         parent::__construct();
         $this->expected = $expected_cn;