From fbde1a0406b63d73d4cfb5e51fa4bbe56a33da59 Mon Sep 17 00:00:00 2001
From: JelmerHinssen <jelmerhinssen@hotmail.com>
Date: Mon, 3 Apr 2017 18:49:46 +0200
Subject: [PATCH] Echte grafiekfix en andere dingen

---
 src/Application/Helper/Overboekingen.php | 38 +++++++++++-------------
 src/Application/Transacties.php          | 14 ++-------
 templates/statistieken.phtml             |  2 +-
 3 files changed, 22 insertions(+), 32 deletions(-)

diff --git a/src/Application/Helper/Overboekingen.php b/src/Application/Helper/Overboekingen.php
index 672623b..d76f557 100644
--- a/src/Application/Helper/Overboekingen.php
+++ b/src/Application/Helper/Overboekingen.php
@@ -59,35 +59,33 @@ class Overboekingen
         return $message;
     }
 
-    private static function isValidFloat($bedrag)
+    public static function isValidFloat($bedrag)
     {
-        $hasdecimal = false;
-        $decimals = 0;
-        $voorcijfers = 0;
+        $komma = 0;
+        $decimalen = 0;
+        $cijfers = 0;
         for ($i = 0; $i < strlen($bedrag); $i++) {
             $char = substr($bedrag, $i, 1);
             $ascii = ord($char);
-            if ($hasdecimal) {
-                if ($char === "." || $char === ",") {
+            if ($char === "." || $char === ",") {
+                $komma++;
+                if($komma > 1){
                     //Twee komma's
                     return false;
-                } elseif ($ascii >= ord("0") && $ascii <= ord("9")) {
-                    $decimals++;
-                    if ($decimals > 2) {
+                }
+            } elseif (($ascii >= ord("0") && $ascii <= ord("9") || $char === "-")) {
+                if($komma >= 1){
+                    $decimalen++;
+                    if ($decimalen > 2) {
                         //Meer dan twee cijfers achter de komma
                         return false;
                     }
-                } else {
-                    //Ongeldig teken
-                    return false;
-                }
-            } elseif ($char === "." || $char === ",") {
-                $hasdecimal = true;
-            } elseif ($ascii >= ord("0") || $ascii <= ord("9")) {
-                $voorcijfers++;
-                if ($voorcijfers > 9) {
-                    //Te veel cijfers voor de komma
-                    return false;
+                }else{
+                    $cijfers++;
+                    if($cijfers > 9){
+                        //Te veel cijfers voor de komma
+                        return false;
+                    }
                 }
             } else {
                 //Ongeldig teken
diff --git a/src/Application/Transacties.php b/src/Application/Transacties.php
index 9e72153..6cd7296 100644
--- a/src/Application/Transacties.php
+++ b/src/Application/Transacties.php
@@ -28,20 +28,12 @@ class Transacties
             $rekeningen = Rekeningen::getCurrentUserRekeningen();
 
             $ibanhelper = new IBAN;
-            $rekening = false;
+            $rekening = $rekening = Rekeningen::isEigenRekeningnr($args['rekeningnummer']);
             $rekeningnr = $args['rekeningnummer'];
-
-            //Controleer of deze rekening wel van de ingelogde klant is
-            foreach ($rekeningen as $rek) {
-                if ($rek['nr'] === $rekeningnr) {
-                    $rekening = $rek;
-                    break;
-                }
-            }
             if (!$rekening) {
                 return $this->renderer->render($response, 'transacties.phtml', [
-                            'klant' => $klant,
-                            'rekening' => false
+                    'header' => Header::getHeaderData(),
+                    'rekening' => false
                 ]);
             }
             $container = Application::getContainer();
diff --git a/templates/statistieken.phtml b/templates/statistieken.phtml
index 4dd6e0d..38a26f6 100644
--- a/templates/statistieken.phtml
+++ b/templates/statistieken.phtml
@@ -35,7 +35,7 @@ if (!$statistieken) {
                     }],
                 yAxes: [{
                         ticks: {
-                            beginAtZero:true
+                            min: 0
                         }
                 }]
             }