From b8177ade483460255fc610a0855685e58103dbd3 Mon Sep 17 00:00:00 2001 From: Monique Date: Fri, 19 Nov 2021 10:31:31 +0100 Subject: [PATCH] tussenstand --- application/controllers/monpas/Sop.php | 2 +- application/models/monpas/Ambities_model.php | 9 ++- application/models/monpas/Ijkpunten_model.php | 3 +- .../models/monpas/Sopinhouden_model.php | 66 +++++++++++-------- .../views/monpas/sopkeuze/agendagemiddeld.php | 64 ++++++++++++------ .../views/monpas/sopkeuze/ijkpuntentabel.php | 11 ++-- .../monpas/sopkeuze/ijktabelgemiddeld.php | 2 +- public/js/monpas/models/ontwikkelmodel.js | 2 - 8 files changed, 96 insertions(+), 63 deletions(-) diff --git a/application/controllers/monpas/Sop.php b/application/controllers/monpas/Sop.php index 786a9da..9ddc4e7 100644 --- a/application/controllers/monpas/Sop.php +++ b/application/controllers/monpas/Sop.php @@ -117,7 +117,7 @@ class Sop extends MY_Controller{ $nieuweInhoud = $this->Sopinhouden_model->get(['id' => $newID], false, true); // om de vergelijkende versies bij te houden dient // per versie een record in de tabel tb_flexsop_gebruikteversies - // te worden gemaakt. + // te worden gemaakt. // de originele versie staat in de tabel tb_sopinhouden $index = 1; foreach ($vergelijkingen ? $vergelijkingen : [] as $vergelijking){ diff --git a/application/models/monpas/Ambities_model.php b/application/models/monpas/Ambities_model.php index 2949bd3..4b770dc 100644 --- a/application/models/monpas/Ambities_model.php +++ b/application/models/monpas/Ambities_model.php @@ -34,10 +34,15 @@ class Ambities_model extends MY_Model{ return $this->get_query_row(); } function getambitie($orgversieid, $orgid){ - $this->sql="SELECT A.id, gem_score AS gemscore, x_ijkpunten AS xijkpunten , z_procent AS zprocent + $this->sql="SELECT A.id, gem_score AS gemscore, x_ijkpunten AS xijkpuntenoud, COUNT(Y.id) AS xijkpunten, z_procent AS zprocent FROM tb_ambities A + JOIN tb_orgversies OV ON A.orgversie_id = OV.id + JOIN tb_versies V ON OV.versie_id = V.id + JOIN tb_ijkpunten Y ON Y.versie_id = V.id WHERE A.organisatie_id = {$orgid} - AND A.orgversie_id = {$orgversieid}"; + AND A.orgversie_id = {$orgversieid} + AND Y.disabled = 'F' + GROUP BY A.id "; return $this->get_query_row(); } function getdefault($aantalY){ diff --git a/application/models/monpas/Ijkpunten_model.php b/application/models/monpas/Ijkpunten_model.php index 5894fad..9c1b190 100644 --- a/application/models/monpas/Ijkpunten_model.php +++ b/application/models/monpas/Ijkpunten_model.php @@ -37,7 +37,8 @@ class Ijkpunten_model extends MY_Model{ $this->sql="SELECT COUNT(Y.id) AS aantalY FROM tb_orgversies OV JOIN tb_ijkpunten Y ON Y.versie_id = OV.versie_id - WHERE OV.id ={$orgversieid} "; + WHERE OV.id ={$orgversieid} + AND Y.disabled = 'F' "; return $this->get_query_row('aantalY'); } diff --git a/application/models/monpas/Sopinhouden_model.php b/application/models/monpas/Sopinhouden_model.php index a78f416..5987a73 100644 --- a/application/models/monpas/Sopinhouden_model.php +++ b/application/models/monpas/Sopinhouden_model.php @@ -552,7 +552,7 @@ class Sopinhouden_model extends MY_Model{ $targetversieid = $this->gettargetversie($profielid, $orgid,$chapterid); } if ($targetversieid) { - $scores = $this->Scores_model->calculateschool($targetversieid, $orgid); + $scores = $this->Scores_model->getgemscores($targetversieid, $orgid); $ambitie = $this->Ambities_model->getOrgAmbities($targetversieid, $orgid); } @@ -569,34 +569,42 @@ class Sopinhouden_model extends MY_Model{ $hoogstescore = -1; $welkelaagste = 0; $laagstescore = 101; - if ($scores){ - foreach($scores as $score){ - $gem = 0; + if ($scores) { + foreach ($scores as $score) { +// $gem = 0; +// +// if ($score->num > 0) { +// $gem=$score->somscore / $score->num; +// }; +// $score->kleurcode = $this->kleurcodes->getkleur($score->gem); - if ($score->num > 0) { - $gem=$score->somscore / $score->num; - }; - $totgem= $totgem + $gem; - $totaantal=$totaantal + 1; + if ($score->disabled == 'F' && $score->disablenvt == 'F') { + $totgem = $totgem + $score->gem; + $totaantal = $totaantal + 1; - $score->kleurcode=$this->kleurcodes->getkleur($gem); + if ($score->gem > $hoogstescore) { + $hoogstescore = $score->gem; + $welkehoogste = $score->nummer; + } + if ($score->gem <= $laagstescore) { + $laagstescore = $score->gem; + $welkelaagste = $score->nummer; + } + if ($ambitie && $score->gem >= $ambitie->zprocent) { + $ijkpuntenok = $ijkpuntenok + 1; + }; - if ($gem > $hoogstescore){ - $hoogstescore = $gem; - $welkehoogste = $score->nummer; - } - if ($gem <= $laagstescore){ - $laagstescore = $gem; - $welkelaagste = $score->nummer; - } - if ($ambitie && $gem >= $ambitie->zprocent) {$ijkpuntenok = $ijkpuntenok + 1;}; - } - if ($totaantal >0){ $totgem= round($totgem/ $totaantal, 2);} - } - $totgemkleur= $this->kleurcodes->getkleur($totgem); - // foreach scores bereken som en dus totgem en ijkpuntenok - $tabelid=''; + if ($totaantal > 0) { + $totgem = round($totgem / $totaantal, 2); + } + } + } + $totgemkleur = $this->kleurcodes->getkleur($totgem); + // foreach scores bereken som en dus totgem en ijkpuntenok + } + + $tabelid=''; switch ($module){ case 1: $tabelid = "tabelmb1"; break; case 3: $tabelid = "tabeltv1"; break; @@ -719,7 +727,7 @@ class Sopinhouden_model extends MY_Model{ } if ($targetversieid){ $targetversie = $this->Orgversies_model->get_record_by_id($targetversieid); - $scores = $this->Scores_model->calculateschool($targetversieid, $orgid); + $scores = $this->Scores_model->getgemscores($targetversieid, $orgid); $ambitie = $this->Ambities_model->getOrgAmbities($targetversieid, $orgid); if($targetversie->scoretype == 7){ @@ -742,11 +750,11 @@ class Sopinhouden_model extends MY_Model{ $scores = $scores ? $scores : []; foreach($scores as $score){ - $score->gem = 0; +// $score->gem = 0; if ($score->num > 0) { - $score->gem= round($score->somscore / $score->num,1); + $score->gem= round($score->gem); } - $score->huidig = $this->kleurcodes->getnummer($score->gem); + $score->huidig = $score->nummer; if($targetversie->scoretype == 7){ if($score->gem == $grens3) $score->huidig = 4; if($score->gem < $grens3) $score->huidig = 3; diff --git a/application/views/monpas/sopkeuze/agendagemiddeld.php b/application/views/monpas/sopkeuze/agendagemiddeld.php index 7d81eca..2d33fc7 100644 --- a/application/views/monpas/sopkeuze/agendagemiddeld.php +++ b/application/views/monpas/sopkeuze/agendagemiddeld.php @@ -16,13 +16,13 @@ - + - huidige situatie + huidige situatie - gewenste situatie + gewenste situatie nr @@ -31,11 +31,14 @@ 2 3 4 + + 1 2 3 - 4 + 4 + scores){ @@ -43,30 +46,43 @@ if($ijkpunt->nummer != 'gem'){?> nummer;?> - omschrijving)){echo $ijkpunt->omschrijving;}?> - - huidig == 1){?> -
- + omschrijving)){echo $ijkpunt->omschrijving;}?> + omschrijving)){echo $ijkpunt->omschrijving;}?> + disabled== 'T'){echo '(uit)';} ?> + disablenvt == 'T' &&$ijkpunt->disabled== 'F' ){echo '(nvt)';} ?> + + disabled == 'F' && $ijkpunt->disablenvt == 'F'){?> + + huidig == 1 ){?> +
+ + + + huidig == 2){?> +
+ + + + huidig == 3){?> +
+ + + + huidig == 4){?> +
+ + + - huidig == 2){?> -
- - - - huidig == 3){?> -
- - - - huidig == 4){?> -
+ disabled == 'T' || $ijkpunt->disablenvt == 'T' ){?> +
+ disabled == 'F' && $ijkpunt->disablenvt == 'F'){?> wens) && $ijkpunt->wens == 1){ ?>
@@ -88,6 +104,12 @@
+ + + disabled == 'T' || $ijkpunt->disablenvt == 'T' ){ ?> +
+ + diff --git a/application/views/monpas/sopkeuze/ijkpuntentabel.php b/application/views/monpas/sopkeuze/ijkpuntentabel.php index cdad671..5c83790 100644 --- a/application/views/monpas/sopkeuze/ijkpuntentabel.php +++ b/application/views/monpas/sopkeuze/ijkpuntentabel.php @@ -41,12 +41,11 @@ - scores as $score){?> + scores as $score){ + $kleur = $score->disabled == 'T' || $score->disablenvt == 'T' ? '#808080' : $score->kleur; ?> - - - - gem ?> + + disabled == 'T' ? 'uit' : ($score->disablenvt == 'T' ? 'nvt' : (isset($score->gem) ? $score->gem : '')) ?> @@ -54,7 +53,7 @@ -

Scores in procenten.

+

Scores in procenten.

diff --git a/application/views/monpas/sopkeuze/ijktabelgemiddeld.php b/application/views/monpas/sopkeuze/ijktabelgemiddeld.php index 89dfa34..eb65e56 100644 --- a/application/views/monpas/sopkeuze/ijktabelgemiddeld.php +++ b/application/views/monpas/sopkeuze/ijktabelgemiddeld.php @@ -46,7 +46,7 @@ - + gem ?> diff --git a/public/js/monpas/models/ontwikkelmodel.js b/public/js/monpas/models/ontwikkelmodel.js index 77d19e3..2ecf3f5 100644 --- a/public/js/monpas/models/ontwikkelmodel.js +++ b/public/js/monpas/models/ontwikkelmodel.js @@ -88,7 +88,6 @@ ls.initOntwikkelModel = function(){ ,this) } function fAddResult(p){ - console.log(p, "p result") var dit = this; this.ijkpuntid = p.ijkpunt_id; this.wensid = 0; @@ -112,7 +111,6 @@ ls.initOntwikkelModel = function(){ this.toelichting = p.toelichting; this.aantalscholen = p.aantalscholen; - console.log(this.disablentv, 'disablenvt') }