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/Antwoordtypes_model",
"monpas/Wenssituaties_model",
"monpas/Scorekleuren_model",
"monpas/Scores_model",
'shared/Session_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)){
$user->scores[$i]->disablenvt= 'F';
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.98';// instrumenten voor ouders
// 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){
$grensnummer = 1;
if ($score>= $this->kleuren->grens1) $grensnummer = 2;
if ($score>= $this->kleuren->grens2) $grensnummer = 3;
if ($score>= $this->kleuren->grens3) $grensnummer = 4;
$kleur = $this->CI->Scorekleuren_model->getGemiddeldeKleur($score->scoretype, $score->gem);
$kleur = $kleur->kleurcode;
if(!$kleur) $kleur = '#ffffff';
$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;
}
@@ -243,14 +228,15 @@ class Gemiddeldenafbeeldingen {
}
private function loopIJkpunten2(){
if(!$this->ijkpunten) return;
$scoretype = $this->ijkpunten[0]->scoretype;
foreach($this->ijkpunten as $index => $this->ijkpunt){
if(!isset($this->ijkpunt->score)){continue;};
if(! $this->ijkpunt->score) $this->ijkpuntscore = 0;
$this->setScorenummer();
$this->rondScoresAf();
$this->setScoreKleuren();
$this->setScoreKleuren($scoretype);
$this->addGewensteScores($index);
$this->setWenskleuren();
$this->setWenskleuren($scoretype);
$this->setAdvies();
$this->setResultatenOverzicht();
$this->telIJkpuntenok();
@@ -276,23 +262,31 @@ class Gemiddeldenafbeeldingen {
$this->ijkpunt->score = round($this->ijkpunt->score);
}
private function setScoreKleuren(){
switch ($this->ijkpunt->huidig) {
case 1:
$this->ijkpunt->kleur = $this->kleuren->kleur1;
break;
case 2:
$this->ijkpunt->kleur = $this->kleuren->kleur2;
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;
}
private function setScoreKleuren($scoretype){
$kleur = $this->CI->Scorekleuren_model->getGemiddeldeKleur($scoretype, $this->ijkpunt->score);
$kleur = $kleur->kleurcode;
if(!$kleur) $kleur = '#ffffff';
$this->i->kljkpunt = $kleur;
// switch ($this->ijkpunt->huidig) {
// case 1:
// $this->ijkpunt->kleur = $this->kleuren->kleur1;
// break;
// case 2:
// $this->ijkpunt->kleur = $this->kleuren->kleur2;
// 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;
switch ($this->ijkpunt->wens) {
case 1:
$this->ijkpunt->wenskleur = $this->kleuren->kleur1;
break;
case 2:
$this->ijkpunt->wenskleur = $this->kleuren->kleur2;
break;
case 3:
$this->ijkpunt->wenskleur = $this->kleuren->kleur3;
break;
case 4:
$this->ijkpunt->wenskleur = $this->kleuren->kleur4;
break;
default:
$this->ijkpunt->wenskleur = '#ffffff';
}
$kleur = $this->CI->Scorekleuren_model->getWensKleur($scoretype, $this->ijkpunt->wens);
if(!$kleur) $kleur = '#ffffff';
$this->ijkpunt->wenskleur = $kleur;
// switch ($this->ijkpunt->wens) {
// case 1:
// $this->ijkpunt->wenskleur = $this->kleuren->kleur1;
// break;
// case 2:
// $this->ijkpunt->wenskleur = $this->kleuren->kleur2;
// break;
// 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(){
application/libraries/monpas/Schoolafbeeldingen.php
+33 -39
View File
@@ -224,12 +224,13 @@ class Schoolafbeeldingen {
private function loopScores2()
{
if (!$this->scores) return;
$scoretype = $this->scores[0]->scoretype;
foreach ($this->scores as $index => $this->score) {
$this->setScoreNummer();
$this->rondScoresAf();
$this->setScoreKleuren();
$this->setScoreKleuren($scoretype);
$this->addGewensteScores($index);
$this->setWensKleuren();
$this->setWensKleuren($scoretype);
$this->setAdvies();
$this->setResultatenOverzicht();
$this->telIJKpuntenok();
@@ -257,23 +258,31 @@ class Schoolafbeeldingen {
}
private function setScoreKleuren(){
switch ($this->score->huidig) {
case 1:
$this->score->kleur = $this->kleuren->kleur1;
break;
case 2:
$this->score->kleur = $this->kleuren->kleur2;
break;
case 3:
$this->score->kleur = $this->kleuren->kleur3;
break;
case 4:
$this->score->kleur = $this->kleuren->kleur4;
break;
default:
$this->score->kleur = '#ffffff';
}
private function setScoreKleuren($scoretype){
$kleur = $this->CI->Scorekleuren_model->getGemiddeldeKleur($scoretype, $this->score->gem);
$kleur = $kleur->kleurcode;
if(!$kleur) $kleur = '#ffffff';
$this->score->kleur = $kleur;
// switch ($this->score->huidig) {
// case 1:
// $this->score->kleur = $this->kleuren->kleur1;
// break;
// case 2:
// $this->score->kleur = $this->kleuren->kleur2;
// 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->score->wens)) return;
if($this->score->disabled == 'T')
{$this->score->wenskleur = '#ffffff';
return;}
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';
}
$kleur = $this->CI->Scorekleuren_model->getWensKleur($scoretype, $this->score->wens);
if(!$kleur) $kleur = '#ffffff';
$this->score->wenskleur = $kleur;
}
private function setAdvies(){
application/libraries/monpas/Schoolscores.php
+1 -1
View File
@@ -5,6 +5,7 @@ class Schoolscores
private $CI;
private $ijkpunten;
private $scores;
private $score;
private $ambities;
private $wensen;
private $schoolScores;
@@ -57,7 +58,6 @@ class Schoolscores
$this->scores = $this->CI->Scores_model->getgemscores($this->orgversieid, $this->orgid);
}
private function setAmbities(){
$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->setSWVid();
$this->setOrgversie();
$this->setKleurcodes();
// $this->setKleurcodes();
$this->setKleuren();
$this->loopScores1();
$this->setGemiddeldeScore();
@@ -185,10 +185,11 @@ class Afbeeldingen {
private function loopScores2(){
if (!$this->scores) return;
$scoretype = $this->scores[0]->scoretype;
foreach ($this->scores as $index => $this->score) {
$this->setScoreNummer();
$this->rondScoresAf();
$this->setScoreKleuren();
$this->setScoreKleuren($scoretype);
$this->setResultatenOverzicht();
$this->telIJkpuntenok();
}
@@ -206,23 +207,29 @@ class Afbeeldingen {
}
private function setScoreKleuren(){
switch ($this->score->huidig) {
case 1:
$this->score->kleur = $this->kleuren->kleur1;
break;
case 2:
$this->score->kleur = $this->kleuren->kleur2;
break;
case 3:
$this->score->kleur = $this->kleuren->kleur3;
break;
case 4:
$this->score->kleur = $this->kleuren->kleur4;
break;
default:
$this->score->kleur = '#ffffff';
}
private function setScoreKleuren($scoretype){
$kleur = $this->CI->Scorekleuren_model->getGemiddeldeKleur($scoretype, $this->score->gem);
$kleur = $kleur->kleurcode;
if(!$kleur) $kleur = '#ffffff';
$this->score->kleur = $kleur;
// switch ($this->score->huidig) {
// case 1:
// $this->score->kleur = $this->kleuren->kleur1;
// break;
// case 2:
// $this->score->kleur = $this->kleuren->kleur2;
// break;
// case 3:
// $this->score->kleur = $this->kleuren->kleur3;
// break;
// case 4:
// $this->score->kleur = $this->kleuren->kleur4;
// break;
// default:
// $this->score->kleur = '#ffffff';
// }
}
private function setResultatenOverzicht(){
application/libraries/monpas/ijkpunten/Lijsten.php
+5 -8
View File
@@ -197,10 +197,6 @@ class Lijsten
}
}
private function setSchoolScores(){
if ($this->soort != 4 || $this->lijst->vorm != 1) return;
$this->schoolScores = new Schoolscores($this->lijst->orgversieid, $this->orgid);
@@ -365,14 +361,15 @@ class Lijsten
}
private function loopVergelijkingScores(){
if( (!isset($this->lijst->vergelijkingen) || !$this->lijst->vergelijkingen)) return;
for($i = 0; $i < count($this->lijst->vergelijkingen); $i++){
$this->setVergelijkingScores($i);
if(!isset($this->lijst->vergelijkingen) || !$this->lijst->vergelijkingen) return;
foreach($this->lijst->vergelijkingen AS $index => $vergelijking){
$this->setVergelijkingScores($index);
}
}
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]->kleurV = $this->lijst->vergelijkingen[$i]->kleur;
$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();
}
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){
$gevuldeversies = $this->fGetGevuldeVersies($orgversieid, $orgids);
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 = Q.aantalnvt THEN
'T'
@@ -644,11 +644,11 @@ class Scores_model extends MY_Model{
END
ELSE 'F'
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 (
SELECT Y.omschrijving, Y.toelichting,Y.id as ijkpunt_id,
Y.nummer, Y.disabled, SC.scoretype, SK.nummer AS kleurnummer,
SK.kleurcode, SK.ondergrens, SK.bovengrens,
Y.nummer, Y.disabled, SC.scoretype,
SUM(CASE WHEN Y.disabled = 'T' THEN 0
WHEN Y.disabled = 'F' THEN
CASE WHEN SC.scoretype = 1
@@ -715,12 +715,9 @@ class Scores_model extends MY_Model{
FROM tb_scores SC
JOIN tb_indicatoren I ON I.id = SC.indicator_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})
AND SK.disabled = Y.disabled
GROUP BY SC.ijkpunt_id
ORDER BY Y.nummer ) AS Q
HAVING Q.ondergrens <= gem AND Q.bovengrens > gem ";
ORDER BY Y.nummer ) AS Q ";
}
else return false;
return $this->get_query_result();
application/views/monpas/pages/matrix.php
+1
View File
@@ -227,6 +227,7 @@
<span data-bind="text: subject "></span>
</td>
<!-- ko foreach: results-->
<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'}">
</td>
public/js/monpas/models/ijkpuntenmodel.js
+2 -2
View File
@@ -464,7 +464,7 @@ ls.initIJkpuntenModel = function(){
self.ijkpunten.removeAll();
self.aantal('');
var teller = 0;
if(result ) {
if(result.lijsten[0].scores ) {
$.each(result.lijsten, function (i, p) {
if (p.scores || p.vergelijkingen) {
self.erzijnlijsten(true);
@@ -587,7 +587,7 @@ ls.initIJkpuntenModel = function(){
'selectie': selectieOrgversies,
'orgid': self.selectedOrg().id
};
console.log(submitData, "submitdata")
// showAjaxLoader();
self.lijsten.removeAll();
public/js/monpas/models/matrixmodel.js
+5 -2
View File
@@ -91,7 +91,9 @@ $(function () {
this.organisaties = ko.observableArray([]);
if (p.users) {
$.each(p.users, function (j, k) {
dit.organisaties.push(new fAddOrgIndeling(k));
})
}
}
@@ -126,8 +128,8 @@ $(function () {
this.gem = Math.round(parseFloat(k.gem));
}
this.num = k.nummer;
this.colorcode = '#e08885';
if(k.kleurcode) this.colorcode = k.kleurcode;
// this.colorcode = '#e08885';
this.colorcode = k.kleur ? k.kleur : "#cccccc";
if (k.disabled == "T" || k.disablenvt == "T") {
this.gem='';
this.colorcode="#cccccc";
@@ -264,6 +266,7 @@ $(function () {
// wordt aangeroepen voor dat alle orgversions zijn ingelezen en selectedOrgversion is gezet.
self.bezig(true);
self.orgversions.removeAll();
if (data) {
$.each(data, function (i, p) {
self.orgversions.push(new fAddOrgversion(p));