diff --git a/application/libraries/monpas/ijkpunten/Userlijsten.php b/application/libraries/monpas/ijkpunten/Userlijsten.php index 336628c..4cdac5c 100644 --- a/application/libraries/monpas/ijkpunten/Userlijsten.php +++ b/application/libraries/monpas/ijkpunten/Userlijsten.php @@ -111,7 +111,6 @@ class Userlijsten } } private function setAmbities(){ - if(!isset($this->scores) || !$this->scores) return; $standaard = new stdClass(); $standaard->ambitievan = 'standaard'; $standaard->id = 0; @@ -125,6 +124,12 @@ class Userlijsten $standaard->xijkpunten = ROUND($this->results['aantalY'] * 0.8); $standaard->zprocent = 60; + // lijst niet ingevuld: standaard ambitie, zodat de periode toch een (lege) kolom krijgt + if(!isset($this->scores) || !$this->scores) { + $this->lijst->ambities = $standaard; + return; + } + if($this->results && $this->results['ambities'] && $this->results['ambities']->gemscore > 0) $this->lijst->ambities = $this->results['ambities']; else $this->lijst->ambities = $standaard; @@ -141,7 +146,7 @@ class Userlijsten private function setLijst(){ $this->lijst->naam = $this->results['orgnaam']; $this->lijst->scores = $this->scores; - if(isset($this->scores)) { + if($this->scores) { // false = niet ingevuld; isset() is ook waar voor false $this->lijst->metingen = 1 ; $this->lijst->aantalorgs = 1 ; $this->lijst->adviezen = $this->results['adviezen']; diff --git a/application/views/monpas/pages/ijkpunten.php b/application/views/monpas/pages/ijkpunten.php index 40e4e5f..26cc802 100644 --- a/application/views/monpas/pages/ijkpunten.php +++ b/application/views/monpas/pages/ijkpunten.php @@ -256,7 +256,7 @@ - +
@@ -367,7 +367,7 @@
- +
@@ -403,7 +403,7 @@
- +
@@ -438,6 +438,43 @@
+ + + + + + + + + + + + + + + + + + + + + + + + + + + +
+
+
+
+
+
+
+
niet ingevuld +
+
 
 
diff --git a/public/js/monpas/models/ijkpuntenmodel.js b/public/js/monpas/models/ijkpuntenmodel.js index a22089e..d8369ea 100644 --- a/public/js/monpas/models/ijkpuntenmodel.js +++ b/public/js/monpas/models/ijkpuntenmodel.js @@ -556,10 +556,10 @@ ls.initIJkpuntenModel = function(){ $.each(result.lijsten, function (i, p) { if (p.scores) { self.erzijnlijsten(true); - self.nietIngevuld(false); } - else{self.nietIngevuld(true)} }) + // melding alleen als geen enkele gekozen periode is ingevuld + self.nietIngevuld(!self.erzijnlijsten()); } else{ self.nietIngevuld(true) @@ -608,7 +608,12 @@ ls.initIJkpuntenModel = function(){ }) + var eerste = true; // eerste lijst met scores bepaalt x-as en ambitielijn $.each(lijsten, function(i, l) { + if (!l.erzijnscores()) { // periode zonder ingevulde lijst niet in de grafiek + lijsten[i].scores = l.scores(); + return; + } var labels = []; var scores = []; var ambities = []; @@ -617,7 +622,7 @@ ls.initIJkpuntenModel = function(){ if(l.label){ xastitel = l.label; } - if(i === 0 || self.eigenambitie()) { + if(eerste || self.eigenambitie()) { if (l.gemscore && Number(l.gemscore) > 0) { ambitiewaarde = Number(l.gemscore); @@ -639,7 +644,7 @@ ls.initIJkpuntenModel = function(){ name : l.periodenummer }); - if(i === 0 || self.eigenambitie()) { + if(eerste || self.eigenambitie()) { ambities.push(ambitiewaarde) } @@ -667,7 +672,7 @@ ls.initIJkpuntenModel = function(){ data : scores }); - if(i === 0 || self.eigenambitie()) { + if(eerste || self.eigenambitie()) { ambities.unshift(ambitiewaarde); ambities.push(ambitiewaarde); @@ -684,9 +689,10 @@ ls.initIJkpuntenModel = function(){ balkGegevens.push(lijnGegevens); } - if(i === 0) { + if(eerste) { xas = labels; } + eerste = false; }); self.ijkgrafiek = Highcharts.chart('hc-chart1', {