From 265d1374774a414bfe668b15a4508d5315309ae3 Mon Sep 17 00:00:00 2001
From: =?utf8?q?J=C3=A9r=C3=B4me=20Benoit?= <jerome.benoit@piment-noir.org>
Date: Tue, 9 Jan 2018 11:00:14 +0100
Subject: [PATCH] Fix the dynamic content routing functions.
MIME-Version: 1.0
Content-Type: text/plain; charset=utf8
Content-Transfer-Encoding: 8bit

Signed-off-by: Jérôme Benoit <jerome.benoit@piment-noir.org>
---
 index.php | 24 +++++++++++++++---------
 login.php |  5 +++++
 2 files changed, 20 insertions(+), 9 deletions(-)

diff --git a/index.php b/index.php
index 01b3ce3..2829b26 100644
--- a/index.php
+++ b/index.php
@@ -14,6 +14,9 @@ $configs = include('config.php');
  */
 session_start();
 
+//$page = filter_input(INPUT_GET, $_GET['page'], FILTER_SANITIZE_URL);
+$page = $_GET['page'];
+
 if (!isset($page)) {
     $page = "";
 }
@@ -29,31 +32,34 @@ function valid_page($page_name)
     global $configs;
 
     foreach ($configs['actions'] as $action) {
-        if ($page_name === $action) {
-            return $action;
+        if (strcmp($page_name, $action) === 0) {
             // The actions list can't have duplicated entries
-            break;
-        } else {
-            return $configs['actions'][0];
+            return $action;
         }
     }
+    // Return the default action
+    return $configs['actions'][0];
 }
 
 function display_action($action_name)
 {
     global $configs;
 
+    $found_action = false;
     foreach ($configs['actions'] as $action) {
-        if ($action_name === $action) {
+        if (strcmp($action_name, $action) === 0) {
             include("$action.php");
+            $found_action = true;
             break;
-        } else {
-            echo "Action to display do not exist";
         }
     }
+    if (!$found_action) {
+        echo "Action to display do not exist";
+    }
 }
 
-display_action(valid_page($page));
+$action = valid_page($page);
+display_action($action);
 
 session_destroy();
 
diff --git a/login.php b/login.php
index e69de29..89b3b2d 100644
--- a/login.php
+++ b/login.php
@@ -0,0 +1,5 @@
+<form action="action.php" method="post">
+ <p>Email : <input type="text" name="email" /></p>
+ <p>Mot de passe : <input type="text" name="password" /></p>
+ <p><input type="submit" value="OK"></p>
+</form>
-- 
2.34.1