From 39f0ca3c1cf0464925c21d698bd0339d9df0842f Mon Sep 17 00:00:00 2001
From: Lukas Halbritter <halbritl@informatik.uni-freiburg.de>
Date: Tue, 25 Sep 2018 15:11:25 +0200
Subject: [PATCH] Fixed some issues in the person form: 1. Added csrf field. 2.
 Added dropdown menue for the title. 3. Added an input for the date of birth.
 4. Fixed the appearance of the member state edit section.

---
 resources/views/layout/forms/person.blade.php | 38 ++++++++++++++++---
 1 file changed, 32 insertions(+), 6 deletions(-)

diff --git a/resources/views/layout/forms/person.blade.php b/resources/views/layout/forms/person.blade.php
index 652802d..11e70c9 100644
--- a/resources/views/layout/forms/person.blade.php
+++ b/resources/views/layout/forms/person.blade.php
@@ -1,16 +1,34 @@
 @extends('layout.master')
 
+@section('subnav')
+    <a href='{{ url()->previous() }}'>Zurück</a>
+@endsection
+
 @section('content')
     <h1>@yield('title')</h1>
     <form action="/people/@if (!is_null($person->id)) {{ $person->id }} @endif" method='post'>
         @if (!is_null($person->id))
-            {!! method_field('patch') !!}
+            {{ method_field('patch') }}
         @endif
+        {{ csrf_field() }}
         <h3>Stammdaten</h3>
         <table>
             <tr>
                 <td>{{ Form::label('title', 'Anrede') }}</td>
-                <td>{{ Form::text('title', $person->title) }}</td>
+                <td>
+                    @if (!is_null($person->id))
+                        {{ Form::select('title', [
+                            'Herr' => 'Herr',
+                            'Frau' => 'Frau',
+                            'Firma' => 'Firma'],
+                            $person->title) }}
+                    @else
+                        {{ Form::select('title', [
+                            'Herr' => 'Herr',
+                            'Frau' => 'Frau',
+                            'Firma' => 'Firma']) }}
+                    @endif
+                </td>
             </tr>
             <tr>
                 <td>{{ Form::label('name_or_company', 'Name / Firma') }}</td>
@@ -44,6 +62,10 @@
                 <td>{{ Form::label('phone', 'Telefon') }}</td>
                 <td>{{ Form::text('phone', $person->phone) }}</td>
             </tr>
+            <tr>
+                <td>{{ Form::label('date_of_birth', 'Geburtsdatum') }}</td>
+                <td>{{ Form::text('date_of_birth', $person->date_of_birth) }}</td>
+            </tr>
         </table>
         
         @if ($person->title != 'Firma')
@@ -53,18 +75,22 @@
                     <th>Eintrittsdatum</th>
                     <th>Austrittsdatum</th>
                 </tr>
+                {{-- List all member states, which are already in the database --}}
                 @if ($person->wasMember())
                     @foreach ($person->memberStates as $memberState)
                         <tr>
-                            <td>{{ Form::hidden('meberStateId[]', $memberState->id) }}
+                            <td>{{ Form::hidden('memberStateId[]', $memberState->id) }}
                                 {{ Form::text('joining_date[]', $memberState->joining_date) }}</td>
                             <td>{{ Form::text('leaving_date[]', $memberState->leaving_date) }}</td>
                         </tr>
                     @endforeach
-                @else
+                @endif
+
+                {{-- Create empty input field for a new member state record. --}}
+                @if (!$person->isMember())
                     <tr>
-                        <td>{{ Form::text('joining_date[]', $memberState->joining_date) }}</td>
-                        <td>{{ Form::text('leaving_date[]', $memberState->leaving_date) }}</td>
+                        <td>{{ Form::text('joining_date[]') }}</td>
+                        <td>{{ Form::text('leaving_date[]') }}</td>
                     </tr>
                 @endif
             </table>