Aircraft Controller refactor and localization

README.md

@@ -32,10 +32,6 @@
 
 - Form Localization
 
-## Aircrafts
-
-- Form Localization
-
 ## Aircraft Type Groups
 
 - Form Localization

controllers/Aircraft.php

@@ -3,7 +3,7 @@
 use Backend\Classes\Controller;
 use BackendMenu;
 
-class Aircrafts extends Controller
+class Aircraft extends Controller
 {
   public $implement = ['Backend\Behaviors\ListController', 'Backend\Behaviors\FormController'];
 

controllers/aircraft/_list_toolbar.htm

@@ -1,5 +1,5 @@
 <div data-control="toolbar">
-  <a href="<?= Backend::url('germanairlinesva/fleet/aircrafts/create') ?>" class="btn btn-primary oc-icon-plus"
+  <a href="<?= Backend::url('germanairlinesva/fleet/aircraft/create') ?>" class="btn btn-primary oc-icon-plus"
     ><?= e(trans('backend::lang.form.create')) ?></a
   >
   <button

controllers/aircraft/config_form.yaml

@@ -0,0 +1,10 @@
+name: Aircraft
+form: $/germanairlinesva/fleet/models/aircraft/fields.yaml
+modelClass: GermanAirlinesVa\Fleet\Models\Aircraft
+defaultRedirect: germanairlinesva/fleet/aircraft
+create:
+  redirect: 'germanairlinesva/fleet/aircraft/update/:id'
+  redirectClose: germanairlinesva/fleet/aircraft
+update:
+  redirect: germanairlinesva/fleet/aircraft
+  redirectClose: germanairlinesva/fleet/aircraft

controllers/aircraft/config_list.yaml

@@ -1,6 +1,6 @@
 list: $/germanairlinesva/fleet/models/aircraft/columns.yaml
 modelClass: GermanAirlinesVa\Fleet\Models\Aircraft
-title: Aircrafts
+title: Aircraft
 noRecordsMessage: 'backend::lang.list.no_records'
 showSetup: true
 showCheckboxes: true
@@ -9,4 +9,4 @@ toolbar:
   buttons: list_toolbar
   search:
     prompt: 'backend::lang.list.search_prompt'
-recordUrl: 'germanairlinesva/fleet/aircrafts/update/:id'
+recordUrl: 'germanairlinesva/fleet/aircraft/update/:id'

controllers/aircraft/create.htm

@@ -35,7 +35,7 @@
     </button>
     <span class="btn-text">
       <?= e(trans('backend::lang.form.or')) ?>
-      <a href="<?= Backend::url('germanairlinesva/fleet/aircrafts') ?>"><?= e(trans('backend::lang.form.cancel')) ?></a>
+      <a href="<?= Backend::url('germanairlinesva/fleet/aircraft') ?>"><?= e(trans('backend::lang.form.cancel')) ?></a>
     </span>
   </div>
 </div>
@@ -45,7 +45,7 @@
 <?php else: ?>
 <p class="flash-message static error"><?= e(trans($this->fatalError)) ?></p>
 <p>
-  <a href="<?= Backend::url('germanairlinesva/fleet/aircrafts') ?>" class="btn btn-default"
+  <a href="<?= Backend::url('germanairlinesva/fleet/aircraft') ?>" class="btn btn-default"
     ><?= e(trans('backend::lang.form.return_to_list')) ?></a
   >
 </p>

controllers/aircraft/preview.htm

@@ -1,6 +1,6 @@
 <?php Block::put('breadcrumb') ?>
 <ul>
-  <li><a href="<?= Backend::url('germanairlinesva/fleet/aircrafts') ?>">Aircrafts</a></li>
+  <li><a href="<?= Backend::url('germanairlinesva/fleet/aircraft') ?>">Aircrafts</a></li>
   <li><?= e($this->pageTitle) ?></li>
 </ul>
 <?php Block::endPut() ?>
@@ -14,7 +14,7 @@
 <?php endif ?>
 
 <p>
-  <a href="<?= Backend::url('germanairlinesva/fleet/aircrafts') ?>" class="btn btn-default oc-icon-chevron-left">
+  <a href="<?= Backend::url('germanairlinesva/fleet/aircraft') ?>" class="btn btn-default oc-icon-chevron-left">
     <?= e(trans('backend::lang.form.return_to_list')) ?>
   </a>
 </p>

controllers/aircraft/update.htm

@@ -1,6 +1,6 @@
 <?php Block::put('breadcrumb') ?>
 <ul>
-  <li><a href="<?= Backend::url('germanairlinesva/fleet/aircrafts') ?>">Aircrafts</a></li>
+  <li><a href="<?= Backend::url('germanairlinesva/fleet/aircraft') ?>">Aircrafts</a></li>
   <li><?= e($this->pageTitle) ?></li>
 </ul>
 <?php Block::endPut() ?>
@@ -44,7 +44,7 @@
 
     <span class="btn-text">
       <?= e(trans('backend::lang.form.or')) ?>
-      <a href="<?= Backend::url('germanairlinesva/fleet/aircrafts') ?>"><?= e(trans('backend::lang.form.cancel')) ?></a>
+      <a href="<?= Backend::url('germanairlinesva/fleet/aircraft') ?>"><?= e(trans('backend::lang.form.cancel')) ?></a>
     </span>
   </div>
 </div>
@@ -53,7 +53,7 @@
 <?php else: ?>
 <p class="flash-message static error"><?= e(trans($this->fatalError)) ?></p>
 <p>
-  <a href="<?= Backend::url('germanairlinesva/fleet/aircrafts') ?>" class="btn btn-default"
+  <a href="<?= Backend::url('germanairlinesva/fleet/aircraft') ?>" class="btn btn-default"
     ><?= e(trans('backend::lang.form.return_to_list')) ?></a
   >
 </p>

controllers/aircrafts/config_form.yaml

@@ -1,10 +0,0 @@
-name: Aircrafts
-form: $/germanairlinesva/fleet/models/aircraft/fields.yaml
-modelClass: GermanAirlinesVa\Fleet\Models\Aircraft
-defaultRedirect: germanairlinesva/fleet/aircrafts
-create:
-  redirect: 'germanairlinesva/fleet/aircrafts/update/:id'
-  redirectClose: germanairlinesva/fleet/aircrafts
-update:
-  redirect: germanairlinesva/fleet/aircrafts
-  redirectClose: germanairlinesva/fleet/aircrafts

lang/en/lang.php

@@ -4,9 +4,15 @@
     'description' => '',
   ],
   'aircraft_manufacturers' => [
-    'name' => 'Name',
-    'link' => 'Link',
-    'history' => 'History',
+    'name' => 'Manufacturer Name',
+    'link' => 'Link to Homepage',
+    'history' => 'History of Manufacturer',
+  ],
+  'aircrafts' => [
+    'type' => 'Aircraft Type',
+    'registration' => 'Aircraft Registration',
+    'name' => 'Aircraft Name',
+    'home_base' => 'Aircraft Home Base',
   ],
   'menu' => [
     'main' => 'GA Fleet',

models/Aircraft.php

@@ -24,7 +24,12 @@ class Aircraft extends Model
   /**
    * @var array Validation rules
    */
-  public $rules = [];
+  public $rules = [
+    'aircraft_type' => 'required',
+    'registration' => 'required',
+    'name' => 'required',
+    'home_airport_id' => 'required',
+  ];
 
   public $belongsTo = [
     'aircraft_type' => 'GermanAirlinesVa\Fleet\Models\AircraftType',

models/aircraft/fields.yaml

@@ -1,23 +1,23 @@
 fields:
   aircraft_type:
-    label: Type
+    label: 'germanairlinesva.fleet::lang.aircrafts.type'
     nameFrom: type
     descriptionFrom: description
     span: auto
     required: 1
     type: relation
   registration:
-    label: Registration
+    label: 'germanairlinesva.fleet::lang.aircrafts.registration'
     span: auto
     required: 1
     type: text
   name:
-    label: Name
+    label: 'germanairlinesva.fleet::lang.aircrafts.name'
     span: auto
     required: 1
     type: text
   home_airport_id:
-    label: 'Home Base'
+    label: 'germanairlinesva.fleet::lang.aircrafts.home_base'
     span: auto
     required: 1
     type: number