Fix unrelated style issues

Author: Jamie Snape

.styleci.yml

@@ -33,6 +33,7 @@ finder:
   exclude:
     - data
     - docs
+    - library/REST
     - log
     - provisioning
     - tmp

core/controllers/ShareController.php

@@ -254,7 +254,7 @@ public function dialogAction()
         $this->view->jsonShare['type'] = $type;
         $this->view->jsonShare['element'] = $element->getKey();
         $this->view->jsonShare = JsonComponent::encode($this->view->jsonShare);
-    } //end dialogAction
+    }
 
     /** controller for applying policies recursively to a folder */
     public function applyrecursivedialogAction()

core/models/pdo/MetadataModel.php

@@ -83,10 +83,9 @@ public function getMetadataQualifiers($type, $element)
         return $qualifiers;
     }
 
-    /** Return an item by its name
-     * @return MetadataDao
-     */
     /**
+     * Return an item by its name.
+     *
      * @param int $type
      * @param string $element
      * @param string $qualifier

modules/api/tests/controllers/CallItemMethodsTest.php

@@ -1220,7 +1220,7 @@ public function testBitstreamDelete()
         $this->assertFalse($bitstreamDao, 'Bitstream should have been deleted, but was not.');
     }
 
-    /* helper function for item.setmetadata calls */
+    /** Helper function for item.setmetadata calls */
     private function _callSetmetadata(
         $itemId,
         $element,
@@ -1255,7 +1255,7 @@ private function _callSetmetadata(
         return $resp;
     }
 
-    /* helper function for item.setmultiplemetadata calls */
+    /** Helper function for item.setmultiplemetadata calls */
     private function _callSetmultiplemetadata($itemId, $metadata, $count = null, $revision = null, $failureCode = null)
     {
         $this->resetAll();
@@ -1291,7 +1291,7 @@ private function _callSetmultiplemetadata($itemId, $metadata, $count = null, $re
         return $resp;
     }
 
-    /* helper function for item.getmetadata calls */
+    /** Helper function for item.getmetadata calls */
     private function _callGetmetadata($itemId, $revision = null, $failureCode = null)
     {
         $this->resetAll();
@@ -1311,7 +1311,7 @@ private function _callGetmetadata($itemId, $revision = null, $failureCode = null
         return $resp;
     }
 
-    /* helper function for item.deletemetadata calls */
+    /** Helper function for item.deletemetadata calls */
     private function _callDeletemetadata(
         $itemId,
         $element,
@@ -1344,7 +1344,7 @@ private function _callDeletemetadata(
         return $resp;
     }
 
-    /* helper function for item.deletemetadata.all calls */
+    /** Helper function for item.deletemetadata.all calls */
     private function _callDeletemetadataAll($itemId, $revision = null, $failureCode = null)
     {
         $this->resetAll();