librenms / librenmsv2

app/Http/Controllers/General/SearchController.php

Unused Use Statements +1 added lines, -1 removed lines

@@ -25,10 +25,10 @@
 
 namespace App\Http\Controllers\General;
 
+use App\DataTables\General\ArpDataTable;
 use App\DataTables\General\IPv4DataTable;
 use App\DataTables\General\IPv6DataTable;
 use App\DataTables\General\MacDataTable;
-use App\DataTables\General\ArpDataTable;
 use App\Http\Controllers\Controller;
 use Illuminate\Http\Request;
 

app/Http/Controllers/UserController.php

Doc Comments +2 added lines, -2 removed lines

@@ -44,7 +44,7 @@ 
     /**
      * Update a notifications status.
      *
-     * @return \Illuminate\Http\Response
+     * @return \Illuminate\Http\JsonResponse
      */
     public function update($id, $action)
     {
@@ -55,7 +55,7 @@ 
     /**
      * Create a new notification
      *
-     * @return \Illuminate\Http\Response
+     * @return \Illuminate\Http\JsonResponse
      */
     public function create(Request $request)
     {

app/Http/Controllers/UserPortController.php

Doc Comments +2 added lines, -2 removed lines

@@ -46,7 +46,7 @@ 
      * @param AdminOnlyRequest $request
      * @param $user_id
      * @param $port_id
-     * @return mixed
+     * @return \Illuminate\Http\RedirectResponse
      */
     public function store(AdminOnlyRequest $request, $user_id)
     {
@@ -69,7 +69,7 @@ 
      * @param AdminOnlyRequest $request
      * @param $user_id
      * @param $port_id
-     * @return mixed
+     * @return \Illuminate\Http\RedirectResponse
      */
     public function destroy(AdminOnlyRequest $request, $user_id, $port_id)
     {

app/Http/Controllers/UserDeviceController.php

Doc Comments +2 added lines, -2 removed lines

@@ -46,7 +46,7 @@ 
      * @param AdminOnlyRequest $request
      * @param $user_id
      * @param $port_id
-     * @return mixed
+     * @return \Illuminate\Http\RedirectResponse
      */
     public function store(AdminOnlyRequest $request, $user_id)
     {
@@ -69,7 +69,7 @@ 
      * @param AdminOnlyRequest $request
      * @param $user_id
      * @param $port_id
-     * @return mixed
+     * @return \Illuminate\Http\RedirectResponse
      */
     public function destroy(AdminOnlyRequest $request, $user_id, $port_id)
     {

app/DataTables/DeviceDataTable.php

Unused Use Statements -1 removed lines

@@ -25,7 +25,6 @@
 
 namespace App\DataTables;
 
-use App\Models\DeviceGroup;
 use App\Models\Device;
 
 class DeviceDataTable extends BaseDataTable

app/Http/Controllers/DeviceController.php

Unused Use Statements -1 removed lines

@@ -3,7 +3,6 @@
 namespace App\Http\Controllers;
 
 use App\DataTables\DeviceDataTable;
-use App\DataTables\DeviceGroupDataTable;
 use App\Models\DeviceGroup;
 use Dingo\Api\Http;
 use Dingo\Api\Routing\Helpers;

app/Http/Controllers/DeviceGroupController.php

Doc Comments +4 added lines, -4 removed lines

@@ -58,7 +58,7 @@ 
      * Store a newly created resource in storage.
      *
      * @param  \Illuminate\Http\Request $request
-     * @return \Illuminate\Http\Response
+     * @return \Illuminate\Http\JsonResponse
      */
     public function store(DeviceGroupRequest $request)
     {
@@ -71,7 +71,7 @@ 
      * Display the specified resource.
      *
      * @param  int $id
-     * @return \Illuminate\Http\Response
+     * @return \Illuminate\Http\RedirectResponse
      */
     public function show($id)
     {
@@ -95,7 +95,7 @@ 
      *
      * @param  \Illuminate\Http\Request $request
      * @param  int $id
-     * @return \Illuminate\Http\Response
+     * @return \Illuminate\Http\JsonResponse
      */
     public function update(DeviceGroupRequest $request, $id)
     {
@@ -110,7 +110,7 @@ 
      *
      * @param \Illuminate\Http\Request $request
      * @param  int $id
-     * @return \Illuminate\Http\Response
+     * @return \Illuminate\Http\JsonResponse
      * @throws \Exception
      */
     public function destroy(AdminOnlyRequest $request, $id)

app/Models/DeviceGroup.php

Doc Comments +1 added lines, -1 removed lines

@@ -203,7 +203,7 @@
      * Convert a v1 device group pattern to v2 style
      *
      * @param $pattern
-     * @return array
+     * @return string
      */
     private function convertV1Pattern($pattern)
     {

app/QueryBuilderFilter.php

Doc Comments +3 added lines

@@ -38,6 +38,9 @@
         return json_encode(self::generateMacroFilter('alert.macros.rule', self::generateTableFilter()));
     }
 
+    /**
+     * @param string $setting
+     */
     private static function generateMacroFilter($setting, $filter = [])
     {
         foreach (Settings::get($setting, []) as $key => $value) {