From 8a189196899989fd611a710bbbfc6bbbf31b73cc Mon Sep 17 00:00:00 2001
From: Harald Eilertsen <haraldei@anduin.net>
Date: Fri, 17 Sep 2021 16:59:01 +0200
Subject: Move New Venue form to a separate partial.

---
 includes/admin/views/_edit_concert_form.php |  9 ++------
 includes/admin/views/_new_venue_form.php    | 34 +++++++++++++++++++++++++++++
 includes/admin/views/giglog_admin_page.php  | 11 ++++++++--
 3 files changed, 45 insertions(+), 9 deletions(-)
 create mode 100644 includes/admin/views/_new_venue_form.php

diff --git a/includes/admin/views/_edit_concert_form.php b/includes/admin/views/_edit_concert_form.php
index 61f2bf4..7b0164e 100644
--- a/includes/admin/views/_edit_concert_form.php
+++ b/includes/admin/views/_edit_concert_form.php
@@ -45,7 +45,7 @@ if (!class_exists("GiglogAdmin_EditConcertForm"))
             else
                 $c = new GiglogAdmin_Concert((object)[]);
 
-            $content='<div><h3>Form to create/edit concerts and venues</h3><br></div><div class="editform"><div class="concertform">';
+            $content='<div class="concertform">';
             $content.='<form method="POST" action="" class="concert" >'
                 .'<div class="concertitems"><strong>CONCERT DETAILS</strong><br><br><fieldset>'
                 . wp_nonce_field( plugin_basename( __FILE__ ), 'giglog_edit_concert_nonce' )
@@ -71,12 +71,7 @@ if (!class_exists("GiglogAdmin_EditConcertForm"))
                 .'<label for="rev2">Text2:</label>'.$this->user_dropdown_for_role($c,'rev2').'<br>';
 
             $content.='<fieldset></div></form></div>';
-            $content.='<div class="venueform"><form method="POST" action="" class="venue" ><strong>VENUE DETAILS</strong><br><br>'
-                .'<fieldset><label for="venue">Venue Name:</label><input type="text" id="venuename" name="venuename"><br>'
-                .'<label for="eventurl">Venue City:</label><input type="text" id="venuecity" name="venuecity"><br>'
-                .'<p><input type="submit" name="newvenue" value="Create New Venue"></p>'
-                .'<fieldset></form></div>';
-            $content.='</div>';
+
             return $content;
         }
     }
diff --git a/includes/admin/views/_new_venue_form.php b/includes/admin/views/_new_venue_form.php
new file mode 100644
index 0000000..d17f5e0
--- /dev/null
+++ b/includes/admin/views/_new_venue_form.php
@@ -0,0 +1,34 @@
+<?php
+// SPDX-FileCopyrightText: 2021 Andrea Chirulescu <andrea.chirulescu@gmail.com>
+// SPDX-FileCopyrightText: 2021 Harald Eilertsen <haraldei@anduin.net>
+//
+// SPDX-License-Identifier: AGPL-3.0-or-later
+
+if ( !class_exists( "GiglogAdmin_NewVenueForm" ) )
+{
+    class GiglogAdmin_NewVenueForm
+    {
+        public function render() : string
+        {
+            return
+                '<div class="venueform">'
+                . '<p><strong>VENUE DETAILS</strong></p>'
+                . '<form method="POST" action="" class="venue">'
+                . '  <fieldset>'
+                . '    <div class="field venue_name_field">'
+                . '      <label for="venue">Venue Name:</label>'
+                . '      <input type="text" id="venuename" name="venuename">'
+                . '    </div>'
+                . '    <div class="field venue_city_field">'
+                . '      <label for="venuecity">Venue City:</label>'
+                . '      <input type="text" id="venuecity" name="venuecity">'
+                . '    </div>'
+                . '    <div class="actions">'
+                . '      <input type="submit" name="newvenue" value="Create New Venue">'
+                . '    </div>'
+                . '  <fieldset>'
+                . '</form>'
+                . '</div>';
+        }
+    }
+}
diff --git a/includes/admin/views/giglog_admin_page.php b/includes/admin/views/giglog_admin_page.php
index fa853fb..b7f6247 100644
--- a/includes/admin/views/giglog_admin_page.php
+++ b/includes/admin/views/giglog_admin_page.php
@@ -9,6 +9,7 @@ if ( !class_exists( 'GiglogAdmin_AdminPage' ) ) {
     require_once __DIR__ . '/../../venue.php';
     require_once __DIR__ . '/_concerts_table.php';
     require_once __DIR__ . '/_edit_concert_form.php';
+    require_once __DIR__ . '/_new_venue_form.php';
 
     class GiglogAdmin_AdminPage
     {
@@ -52,8 +53,14 @@ if ( !class_exists( 'GiglogAdmin_AdminPage' ) ) {
             </div>
             <?php
             if (current_user_can('administrator')) {
-                $form = new GiglogAdmin_EditConcertForm();
-                echo $form->render();
+                $edit_form = new GiglogAdmin_EditConcertForm();
+                $venue_form = new GiglogAdmin_NewVenueForm(); ?>
+                <div>
+                    <h3>Form to create/edit concerts and venues</h3>
+                </div>
+                <div class="editform">
+                    <?php echo $edit_form->render() . $venue_form->render(); ?>
+                </div><?php
             }
         }
 
-- 
cgit v1.2.3