PatrickLubbers / educom-webshop-oop-1706014168

0 stars 0 forks source link

In deze functie haal je 2x het user_id op uit de $_SESSION en geen een keer met de functie uit SessionManager #22

Open JeroenHeemskerk opened 8 months ago

JeroenHeemskerk commented 8 months ago

In de code hieronder https://github.com/PatrickLubbers/educom-webshop-oop-1706014168/blob/d883e565d7926f4e8ca09493785738a7e583aa83/controllers/pageController.php#L100-L106

Haal je 2x de code op uit $_SESSION['user_id'] op regel 101 en 104, maar je gebruikt niet de functie uit de sessionmanager die je hier wel gebruikt: https://github.com/PatrickLubbers/educom-webshop-oop-1706014168/blob/d883e565d7926f4e8ca09493785738a7e583aa83/controllers/pageController.php#L94 waarom deze inconsistentie?

Daarnaast vraag ik me ook af waarom deze functie in de controller staat, ik zou verwachten dat zoiets in de model wordt afgehandeld