refactored appr model references

This commit is contained in:
Djalim Simaila 2023-01-26 13:48:02 +01:00
parent 332a98958f
commit 5d0e19b45f
2 changed files with 7 additions and 10 deletions

View File

@ -11,9 +11,8 @@ final class ApprController
$S_comment = Utils::getOrDie($A_postParams, "comment"); $S_comment = Utils::getOrDie($A_postParams, "comment");
$I_note = Utils::intOrDie(Utils::getOrDie($A_postParams, "note")); $I_note = Utils::intOrDie(Utils::getOrDie($A_postParams, "note"));
$O_apprModel = new ApprModel(); $O_appr = new ApprModel($S_comment, $I_note,date("Y-m-d"),$_SESSION["ID"],$I_recipe_id);
$O_apprModel->createAppr($_SESSION["ID"], $I_recipe_id, $S_comment, $I_note); $O_appr->insert();
header("Location: ".$_SERVER['HTTP_REFERER']); header("Location: ".$_SERVER['HTTP_REFERER']);
} }
@ -23,18 +22,17 @@ final class ApprController
$I_appr_id = Utils::intOrDie($A_urlParams[0]); $I_appr_id = Utils::intOrDie($A_urlParams[0]);
$O_apprModel = new ApprModel(); $O_appr = ApprModel::getApprById($I_appr_id);
$A_appr = $O_apprModel->getApprById($I_appr_id);
if ($A_appr === null) { if ($O_appr === null) {
throw new HTTPSpecialCaseException(404); throw new HTTPSpecialCaseException(404);
} }
if ($A_appr["AUTHOR_ID"] !== $_SESSION["ID"]) { if ($O_appr->I_AUTHOR_ID !== $_SESSION["ID"]) {
Session::admin_or_die(); Session::admin_or_die();
} }
$O_apprModel->deleteAppr($I_appr_id); $O_appr->delete();
header("Location: ".$_SERVER['HTTP_REFERER']); header("Location: ".$_SERVER['HTTP_REFERER']);
} }

View File

@ -142,8 +142,7 @@ final class RecipeModel
public function getApprs(){ public function getApprs(){
if ($this->A_APPRS === null) { if ($this->A_APPRS === null) {
$O_apprModel = new ApprModel(); $this->A_APPRS = ApprModel::searchRecipeApprs($this->I_ID);
$this->A_APPRS = $O_apprModel->searchRecipeApprsWithAuthors($this->I_ID);
} }
return $this->A_APPRS; return $this->A_APPRS;
} }