Overigen/monpas
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

kleuren werden niet goed opgehaald bij resultaat matrix

Browse Source
This commit is contained in:
Monique
2022-06-10 10:35:52 +02:00
parent e4c852671b
commit e8a6bf85b7
13 changed files with 153 additions and 139 deletions
Download Patch File Download Diff File Expand all files Collapse all files
application/controllers/monpas/Rapijkpunten.php
+1
View File
@@ -16,6 +16,7 @@ class Rapijkpunten extends MY_Controller{
"monpas/Indicatoren_model", "monpas/Indicatoren_model",
"monpas/Antwoordtypes_model", "monpas/Antwoordtypes_model",
"monpas/Wenssituaties_model", "monpas/Wenssituaties_model",
"monpas/Scorekleuren_model",
"monpas/Scores_model", "monpas/Scores_model",
'shared/Session_model', 'shared/Session_model',
"monpas/Ambities_model", "monpas/Ambities_model",
application/controllers/monpas/Rapmatrix.php
+2 -1
View File
@@ -663,7 +663,8 @@ private function getGemiddeldeScores($labels, $ijkpunten, $kleuren, $scoretype){
} }
} }
$user->scores[$i]->kleur = $this->Scorekleuren_model->getGemiddeldeKleur($scoretype, $user->scores[$i]->gem); $kleur = $this->Scorekleuren_model->getGemiddeldeKleur($scoretype, $user->scores[$i]->gem);
$user->scores[$i]->kleur = $kleur->kleurcode;
if(!isset($user->scores[$i]->disablenvt)){ if(!isset($user->scores[$i]->disablenvt)){
$user->scores[$i]->disablenvt= 'F'; $user->scores[$i]->disablenvt= 'F';
if($user->scores[$i]->aantalvier > 0 && if($user->scores[$i]->aantalvier > 0 &&
application/core/MY_Controller.php
+2 -1
View File
@@ -73,7 +73,8 @@ class MY_Controller extends CI_Controller{
// public $version = '1.17.97';// omvang geen grafiek bij 0 resultaten // public $version = '1.17.97';// omvang geen grafiek bij 0 resultaten
// public $version = '1.17.98';// instrumenten voor ouders // public $version = '1.17.98';// instrumenten voor ouders
// public $version = '1.17.99';// instrumenten ook voor leerlingen // public $version = '1.17.99';// instrumenten ook voor leerlingen
public $version = '1.18.00';// alle instrumenten in het compact sop // public $version = '1.18.00';// alle instrumenten in het compact sop
public $version = '1.18.01';// fout in resultaatmatrix
application/libraries/monpas/Gemiddeldenafbeeldingen.php
+57 -56
View File
@@ -198,27 +198,12 @@ class Gemiddeldenafbeeldingen {
} }
private function getKleur ($score){ private function getKleur ($score){
$grensnummer = 1; $kleur = $this->CI->Scorekleuren_model->getGemiddeldeKleur($score->scoretype, $score->gem);
if ($score>= $this->kleuren->grens1) $grensnummer = 2; $kleur = $kleur->kleurcode;
if ($score>= $this->kleuren->grens2) $grensnummer = 3; if(!$kleur) $kleur = '#ffffff';
if ($score>= $this->kleuren->grens3) $grensnummer = 4; $this->score->kleur = $kleur;
switch ( $grensnummer) {
case 1:
$kleur = $this->kleuren->kleur1;
break;
case 2:
$kleur = $this->kleuren->kleur2;
break;
case 3:
$kleur = $this->kleuren->kleur3;
break;
case 4:
$kleur = $this->kleuren->kleur4;
break;
default:
$kleur = $this->kleuren->kleur1;
}
return $kleur; return $kleur;
} }
@@ -243,14 +228,15 @@ class Gemiddeldenafbeeldingen {
} }
private function loopIJkpunten2(){ private function loopIJkpunten2(){
if(!$this->ijkpunten) return; if(!$this->ijkpunten) return;
$scoretype = $this->ijkpunten[0]->scoretype;
foreach($this->ijkpunten as $index => $this->ijkpunt){ foreach($this->ijkpunten as $index => $this->ijkpunt){
if(!isset($this->ijkpunt->score)){continue;}; if(!isset($this->ijkpunt->score)){continue;};
if(! $this->ijkpunt->score) $this->ijkpuntscore = 0; if(! $this->ijkpunt->score) $this->ijkpuntscore = 0;
$this->setScorenummer(); $this->setScorenummer();
$this->rondScoresAf(); $this->rondScoresAf();
$this->setScoreKleuren(); $this->setScoreKleuren($scoretype);
$this->addGewensteScores($index); $this->addGewensteScores($index);
$this->setWenskleuren(); $this->setWenskleuren($scoretype);
$this->setAdvies(); $this->setAdvies();
$this->setResultatenOverzicht(); $this->setResultatenOverzicht();
$this->telIJkpuntenok(); $this->telIJkpuntenok();
@@ -276,23 +262,31 @@ class Gemiddeldenafbeeldingen {
$this->ijkpunt->score = round($this->ijkpunt->score); $this->ijkpunt->score = round($this->ijkpunt->score);
} }
private function setScoreKleuren(){ private function setScoreKleuren($scoretype){
switch ($this->ijkpunt->huidig) { $kleur = $this->CI->Scorekleuren_model->getGemiddeldeKleur($scoretype, $this->ijkpunt->score);
case 1: $kleur = $kleur->kleurcode;
$this->ijkpunt->kleur = $this->kleuren->kleur1; if(!$kleur) $kleur = '#ffffff';
break; $this->i->kljkpunt = $kleur;
case 2:
$this->ijkpunt->kleur = $this->kleuren->kleur2;
break;
case 3:
$this->ijkpunt->kleur = $this->kleuren->kleur3; // switch ($this->ijkpunt->huidig) {
break; // case 1:
case 4: // $this->ijkpunt->kleur = $this->kleuren->kleur1;
$this->ijkpunt->kleur = $this->kleuren->kleur4; // break;
break; // case 2:
default: // $this->ijkpunt->kleur = $this->kleuren->kleur2;
$this->ijkpunt->kleur = $this->kleuren->kleur1; // break;
} // case 3:
// $this->ijkpunt->kleur = $this->kleuren->kleur3;
// break;
// case 4:
// $this->ijkpunt->kleur = $this->kleuren->kleur4;
// break;
// default:
// $this->ijkpunt->kleur = $this->kleuren->kleur1;
// }
} }
@@ -305,24 +299,31 @@ class Gemiddeldenafbeeldingen {
} }
} }
private function setWensKleuren(){ private function setWensKleuren($scoretype){
if(!isset($this->ijkpunt->wens)) return; if(!isset($this->ijkpunt->wens)) return;
switch ($this->ijkpunt->wens) {
case 1: $kleur = $this->CI->Scorekleuren_model->getWensKleur($scoretype, $this->ijkpunt->wens);
$this->ijkpunt->wenskleur = $this->kleuren->kleur1; if(!$kleur) $kleur = '#ffffff';
break; $this->ijkpunt->wenskleur = $kleur;
case 2:
$this->ijkpunt->wenskleur = $this->kleuren->kleur2;
break;
case 3: // switch ($this->ijkpunt->wens) {
$this->ijkpunt->wenskleur = $this->kleuren->kleur3; // case 1:
break; // $this->ijkpunt->wenskleur = $this->kleuren->kleur1;
case 4: // break;
$this->ijkpunt->wenskleur = $this->kleuren->kleur4; // case 2:
break; // $this->ijkpunt->wenskleur = $this->kleuren->kleur2;
default: // break;
$this->ijkpunt->wenskleur = '#ffffff'; // case 3:
} // $this->ijkpunt->wenskleur = $this->kleuren->kleur3;
// break;
// case 4:
// $this->ijkpunt->wenskleur = $this->kleuren->kleur4;
// break;
// default:
// $this->ijkpunt->wenskleur = '#ffffff';
// }
} }
private function setAdvies(){ private function setAdvies(){
application/libraries/monpas/Schoolafbeeldingen.php
+33 -39
View File
@@ -224,12 +224,13 @@ class Schoolafbeeldingen {
private function loopScores2() private function loopScores2()
{ {
if (!$this->scores) return; if (!$this->scores) return;
$scoretype = $this->scores[0]->scoretype;
foreach ($this->scores as $index => $this->score) { foreach ($this->scores as $index => $this->score) {
$this->setScoreNummer(); $this->setScoreNummer();
$this->rondScoresAf(); $this->rondScoresAf();
$this->setScoreKleuren(); $this->setScoreKleuren($scoretype);
$this->addGewensteScores($index); $this->addGewensteScores($index);
$this->setWensKleuren(); $this->setWensKleuren($scoretype);
$this->setAdvies(); $this->setAdvies();
$this->setResultatenOverzicht(); $this->setResultatenOverzicht();
$this->telIJKpuntenok(); $this->telIJKpuntenok();
@@ -257,23 +258,31 @@ class Schoolafbeeldingen {
} }
private function setScoreKleuren(){ private function setScoreKleuren($scoretype){
switch ($this->score->huidig) {
case 1:
$this->score->kleur = $this->kleuren->kleur1; $kleur = $this->CI->Scorekleuren_model->getGemiddeldeKleur($scoretype, $this->score->gem);
break; $kleur = $kleur->kleurcode;
case 2: if(!$kleur) $kleur = '#ffffff';
$this->score->kleur = $this->kleuren->kleur2; $this->score->kleur = $kleur;
break;
case 3:
$this->score->kleur = $this->kleuren->kleur3; // switch ($this->score->huidig) {
break; // case 1:
case 4: // $this->score->kleur = $this->kleuren->kleur1;
$this->score->kleur = $this->kleuren->kleur4; // break;
break; // case 2:
default: // $this->score->kleur = $this->kleuren->kleur2;
$this->score->kleur = '#ffffff'; // break;
} // case 3:
// $this->score->kleur = $this->kleuren->kleur3;
// break;
// case 4:
// $this->score->kleur = $this->kleuren->kleur4;
// break;
// default:
// $this->score->kleur = '#ffffff';
// }
} }
@@ -284,28 +293,13 @@ class Schoolafbeeldingen {
} }
} }
private function setWensKleuren(){ private function setWensKleuren($scoretype){
if(! isset($this->wensen)) return; if(! isset($this->wensen)) return;
if(!isset($this->score->wens)) return; if(!isset($this->score->wens)) return;
if($this->score->disabled == 'T') $kleur = $this->CI->Scorekleuren_model->getWensKleur($scoretype, $this->score->wens);
{$this->score->wenskleur = '#ffffff'; if(!$kleur) $kleur = '#ffffff';
return;} $this->score->wenskleur = $kleur;
switch ($this->score->wens) {
case 1:
$this->score->wenskleur = $this->kleuren->kleur1;
break;
case 2:
$this->score->wenskleur = $this->kleuren->kleur2;
break;
case 3:
$this->score->wenskleur = $this->kleuren->kleur3;
break;
case 4:
$this->score->wenskleur = $this->kleuren->kleur4;
break;
default:
$this->score->wenskleur = '#ffffff';
}
} }
private function setAdvies(){ private function setAdvies(){
application/libraries/monpas/Schoolscores.php
+1 -1
View File
@@ -5,6 +5,7 @@ class Schoolscores
private $CI; private $CI;
private $ijkpunten; private $ijkpunten;
private $scores; private $scores;
private $score;
private $ambities; private $ambities;
private $wensen; private $wensen;
private $schoolScores; private $schoolScores;
@@ -57,7 +58,6 @@ class Schoolscores
$this->scores = $this->CI->Scores_model->getgemscores($this->orgversieid, $this->orgid); $this->scores = $this->CI->Scores_model->getgemscores($this->orgversieid, $this->orgid);
} }
private function setAmbities(){ private function setAmbities(){
$this->ambities = $this->CI->Ambities_model->getOrgAmbities($this->orgversieid, $this->orgid); $this->ambities = $this->CI->Ambities_model->getOrgAmbities($this->orgversieid, $this->orgid);
} }
application/libraries/monpas/ijkpunten/Afbeeldingen.php
+26 -19
View File
@@ -67,7 +67,7 @@ class Afbeeldingen {
$this->setScores(); $this->setScores();
$this->setSWVid(); $this->setSWVid();
$this->setOrgversie(); $this->setOrgversie();
$this->setKleurcodes(); // $this->setKleurcodes();
$this->setKleuren(); $this->setKleuren();
$this->loopScores1(); $this->loopScores1();
$this->setGemiddeldeScore(); $this->setGemiddeldeScore();
@@ -185,10 +185,11 @@ class Afbeeldingen {
private function loopScores2(){ private function loopScores2(){
if (!$this->scores) return; if (!$this->scores) return;
$scoretype = $this->scores[0]->scoretype;
foreach ($this->scores as $index => $this->score) { foreach ($this->scores as $index => $this->score) {
$this->setScoreNummer(); $this->setScoreNummer();
$this->rondScoresAf(); $this->rondScoresAf();
$this->setScoreKleuren(); $this->setScoreKleuren($scoretype);
$this->setResultatenOverzicht(); $this->setResultatenOverzicht();
$this->telIJkpuntenok(); $this->telIJkpuntenok();
} }
@@ -206,23 +207,29 @@ class Afbeeldingen {
} }
private function setScoreKleuren(){ private function setScoreKleuren($scoretype){
switch ($this->score->huidig) { $kleur = $this->CI->Scorekleuren_model->getGemiddeldeKleur($scoretype, $this->score->gem);
case 1: $kleur = $kleur->kleurcode;
$this->score->kleur = $this->kleuren->kleur1; if(!$kleur) $kleur = '#ffffff';
break; $this->score->kleur = $kleur;
case 2:
$this->score->kleur = $this->kleuren->kleur2;
break; // switch ($this->score->huidig) {
case 3: // case 1:
$this->score->kleur = $this->kleuren->kleur3; // $this->score->kleur = $this->kleuren->kleur1;
break; // break;
case 4: // case 2:
$this->score->kleur = $this->kleuren->kleur4; // $this->score->kleur = $this->kleuren->kleur2;
break; // break;
default: // case 3:
$this->score->kleur = '#ffffff'; // $this->score->kleur = $this->kleuren->kleur3;
} // break;
// case 4:
// $this->score->kleur = $this->kleuren->kleur4;
// break;
// default:
// $this->score->kleur = '#ffffff';
// }
} }
private function setResultatenOverzicht(){ private function setResultatenOverzicht(){
application/libraries/monpas/ijkpunten/Lijsten.php
+5 -8
View File
@@ -197,10 +197,6 @@ class Lijsten
} }
} }
private function setSchoolScores(){ private function setSchoolScores(){
if ($this->soort != 4 || $this->lijst->vorm != 1) return; if ($this->soort != 4 || $this->lijst->vorm != 1) return;
$this->schoolScores = new Schoolscores($this->lijst->orgversieid, $this->orgid); $this->schoolScores = new Schoolscores($this->lijst->orgversieid, $this->orgid);
@@ -365,14 +361,15 @@ class Lijsten
} }
private function loopVergelijkingScores(){ private function loopVergelijkingScores(){
if( (!isset($this->lijst->vergelijkingen) || !$this->lijst->vergelijkingen)) return; if(!isset($this->lijst->vergelijkingen) || !$this->lijst->vergelijkingen) return;
for($i = 0; $i < count($this->lijst->vergelijkingen); $i++){
$this->setVergelijkingScores($i); foreach($this->lijst->vergelijkingen AS $index => $vergelijking){
$this->setVergelijkingScores($index);
} }
} }
private function setVergelijkingScores($i){ private function setVergelijkingScores($i){
if($this->lijst->scores[$i]) { if($this->lijst->scores && $this->lijst->scores[$i]) {
$this->lijst->scores[$i]->gemV = $this->lijst->vergelijkingen[$i]->gem; $this->lijst->scores[$i]->gemV = $this->lijst->vergelijkingen[$i]->gem;
$this->lijst->scores[$i]->kleurV = $this->lijst->vergelijkingen[$i]->kleur; $this->lijst->scores[$i]->kleurV = $this->lijst->vergelijkingen[$i]->kleur;
$this->lijst->scores[$i]->disabledV = $this->lijst->vergelijkingen[$i]->disabled; $this->lijst->scores[$i]->disabledV = $this->lijst->vergelijkingen[$i]->disabled;
application/models/monpas/Scorekleuren_model.php
+11
View File
@@ -23,6 +23,17 @@ class Scorekleuren_model extends MY_Model{
return $this->get_query_row(); return $this->get_query_row();
} }
function getWensKleur($scoretype, $nummer){
$this->sql="SELECT kleurcode
FROM tb_scorekleuren
WHERE scoretype = {$scoretype}
AND nummer = {$nummer}
AND disabled = 'F' ";
return $this->get_query_row('kleurcode');
}
application/models/monpas/Scores_model.php
+5 -8
View File
@@ -636,7 +636,7 @@ class Scores_model extends MY_Model{
function getgemscores($orgversieid, $orgids){ function getgemscores($orgversieid, $orgids){
$gevuldeversies = $this->fGetGevuldeVersies($orgversieid, $orgids); $gevuldeversies = $this->fGetGevuldeVersies($orgversieid, $orgids);
if ($gevuldeversies) { if ($gevuldeversies) {
$this->sql = " SELECT Q.omschrijving, Q.toelichting, Q.ijkpunt_id, Q.kleurnummer, Q.disabled, Q.scoretype, $this->sql = " SELECT Q.omschrijving, Q.toelichting, Q.ijkpunt_id, Q.disabled, Q.scoretype,
CASE WHEN Q.aantalvier > 0 THEN CASE WHEN Q.aantalvier > 0 THEN
CASE WHEN Q.aantalvier = Q.aantalnvt THEN CASE WHEN Q.aantalvier = Q.aantalnvt THEN
'T' 'T'
@@ -644,11 +644,11 @@ class Scores_model extends MY_Model{
END END
ELSE 'F' ELSE 'F'
END AS disablenvt, END AS disablenvt,
Q.sumscores/Q.sumgewicht AS gem, Q.kleurcode, Q.nummer, Q.ondergrens, Q.bovengrens Q.sumscores/Q.sumgewicht AS gem, Q.nummer
FROM ( FROM (
SELECT Y.omschrijving, Y.toelichting,Y.id as ijkpunt_id, SELECT Y.omschrijving, Y.toelichting,Y.id as ijkpunt_id,
Y.nummer, Y.disabled, SC.scoretype, SK.nummer AS kleurnummer, Y.nummer, Y.disabled, SC.scoretype,
SK.kleurcode, SK.ondergrens, SK.bovengrens,
SUM(CASE WHEN Y.disabled = 'T' THEN 0 SUM(CASE WHEN Y.disabled = 'T' THEN 0
WHEN Y.disabled = 'F' THEN WHEN Y.disabled = 'F' THEN
CASE WHEN SC.scoretype = 1 CASE WHEN SC.scoretype = 1
@@ -715,12 +715,9 @@ class Scores_model extends MY_Model{
FROM tb_scores SC FROM tb_scores SC
JOIN tb_indicatoren I ON I.id = SC.indicator_id JOIN tb_indicatoren I ON I.id = SC.indicator_id
JOIN tb_ijkpunten Y ON Y.id = SC.ijkpunt_id JOIN tb_ijkpunten Y ON Y.id = SC.ijkpunt_id
JOIN tb_scorekleuren SK ON SK.scoretype = SC.scoretype
WHERE SC.filledversion_id IN ({$gevuldeversies}) WHERE SC.filledversion_id IN ({$gevuldeversies})
AND SK.disabled = Y.disabled
GROUP BY SC.ijkpunt_id GROUP BY SC.ijkpunt_id
ORDER BY Y.nummer ) AS Q ORDER BY Y.nummer ) AS Q ";
HAVING Q.ondergrens <= gem AND Q.bovengrens > gem ";
} }
else return false; else return false;
return $this->get_query_result(); return $this->get_query_result();
application/views/monpas/pages/matrix.php
+1
View File
@@ -227,6 +227,7 @@
<span data-bind="text: subject "></span> <span data-bind="text: subject "></span>
</td> </td>
<!-- ko foreach: results--> <!-- ko foreach: results-->
<td style="text-align:right; color:#ffffff; width:35px; padding:; border: 3px solid white; " <td style="text-align:right; color:#ffffff; width:35px; padding:; border: 3px solid white; "
data-bind="text: gem, style:{ backgroundColor: colorcode}, css: {matrixstreep: $parent.subject == 'gemiddeld'}"> data-bind="text: gem, style:{ backgroundColor: colorcode}, css: {matrixstreep: $parent.subject == 'gemiddeld'}">
</td> </td>
public/js/monpas/models/ijkpuntenmodel.js
+2 -2
View File
@@ -464,7 +464,7 @@ ls.initIJkpuntenModel = function(){
self.ijkpunten.removeAll(); self.ijkpunten.removeAll();
self.aantal(''); self.aantal('');
var teller = 0; var teller = 0;
if(result ) { if(result.lijsten[0].scores ) {
$.each(result.lijsten, function (i, p) { $.each(result.lijsten, function (i, p) {
if (p.scores || p.vergelijkingen) { if (p.scores || p.vergelijkingen) {
self.erzijnlijsten(true); self.erzijnlijsten(true);
@@ -587,7 +587,7 @@ ls.initIJkpuntenModel = function(){
'selectie': selectieOrgversies, 'selectie': selectieOrgversies,
'orgid': self.selectedOrg().id 'orgid': self.selectedOrg().id
}; };
console.log(submitData, "submitdata")
// showAjaxLoader(); // showAjaxLoader();
self.lijsten.removeAll(); self.lijsten.removeAll();
public/js/monpas/models/matrixmodel.js
+5 -2
View File
@@ -91,7 +91,9 @@ $(function () {
this.organisaties = ko.observableArray([]); this.organisaties = ko.observableArray([]);
if (p.users) { if (p.users) {
$.each(p.users, function (j, k) { $.each(p.users, function (j, k) {
dit.organisaties.push(new fAddOrgIndeling(k)); dit.organisaties.push(new fAddOrgIndeling(k));
}) })
} }
} }
@@ -126,8 +128,8 @@ $(function () {
this.gem = Math.round(parseFloat(k.gem)); this.gem = Math.round(parseFloat(k.gem));
} }
this.num = k.nummer; this.num = k.nummer;
this.colorcode = '#e08885'; // this.colorcode = '#e08885';
if(k.kleurcode) this.colorcode = k.kleurcode; this.colorcode = k.kleur ? k.kleur : "#cccccc";
if (k.disabled == "T" || k.disablenvt == "T") { if (k.disabled == "T" || k.disablenvt == "T") {
this.gem=''; this.gem='';
this.colorcode="#cccccc"; this.colorcode="#cccccc";
@@ -264,6 +266,7 @@ $(function () {
// wordt aangeroepen voor dat alle orgversions zijn ingelezen en selectedOrgversion is gezet. // wordt aangeroepen voor dat alle orgversions zijn ingelezen en selectedOrgversion is gezet.
self.bezig(true); self.bezig(true);
self.orgversions.removeAll(); self.orgversions.removeAll();
if (data) { if (data) {
$.each(data, function (i, p) { $.each(data, function (i, p) {
self.orgversions.push(new fAddOrgversion(p)); self.orgversions.push(new fAddOrgversion(p));