Luca/thetool
1
0
Fork 0
Code Issues 5 Pull Requests 5 Actions Packages Projects Releases Wiki Activity

Merge branch 'fronkdev' into 'master'

Browse Source 
Fixed bug which prevented Preorder::afterSave() from running

See merge request fronk/thetool!87
This commit is contained in:
Frank Schubert
2023-09-28 09:46:22 +00:00
parent 9585ee1b87 d0bcb9a369
commit 6c35e27906
1 changed files with 4 additions and 1 deletions
Download Patch File Download Diff File Expand all files Collapse all files

5
application/Preorder/Preorder.php
View File

@@ -20,7 +20,8 @@ class Preorder extends mfBaseModel {
}
public function afterSave() {
if(!$this->in_after_save) return true;
// prevent potential infinite loop
if($this->in_after_save) return true;
$this->in_after_save = true;
// update preorder OAID if it's different from the unit OAID
// but only if the unit OAID is of the same origin as the campaign
@@ -34,6 +35,7 @@ class Preorder extends mfBaseModel {
public function setOrCreateOaid($oaid_attributes = false) {
$campaign = new Preordercampaign($this->preordercampaign_id);
//var_dump($campaign);exit;
if(!$campaign->id) {
return false;
}
@@ -53,6 +55,7 @@ class Preorder extends mfBaseModel {
}
} elseif($campaign->oaid_origin == "ofaa") {
$current_oaid = OpenAccessIdModel::getFirstOaid($this->oaid);
//var_dump($current_oaid);exit;
if(!$current_oaid) {
$this->log->error("OAID of Preorder ".$this->id." not found in OpenAccessIds");
// assume it's from a different origin