Het try-catch block voor deze functies staat op de verkeerde plek

In onderstaande functies staat steeds een try-catch block, en wordt een functie uit het model aangeroepen om vervolgens een veld of een error in het model te zetten: https://github.com/Corby7/educom-webshop-oop/blob/839ba29e48aef1a3f6fe2bb29165e3048fcbe0b3/controllers/pagecontroller.php#L42-L74

Dit is niet (meer) correct, dit zou in onderstaande functies moeten gebeuren in het ShopModel:

https://github.com/Corby7/educom-webshop-oop/blob/839ba29e48aef1a3f6fe2bb29165e3048fcbe0b3/models/shopmodel.php#L22-L25 https://github.com/Corby7/educom-webshop-oop/blob/839ba29e48aef1a3f6fe2bb29165e3048fcbe0b3/models/shopmodel.php#L27-L30 https://github.com/Corby7/educom-webshop-oop/blob/839ba29e48aef1a3f6fe2bb29165e3048fcbe0b3/models/shopmodel.php#L32-L34

Tevens mis ik de terugkoppeling zoals deze wel staat bij register https://github.com/Corby7/educom-webshop-oop/blob/839ba29e48aef1a3f6fe2bb29165e3048fcbe0b3/controllers/pagecontroller.php#L96

De case in pagecontroller.php zou alleen moeten zijn:

 case 'webshop': 
     $this->model = new ShopModel($this->model); 

     $this->model->getWebshopData(); 
     break;

en in shopmodel.php zou het moeten zijn:

 public function getWebshopData() { 
     try {
         require_once("mysqlconnect.php"); 
         $this->products = getAllProducts(); 
     } catch (Exception $e) { 
         logError("Get all products failed: " . $e->getMessage()); 
         $this->genericErr = "Sorry technisch probleem, gegevens ophalen niet mogelijk"; 
     } 
 }

Idem voor de andere twee cases

Corby7 / educom-webshop-oop

Het try-catch block voor deze functies staat op de verkeerde plek #22