zes logische fouten (toewijzing i.p.v. vergelijking, typfouten)

- Overzichtscores::geteditrecht: '$recht == true' en '$this->uderrole'
  (typfout) maakten de rechtencontrole voor kamer en bestuur onjuist
- Users::getorgs: 'soort=1' overschreef de soort; blok draait bewust
  voor alle soorten (gebruikersmodel.js gebruikt die boom)
- Sopoud: '$module = 4' -> '=='
- Compacteversies: 'schooltype = 'R'' -> SBO-modules krijgen label SBO
- Diversiteitmonitor: typfout $swvID -> $swvid
- Monfinoverzicht::getbedragen: $budget -> $budgetten

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
This commit is contained in:
2026-10-22 13:43:27 +02:00
co-authored by Claude Opus 5.5
parent 96d351c2a2
commit 0ad26b0583
6 changed files with 10 additions and 7 deletions
@@ -51,7 +51,7 @@ class Diversiteitmonitor extends MY_Controller{
case 2:
case 3: $swvid = $this->Organisaties_model->get_swvid($orgid);
break;
default: $swvID = 0;
default: $swvid = 0;
}
$criteria = $this->Sorteringen_model->get_criteriabijSWV($swvid);
@@ -251,7 +251,7 @@ class Monfinoverzicht extends MY_Controller{
$totbudget = $budgetten->basis + $budgetten->extra + $budgetten->overig;
}
else {
$totbudget = $budget->som+ $budget->overig;
$totbudget = $budgetten->som + $budgetten->overig;
}
}
@@ -53,9 +53,9 @@ class Overzichtscores extends MY_Controller{
$recht = true;
}
if($this->globorgid == $kamerid && $this->userrole <= 32){//werkt bij kamer
$recht == true;
$recht = true;
}
if($this->globorgid == $bgid && $this->uderrole <=128){//werkt bij bg
if($this->globorgid == $bgid && $this->userrole <=128){//werkt bij bg
$recht = true;
}
if($this->globorgid == $orgid && $this->userrole <= 256){//management op school
+1 -1
View File
@@ -332,7 +332,7 @@ class Sopoud extends MY_Controller{
$imageoa = $this->Phantom_model->create_img($result['tabel1'], 'test.png', FALSE, FALSE, array(650, 300));
$id = "oa1image1";
if ( $module = 4) { $id = "oa2image1";};
if ( $module == 4) { $id = "oa2image1";};
$imagedata = "<img src='data:image/png;base64,".$imageoa."' id='".$id."'/>";
echo json_encode(array('image1' => $imagedata ,
+3 -1
View File
@@ -70,7 +70,9 @@ class Users extends MY_Controller{
}
}
}
if ($userorg->soort=1){
// boom in 'orgs' voor elke soort; gebruikersmodel.js gebruikt alleen deze
// (was 'if ($userorg->soort=1)': een toewijzing die altijd waar was en soort overschreef)
{
$userorg->orgs= $this->Organisaties_model->getorgsvanaf($userorg->id);
if ($userorg->orgs){
foreach($userorg->orgs as $kamer){
@@ -159,7 +159,8 @@ class Compacteversies
$this->onderdeel->groepen = $this->CI->Groepen_model->get_AantalGroepen($this->schooljaarnr, $this->orgid, $module);
$this->onderdeel->erIsDiversiteit = 'T';
$this->onderdeel->gegevens = 'T';
if($this->onderdeel->schooltype = 'R') $this->onderdeel->schoolsoort = 'PO';
// was 'schooltype = 'R'' (toewijzing): alles werd PO en schooltype werd overschreven
$this->onderdeel->schoolsoort = $this->onderdeel->schooltype == 'SBO' ? 'SBO' : 'PO';
$this->onderdeel->versie = $this->schooljaar->naam. ', '. $this->onderdeel->schoolsoort;
}