fix validation for pages

add create btn to edit pages

v3.0.3

Author: ctf0

logs/v3.0.1.txt

@@ -0,0 +1,2 @@
+- fix page validation
+- add create new btn to edit pages

logs/v3.0.3.txt

@@ -23,29 +23,38 @@ protected function sT_uP_Validaiton($request, $id = null)
             $routename = 'required|unique:pages,route_name,' . $id;
         }
 
+        $customMessages = [
+            'template.required_without' => __('validation.required_without', [
+                'attribute' => __('SimpleMenu::validation.attributes.template'),
+                'values'    => __('SimpleMenu::validation.attributes.action'),
+            ]),
+            'route_name.required' => __('validation.required', ['attribute' => __('SimpleMenu::validation.attributes.route_name')]),
+            'url.required'        => __('validation.required', ['attribute' => __('SimpleMenu::validation.attributes.url')]),
+            'title.required'      => __('validation.required', ['attribute' => __('SimpleMenu::validation.attributes.title')]),
+         ];
+
         $validator = Validator::make($request->all(), [
-            'template'    => 'required_without:action',
-            'route_name'  => $routename,
-        ]);
+            'template'   => 'required_without:action',
+            'route_name' => $routename,
+        ], $customMessages);
 
         // because laravel still pretty fucked up when it comes to showing array input errors
-        $validator->after(function ($validator) use ($request) {
+        $validator->after(function ($validator) use ($request, $customMessages) {
             // url
             if (!array_filter($request->url)) {
-                $validator->errors()->add('url', 'Url is required');
+                $validator->errors()->add('url', $customMessages['url.required']);
             }
 
             // title
             if (!array_filter($request->title)) {
-                $validator->errors()->add('title', 'The Title is required');
+                $validator->errors()->add('title', $customMessages['title.required']);
             }
         });
 
         if ($validator->fails()) {
-            throw new ValidationException($validator, $this->buildFailedValidationResponse(
-                $request,
-                $this->formatValidationErrors($validator)
-            ));
+            throw (new ValidationException($validator))
+                        ->errorBag('default')
+                        ->redirectTo(url()->previous());
         }
     }
 

src/Controllers/Admin/Traits/PageOps.php

@@ -0,0 +1,11 @@
+<?php
+
+return [
+    'attributes' => [
+        'title'         => 'Title',
+        'url'           => 'Url',
+        'template'      => 'Template',
+        'action'        => 'Action',
+        'route_name'    => 'Route Name',
+    ],
+];

src/resources/lang/en/validation.php

@@ -15,12 +15,12 @@
         <div class="field has-addons">
             <div class="control is-expanded">
                 {{ Form::text('name', null, ['class' => 'input', 'placeholder'=>'name']) }}
+                @if($errors->has('name'))
+                    <p class="help is-danger">
+                        {{ $errors->first('name') }}
+                    </p>
+                @endif
             </div>
-            @if($errors->has('name'))
-                <p class="help is-danger">
-                    {{ $errors->first('name') }}
-                </p>
-            @endif
             <div class="control">
                 {{ Form::submit(trans('SimpleMenu::messages.app_save'), ['class' => 'button is-success']) }}
             </div>

src/resources/views/admin/bulma/menus/create.blade.php

@@ -4,6 +4,7 @@
 @section('sub')
     <h3 class="title">
         <a href="{{ url()->previous() }}">Go Back</a>
+        <a href="{{ route($crud_prefix.'.menus.create') }}" class="button is-success">@lang('SimpleMenu::messages.app_add_new')</a>
     </h3>
 
     <menu-comp inline-template
@@ -21,12 +22,12 @@
                 <div class="field has-addons">
                     <div class="control is-expanded">
                         {{ Form::text('name', $menu->name, ['class' => 'input']) }}
+                        @if($errors->has('name'))
+                            <p class="help is-danger">
+                                {{ $errors->first('name') }}
+                            </p>
+                        @endif
                     </div>
-                    @if($errors->has('name'))
-                        <p class="help is-danger">
-                            {{ $errors->first('name') }}
-                        </p>
-                    @endif
                     <div class="control">
                         {{ Form::submit(trans('SimpleMenu::messages.app_update'), ['class' => 'button is-warning']) }}
                     </div>

src/resources/views/admin/bulma/menus/edit.blade.php

@@ -9,6 +9,7 @@
 @section('sub')
     <h3 class="title">
         <a href="{{ url()->previous() }}">Go Back</a>
+        <a href="{{ route($crud_prefix.'.pages.create') }}" class="button is-success">@lang('SimpleMenu::messages.app_add_new')</a>
     </h3>
     <hr>
 

src/resources/views/admin/bulma/pages/edit.blade.php

@@ -4,6 +4,7 @@
 @section('sub')
     <h3 class="title">
         <a href="{{ url()->previous() }}">Go Back</a>
+        <a href="{{ route($crud_prefix.'.permissions.create') }}" class="button is-success">@lang('SimpleMenu::messages.app_add_new')</a>
     </h3>
 
     {{ Form::model($permission, ['method' => 'PUT', 'route' => [$crud_prefix.'.permissions.update', $permission->id]]) }}

src/resources/views/admin/bulma/permissions/edit.blade.php

@@ -4,6 +4,7 @@
 @section('sub')
     <h3 class="title">
         <a href="{{ url()->previous() }}">Go Back</a>
+        <a href="{{ route($crud_prefix.'.roles.create') }}" class="button is-success">@lang('SimpleMenu::messages.app_add_new')</a>
     </h3>
 
     {{ Form::model($role, ['method' => 'PUT', 'route' => [$crud_prefix.'.roles.update', $role->id]]) }}

src/resources/views/admin/bulma/roles/edit.blade.php

@@ -4,6 +4,7 @@
 @section('sub')
     <h3 class="title">
         <a href="{{ url()->previous() }}">Go Back</a>
+        <a href="{{ route($crud_prefix.'.users.create') }}" class="button is-success">@lang('SimpleMenu::messages.app_add_new')</a>
     </h3>
     <hr>