mshamaseen / laravel-repository-pattern

src/Bases/Contract.php

Doc Comments +11 added lines, -11 removed lines

@@ -26,30 +26,30 @@ 
     /**
      * @param array $data
      *
-     * @return mixed
+     * @return boolean
      */
     public function insert($data = []);
 
     /**
      * @param array $data
-     * @param $entityId
+     * @param integer $entityId
      *
      * @return bool
      */
     public function update($entityId, $data = []);
 
     /**
-     * @param $entityId
+     * @param integer $entityId
      *
-     * @return mixed
+     * @return boolean
      */
     public function delete($entityId);
 
     /**
-     * @param $entityId
+     * @param integer $entityId
      * @param array $columns
      *
-     * @return mixed
+     * @return Entity
      */
     public function find($entityId, $columns = ['*']);
 
@@ -57,7 +57,7 @@ 
      * @param array $filters
      * @param array $columns
      *
-     * @return mixed
+     * @return Entity
      */
     public function findBy($filters = [], $columns = ['*']);
 
@@ -73,7 +73,7 @@ 
      * @param int $limit
      * @param array $filters
      *
-     * @return Paginator
+     * @return \Illuminate\Contracts\Pagination\Paginator
      */
     public function simplePaginate($limit = 10, $filters = []);
 
@@ -97,7 +97,7 @@ 
      * @param array $filter
      * @param array $columns
      *
-     * @return mixed
+     * @return Entity
      */
     public function first($filter = [], $columns = ['*']);
 
@@ -140,13 +140,13 @@ 
 
     /**
      * @param int $entityId
-     * @return mixed
+     * @return boolean|null
      */
     public function restore($entityId = 0);
 
     /**
      * @param int $categoryId
-     * @return mixed
+     * @return boolean|null
      */
     public function forceDelete($categoryId = 0);
 }

src/Bases/Controller.php

Doc Comments -2 removed lines

@@ -96,7 +96,6 @@ 
     /**
      * Display a listing of the resource.
      *
-     * @param Request $request
      *
      * @return \Illuminate\Http\Response
      */
@@ -249,7 +248,6 @@ 
     /**
      * Update the specified resource in storage.
      *
-     * @param \Illuminate\Http\Request $request
      * @param int                      $entityId
      *
      * @return \Illuminate\Http\RedirectResponse

Spacing +23 added lines, -23 removed lines

@@ -58,7 +58,7 @@ 
         $this->breadcrumbs = new Collection();
 
         $language = $request->header('Language', 'en');
-        if (! in_array($language, \Config::get('app.locales'))) {
+        if (!in_array($language, \Config::get('app.locales'))) {
             $language = 'en';
         }
         $limit = $request->get('limit', 10);
@@ -81,10 +81,10 @@ 
         $this->interface = $interface;
         $this->isAPI = $request->expectsJson();
 
-        if (! $this->isAPI) {
+        if (!$this->isAPI) {
             $this->breadcrumbs = new Collection();
             $this->search = new Collection();
-            \View::share('pageTitle', $this->pageTitle.' | '.\Config::get('app.name'));
+            \View::share('pageTitle', $this->pageTitle . ' | ' . \Config::get('app.name'));
             \View::share('breadcrumbs', $this->breadcrumbs);
             \View::share('menu', $this->menu);
             \View::share('search', $this->search);
@@ -103,8 +103,8 @@ 
     public function index()
     {
         $data = $this->interface->simplePaginate($this->limit, $this->request->all());
-        if (! $this->isAPI) {
-            \View::share('pageTitle', 'List '.$this->pageTitle.' | '.\Config::get('app.name'));
+        if (!$this->isAPI) {
+            \View::share('pageTitle', 'List ' . $this->pageTitle . ' | ' . \Config::get('app.name'));
             $this->breadcrumbs->put('index', [
                 'link' => $this->routeIndex,
                 'text' => $this->pageTitle,
@@ -135,8 +135,8 @@ 
      */
     public function create()
     {
-        if (! $this->isAPI) {
-            \View::share('pageTitle', 'Create '.$this->pageTitle.' | '.\Config::get('app.name'));
+        if (!$this->isAPI) {
+            \View::share('pageTitle', 'Create ' . $this->pageTitle . ' | ' . \Config::get('app.name'));
             $this->breadcrumbs->put('create', [
                 'link' => $this->createRoute,
                 'text' => trans('common/others.create'),
@@ -156,7 +156,7 @@ 
     public function baseStore()
     {
         $entity = $this->interface->create($this->request->except(['_token', '_method']));
-        if (! $this->isAPI) {
+        if (!$this->isAPI) {
             if ($entity) {
                 return \Redirect::to($this->routeIndex)->with('message', __('messages.success'));
             }
@@ -187,11 +187,11 @@ 
     public function show($entityId)
     {
         $entity = $this->interface->find($entityId);
-        if (! $this->isAPI) {
-            if (! $entity) {
+        if (!$this->isAPI) {
+            if (!$entity) {
                 return \Redirect::to($this->routeIndex)->with('warning', __('messages.not_found'));
             }
-            \View::share('pageTitle', 'View '.$this->pageTitle.' | '.\Config::get('app.name'));
+            \View::share('pageTitle', 'View ' . $this->pageTitle . ' | ' . \Config::get('app.name'));
             $this->breadcrumbs->put('view', [
                 'link' => '',
                 'text' => $entity->name ?? $entity->title ?? __('messages.view'),
@@ -203,7 +203,7 @@ 
                 ->with('notes', $this->interface->notes($entityId))
                 ->with('files', $this->interface->files($entityId));
         }
-        if (! $entity) {
+        if (!$entity) {
             return response()->json(null, JsonResponse::HTTP_NOT_FOUND);
         }
 
@@ -223,8 +223,8 @@ 
     public function edit($entityId)
     {
         $entity = $this->interface->find($entityId);
-        if (! $this->isAPI) {
-            if (! $entity) {
+        if (!$this->isAPI) {
+            if (!$entity) {
                 return \Redirect::to($this->routeIndex)->with('warning', __('messages.not_found'));
             }
             $this->breadcrumbs->put('edit', [
@@ -263,8 +263,8 @@ 
             $saved = $this->interface->update($entityId, $this->request->except(['_token', '_method']));
         }
 
-        if (! $this->isAPI) {
-            if (! $entity) {
+        if (!$this->isAPI) {
+            if (!$entity) {
                 return \Redirect::to($this->routeIndex)->with('warning', __('messages.not_found'));
             }
 
@@ -275,7 +275,7 @@ 
             return \Redirect::to($this->routeIndex)->with('error', __('messages.not_modified'));
         }
 
-        if (! $entity) {
+        if (!$entity) {
             return response()->json(null, JsonResponse::HTTP_NOT_FOUND);
         }
 
@@ -304,8 +304,8 @@ 
         if ($entity) {
             $deleted = $this->interface->delete($entityId);
         }
-        if (! $this->isAPI) {
-            if (! $entity) {
+        if (!$this->isAPI) {
+            if (!$entity) {
                 return \Redirect::to($this->routeIndex)->with('warning', __('messages.not_found'));
             }
             if ($deleted) {
@@ -336,8 +336,8 @@ 
     {
         $entity = $this->interface->restore($entityId);
 
-        if (! $this->isAPI) {
-            if (! $entity) {
+        if (!$this->isAPI) {
+            if (!$entity) {
                 return \Redirect::to($this->routeIndex)->with('warning', __('messages.not_found'));
             }
             if ($entity) {
@@ -369,8 +369,8 @@ 
     {
         $entity = $this->interface->forceDelete($entityId);
 
-        if (! $this->isAPI) {
-            if (! $entity) {
+        if (!$this->isAPI) {
+            if (!$entity) {
                 return \Redirect::to($this->routeIndex)->with('warning', __('messages.not_found'));
             }
             if ($entity) {

src/Commands/Generator.php

Doc Comments -1 removed lines

@@ -93,7 +93,6 @@
     }
 
     /**
-     * @param string $name Class name
      * @param string $path Class path
      * @param string $folder default path to generate in
      * @param string $type define which kind of files should generate

Spacing +9 added lines, -9 removed lines

@@ -57,17 +57,17 @@ 
     public function handle()
     {
 
-        $file = explode("/", (string)$this->argument('name'));
+        $file = explode("/", (string) $this->argument('name'));
 
-        $this->repoName=$file[count($file) - 1];
+        $this->repoName = $file[count($file) - 1];
         unset($file[count($file) - 1]);
         $path = implode("/", $file);
 
-        if(count($file) == 0){
-            $this->repoNamespace=$this->repoName;
-        }else{
+        if (count($file) == 0) {
+            $this->repoNamespace = $this->repoName;
+        } else {
             $name = $file[count($file) - 1];
-            $this->repoNamespace=implode("\\", $file);
+            $this->repoNamespace = implode("\\", $file);
         }
 
 
@@ -77,7 +77,7 @@ 
         $this->generate($path, \Config::get('repository.interfaces_path'), 'Interface');
         $this->generate($path, \Config::get('repository.repositories_path'), 'Repository');
 
-        File::append(\Config::get('repository.route_path').'web.php', "\n" . 'Route::resource(\'' . str_plural($name) . "', '{$name}Controller');");
+        File::append(\Config::get('repository.route_path') . 'web.php', "\n" . 'Route::resource(\'' . str_plural($name) . "', '{$name}Controller');");
 
     }
 
@@ -89,7 +89,7 @@ 
      */
     protected function getStub($type)
     {
-        return file_get_contents(\Config::get('repository.resources_path')."/stubs/$type.stub");
+        return file_get_contents(\Config::get('repository.resources_path') . "/stubs/$type.stub");
     }
 
     /**
@@ -112,7 +112,7 @@ 
             ],
             $this->getStub($type)
         );
-        $path = $this->checkFolder(\Config::get('repository.app_path').'/'.$folder.$path."/");
+        $path = $this->checkFolder(\Config::get('repository.app_path') . '/' . $folder . $path . "/");
         file_put_contents($path . "{$this->repoName}{$type}.php", $template);
 
     }

Braces +1 added lines, -1 removed lines

@@ -65,7 +65,7 @@
 
         if(count($file) == 0){
             $this->repoNamespace=$this->repoName;
-        }else{
+        } else{
             $name = $file[count($file) - 1];
             $this->repoNamespace=implode("\\", $file);
         }

src/config/repository.php

Spacing +1 added lines, -1 removed lines

@@ -6,7 +6,7 @@
  * Time: 09:58 ص
  */
 return [
-    'app_path' => realpath(__DIR__.'/../app/'),
+    'app_path' => realpath(__DIR__ . '/../app/'),
     'route_path' => realpath('routes/'),
     'resources_path' => realpath('resources/'),
 

src/GeneratorServiceProvider.php

Spacing +5 added lines, -5 removed lines

@@ -26,13 +26,13 @@
         }
 
         $this->publishes([
-            __DIR__.'/config' => realpath('config'),
-        ],'repository-config');
+            __DIR__ . '/config' => realpath('config'),
+        ], 'repository-config');
         
-        $this->mergeConfigFrom(__DIR__.'/config/repository.php', 'repository-generator');
+        $this->mergeConfigFrom(__DIR__ . '/config/repository.php', 'repository-generator');
         $this->publishes([
-            __DIR__.'/stubs' => \Config::get('repository.resources_path',realpath(__DIR__.'/../../../../resources/'))."/stubs/",
-        ],'repository-stub');
+            __DIR__ . '/stubs' => \Config::get('repository.resources_path', realpath(__DIR__ . '/../../../../resources/')) . "/stubs/",
+        ], 'repository-stub');
     }
 
     /**

src/Bases/Requests.php

Spacing +3 added lines, -3 removed lines

@@ -29,14 +29,14 @@
     protected function failedValidation(Validator $validator)
     {
 
-        if (strpos($this->path(),'api') !== false) {
+        if (strpos($this->path(), 'api') !== false) {
             $errors = (new ValidationException($validator))->errors();
             throw new HttpResponseException(response()->json(['success' => false, 'errors' => $errors,
             ], JsonResponse::HTTP_UNPROCESSABLE_ENTITY));
         }
     }
 
-    public function rules(){
-        \App::setLocale($this->header('Language','en'));
+    public function rules() {
+        \App::setLocale($this->header('Language', 'en'));
     }
 }