Completed
Push — adminicons ( 220b8a...d1a314 )
by Andreas
03:46
created
lib/plugins/auth.php 1 patch
Doc Comments   +1 added lines, -1 removed lines patch added patch discarded remove patch
@@ -183,7 +183,7 @@
 block discarded – undo
183 183
      * @param   string  $user    Username
184 184
      * @param   string  $pass    Cleartext Password
185 185
      * @param   bool    $sticky  Cookie should not expire
186
-     * @return  bool             true on successful auth
186
+     * @return  boolean|null             true on successful auth
187 187
      */
188 188
     public function trustExternal($user, $pass, $sticky = false) {
189 189
         /* some example:
Please login to merge, or discard this patch.
lib/plugins/authplain/auth.php 1 patch
Doc Comments   +4 added lines, -1 removed lines patch added patch discarded remove patch
@@ -82,7 +82,7 @@  discard block
 block discarded – undo
82 82
      * @author  Andreas Gohr <[email protected]>
83 83
      * @param string $user
84 84
      * @param bool $requireGroups  (optional) ignored by this plugin, grps info always supplied
85
-     * @return array|false
85
+     * @return string
86 86
      */
87 87
     public function getUserData($user, $requireGroups=true) {
88 88
         if($this->users === null) $this->_loadUserData();
@@ -380,6 +380,9 @@  discard block
 block discarded – undo
380 380
         return $users;
381 381
     }
382 382
 
383
+    /**
384
+     * @param string $line
385
+     */
383 386
     protected function _splitUserData($line){
384 387
         // due to a bug in PCRE 6.6, preg_split will fail with the regex we use here
385 388
         // refer github issues 877 & 885
Please login to merge, or discard this patch.