Merge pull request #13 from sd-tm/master

Don't test in Interfaces body function of a return Statement.

Oxid/Sniffs/Commenting/FunctionCommentSniff.php

@@ -34,6 +34,13 @@
  */
 class Oxid_Sniffs_Commenting_FunctionCommentSniff extends PEAR_Sniffs_Commenting_FunctionCommentSniff
 {
+    /**
+     * Check it is a PHP Interface, but one time.
+     *
+     * @var array
+     */
+    protected $isFileInterface = array();
+
     /**
      * Process the return comment of this function comment.
      *
@@ -65,6 +72,10 @@ protected function processReturn(PHP_CodeSniffer_File $phpcsFile, $stackPtr, $co
             }
         }
 
+        if ($this->isInterface($phpcsFile)) {
+            return;
+        }
+
         if ($isSpecialMethod === true) {
             return;
         }
@@ -117,4 +128,23 @@ protected function functionHasReturnStatement(PHP_CodeSniffer_File $phpcsFile, $
 
         return false;
     }//end functionHasReturnStatement()
+
+    /**
+     * Test if this a PHP Interface File
+     *
+     * @param PHP_CodeSniffer_File $phpcsFile
+     *
+     * @return bool
+     */
+    protected function isInterface(PHP_CodeSniffer_File $phpcsFile) {
+        $checkFile = "". $phpcsFile->getFilename();
+        if (isset($this->isFileInterface[$checkFile])) {
+            return (bool) $this->isFileInterface[$checkFile];
+        }
+
+        $interface = $phpcsFile->findNext(T_INTERFACE, 0);
+
+        return $this->isFileInterface[$checkFile] = (bool)(empty($interface) == false);
+    }//end isInterfaceClass()
+
 }//end class