Fixed some scrutinizer issues

Author: maab16

README.md

@@ -4,3 +4,27 @@
 
 # laravel-database-manager
 Make your database simple and easyer
+
+# Install package
+
+```
+composer require codexshaper/laravel-database-manager
+```
+
+# Setup database manager
+
+```
+php artisan dbm:install
+```
+
+# Create admin account to access all features
+
+```
+php artisan dbm:admin 'user' 'action' 'options'
+```
+Example
+```
+php artisan [email protected] create --columns=email
+```
+
+In this case ```email``` must be exists in your users table and ```[email protected]``` must be a record

src/Database/Drivers/MongoDB.php

@@ -390,7 +390,7 @@ public function getColumns($collectionName)
      *
      * @param   string $collectionName
      *
-     * @return  \Illuminate\Support\Collection
+     * @return  array
      */
     public function getColumnsName($collectionName)
     {

src/Database/Drivers/MongoDB/Field.php

@@ -10,7 +10,6 @@ class Field
      * Create MongoDB collection field
      *
      * @param string $collection
-     * @param clousure $clousure
      *
      * @return  \CodexShaper\DBM\Database\Drivers\MongoDB\Collection
      */

src/Database/Drivers/MongoDB/Type.php

@@ -56,18 +56,10 @@ public static function decimal(string $value)
     /**
      * Get javascript
      *
-     * @param   string  $code
-     * @param   array   $scope
-     *
      * @return  \MongoDB\BSON\Javascript
      */
-    public static function javascript(string $code, $scope = [])
+    public static function javascript(string $code, array $scope = [])
     {
-        if (!is_array($scope) || !is_array()) {
-            throw new \Exception($scope . " should be array or object");
-
-        }
-
         return new Javascript($code, $scope);
     }
     /**
@@ -102,9 +94,9 @@ public static function objectId(string $value)
      *
      * @return  \MongoDB\BSON\Regex
      */
-    public static function regex(string $pattern, string $flags)
+    public static function regex(string $pattern, string $flags = "")
     {
-        return new Regex($value);
+        return new Regex($pattern, $flags);
     }
     /**
      * Get timestamp
@@ -118,7 +110,7 @@ public static function timestamp(int $increment, int $timestamp)
     /**
      * Get datetime
      *
-     * @param int|nul $milliseconds
+     * @param int|null $milliseconds
      *
      * @return  \MongoDB\BSON\UTCDateTime
      */

src/Database/Platforms/Platform.php

@@ -13,7 +13,7 @@ abstract public static function registerCustomTypeOptions();
      *
      * @param string $platformName
      *
-     * @return \Illuminate\Support\Collection
+     * @return string
      */
     public static function getPlatform($platformName)
     {

src/Database/Schema/Index.php

@@ -64,7 +64,7 @@ public static function create($index)
      * @param string $type
      * @param string|null $table
      *
-     * @return \Doctrine\DBAL\Schema\Index
+     * @return string
      */
     public static function createName($columns, $type, $table = null)
     {

src/Database/Schema/Table.php

@@ -83,9 +83,9 @@ public static function getColumnsName($tableName)
      *
      * @param array $table
      *
-     * @return void
+     * @return array|object|void
      */
-    public static function create($table = [])
+    public static function create($table)
     {
         if (!is_array($table)) {
             $table = json_decode($table, true);
@@ -108,7 +108,7 @@ public static function create($table = [])
      *
      * @return true|void
      */
-    public static function update($table = [])
+    public static function update($table)
     {
         if (!is_array($table)) {
             $table = json_decode($table, true);
@@ -125,7 +125,7 @@ public static function update($table = [])
      *
      * @param string $tableName
      *
-     * @return true|void
+     * @return void
      */
     public static function drop($tableName)
     {
@@ -142,7 +142,7 @@ public static function drop($tableName)
      *
      * @return \Doctrine\DBAL\Schema\Table
      */
-    public static function prepareTable($table = [])
+    public static function prepareTable($table)
     {
 
         if (!is_array($table)) {
@@ -246,7 +246,7 @@ public static function getForeignKeys(DoctrineTable $table)
      *
      * @param string $tableName
      *
-     * @return true
+     * @return boolean
      */
     public static function exists($tableName)
     {
@@ -264,7 +264,7 @@ public static function exists($tableName)
      * Get tables with pagination
      *
      * @param int $perPage
-     * @param string|null $page
+     * @param int|null $page
      * @param array $options
      * @param string $query
      *

src/Database/Schema/UpdateManager.php

@@ -21,7 +21,7 @@ class UpdateManager
      *
      * @return void
      */
-    public function update($table = [])
+    public function update($table)
     {
         if (!is_array($table)) {
             $table = json_decode($table, true);

src/Database/Types/Common/CharType.php

@@ -18,8 +18,8 @@ class CharType extends Type
      */
     public function getSQLDeclaration(array $fieldDeclaration, AbstractPlatform $platform)
     {
-        $field['length'] = empty($field['length']) ? 1 : $field['length'];
+        $fieldDeclaration['length'] = empty($fieldDeclaration['length']) ? 1 : $fieldDeclaration['length'];
 
-        return "char({$field['length']})";
+        return "char({$fieldDeclaration['length']})";
     }
 }

src/Http/Controllers/BackupController.php

@@ -22,7 +22,7 @@ public function index()
     /**
      * Get all backup files
      *
-     * @return \Illuminate\Http\Response
+     * @return \Illuminate\Http\JsonResponse
      */
     public function backups(Request $request)
     {
@@ -54,7 +54,7 @@ public function backups(Request $request)
     /**
      * Create new backup
      *
-     * @return \Illuminate\Http\Response
+     * @return \Illuminate\Http\JsonResponse
      */
     public function backup(Request $request)
     {
@@ -92,7 +92,7 @@ public function backup(Request $request)
     /**
      * Restore from a specific backup
      *
-     * @return \Illuminate\Http\Response
+     * @return \Illuminate\Http\JsonResponse
      */
     public function restore(Request $request)
     {
@@ -124,7 +124,7 @@ public function restore(Request $request)
     /**
      * Return specific backup file for download
      *
-     * @return \Illuminate\Http\Response
+     * @return \Illuminate\Http\JsonResponse
      */
     public function download(Request $request)
     {
@@ -153,7 +153,7 @@ public function download(Request $request)
     /**
      * Remove a backup
      *
-     * @return \Illuminate\Http\Response
+     * @return \Illuminate\Http\JsonResponse
      */
     public function delete(Request $request)
     {