Merge branch 'master' of https://codefirst.iut.uca.fr/git/bastien.ollier/flux_rss

3 years ago · 8bdc5f1e5c
parent 73894839a6 7bf0c8e0a0
commit 8bdc5f1e5c
3 changed files with 16 additions and 10 deletions
--- a/controllers/ControllerAdmin.php
+++ b/controllers/ControllerAdmin.php
@ -50,9 +50,11 @@ class controllerAdmin {
    }

    function ajoutSource() {
-        global $con,$vues;
-        if(isset($_REQUEST["linkSource"])) {
-            $gate = new GatewaySources($con);
+        global $dsn,$user,$pass,$vues;
+        $error =[];
+        $link = $_REQUEST["linkSource"];
+        if(isset($_REQUEST["linkSource"]) && Validation::url_form($link,$error)) {
+            $gate = new GatewaySources(new Connection($dsn,$user,$pass));
            $source = new Sources($_REQUEST["linkSource"]);
            $gate->addSources($source);
        }
--- a/vues/Validation.php
+++ b/vues/Validation.php
@ -38,19 +38,21 @@

        }

-        static function url_form(string &$url,array &$dVueEreur) 
+        static function url_form(string $url,array &$dVueEreur)
        {
-
                if (!isset($url)||$url=="") {
                    $dVueEreur[] = "pas d'url";
                    $url="";
+                    return false;
                }

-                if ($url != filter_var($name, FILTER_VALIDATE_URL))
+                if (!filter_var($url, FILTER_VALIDATE_URL))
                {
                    $dVueEreur[] = "url invalide";
                    $url="";
+                    return false;
                }
+                return true;

        }

--- a/vues/administration.php
+++ b/vues/administration.php
@ -1,7 +1,9 @@
-<form action="index.php?action=ajoutSource" method="POST">
+<html>
+    <form action="../index.php?action=ajoutSource" method="POST">
        <input type="text" name="linkSource" placeholder="source" />
        <input type="submit" value="add source">
    </form>
+</html>

 <?php
        foreach ($listeSources as $n){