🐛 Move opening and closing 'main' tags (#132 )

* 🐛 Move opening and closing 'main' tags Signed-off-by: Luke Tainton <luke@tainton.uk> * Restyled by php-cs-fixer (#133) Co-authored-by: Restyled.io <commits@restyled.io> Co-authored-by: restyled-io[bot] <32688539+restyled-io[bot]@users.noreply.github.com> Co-authored-by: Restyled.io <commits@restyled.io>
2020-08-31 19:16:33 +01:00
3 changed files with 30 additions and 22 deletions
--- a/app/includes/footer.php
+++ b/app/includes/footer.php
@@ -1,3 +1,4 @@
+</main>
 <footer class="footer mt-auto py-3">
  <div class="text-center text-muted">
    <?php
--- a/app/includes/header.php
+++ b/app/includes/header.php
@@ -10,7 +10,7 @@
    <meta name="viewport" content="width=device-width, initial-scale=1, shrink-to-fit=no">
    <meta name="theme-color" content="#563d7c">
    <link rel="icon" type="image/png" href="/favicon.png">
-    <title><?php echo( $PAGE_TITLE ); ?></title>
+    <title><?php echo($PAGE_TITLE); ?></title>

    <!-- Bootstrap core CSS -->
    <link rel="stylesheet" href="/css/bootstrap.min.css">
@@ -19,6 +19,8 @@
  </head>

  <body class="d-flex flex-column h-100">
+    <!-- Begin page content -->
+    <main role="main" class="flex-shrink-0">
    <header>
    <!-- Fixed navbar -->
    <nav class="navbar navbar-expand-md navbar-dark fixed-top bg-dark">
@@ -32,10 +34,14 @@
                <a class="nav-link" href="/">Home</a>
                </li>
                <li class="nav-item">
-                <a class="nav-link <?php if (!is_signed_in()) {echo(' disabled');} ?>" href="/new">New request</a>
+                <a class="nav-link <?php if (!is_signed_in()) {
+    echo(' disabled');
+} ?>" href="/new">New request</a>
                </li>
                <li class="nav-item">
-                <a class="nav-link <?php if (!is_signed_in()) {echo(' disabled');} ?>" href="/existing">Existing requests</a>
+                <a class="nav-link <?php if (!is_signed_in()) {
+    echo(' disabled');
+} ?>" href="/existing">Existing requests</a>
                </li>
            </ul>
        <div class="mt-2 mt-md-0">
--- a/app/public/index.php
+++ b/app/public/index.php
@@ -3,21 +3,17 @@
    require_once __DIR__ . "/../includes/header.php";

    if (is_signed_in()) {
-      $requests = get_my_open_requests($db);
-      $subscriptions = get_open_subscribed_requests($db);
+        $requests = get_my_open_requests($db);
+        $subscriptions = get_open_subscribed_requests($db);
    }

 ?>


-
-<!-- Begin page content -->
-<main role="main" class="flex-shrink-0">
-
  <section>
    <?php
-      if(isset($alert)) {
-        echo("
+      if (isset($alert)) {
+          echo("
        <div class='container'>
          <div class='alert alert-" . $alert[0] . " alert-dismissible fade show' role='alert'>
            " . $alert[1] . "
@@ -27,7 +23,7 @@
          </div>
        </div>
      ");
-      unset($alert);
+          unset($alert);
      }
    ?>
  </section>
@@ -37,7 +33,11 @@
      <h1>Welcome to <?php echo($_ENV['APP_NAME']); ?></h1>
      <p class="lead text-muted">
        <?php
-          if ($_ENV['APP_NAME'] == "FHeD") {echo("The Free HelpDesk");} else {echo($_ENV['APP_NAME']);};
+          if ($_ENV['APP_NAME'] == "FHeD") {
+              echo("The Free HelpDesk");
+          } else {
+              echo($_ENV['APP_NAME']);
+          };
        ?>
        is the one-stop shop for all of your IT-related needs. Let us know how we can help you by opening a request.
      </p>
@@ -63,10 +63,10 @@
            <ul class="list-group list-group-flush">
              <?php
                if (count($requests) == 0) {
-                  echo("<center><b>No open tickets</b></center>");
+                    echo("<center><b>No open tickets</b></center>");
                } else {
-                  foreach($requests as $tkt) {
-              ?>
+                    foreach ($requests as $tkt) {
+                        ?>
              <li class="list-group-item">
                <div class="container">
                  <div class="row">
@@ -80,7 +80,9 @@
                  </div>
                </div>
              </li>
-              <?php } } ?>
+              <?php
+                    }
+                } ?>
            </ul>
          </div>
        </div>
@@ -93,9 +95,9 @@
            <ul class="list-group list-group-flush">
              <?php
                if (count($subscriptions) == 0) {
-                  echo("<center><b>No subscribed tickets</b></center>");
+                    echo("<center><b>No subscribed tickets</b></center>");
                } else {
-                  foreach($subscriptions as $sub) { ?>
+                    foreach ($subscriptions as $sub) { ?>
              <li class="list-group-item">
                <div class="container">
                  <div class="row">
@@ -109,7 +111,8 @@
                  </div>
                </div>
              </li>
-              <?php } } ?>
+              <?php }
+                } ?>
            </ul>
          </div>
        </section>
@@ -118,8 +121,6 @@
    </div>
  <?php } ?>

-</main>
-
 <?php
    require_once __DIR__ . "/../includes/footer.php";
 ?>