Skip to content

Commit

Permalink
Include the reference model name in the build query when getting rela…
Browse files Browse the repository at this point in the history
…ted records

When we overload the getRelated function and join more tables, it can happen that the referencedField/s can't be mapped back correctly to the referenceModel it not specified
  • Loading branch information
alrieckert committed Oct 1, 2024
1 parent 04edb15 commit b33bbb3
Showing 1 changed file with 2 additions and 2 deletions.
4 changes: 2 additions & 2 deletions phalcon/Mvc/Model/Manager.zep
Original file line number Diff line number Diff line change
Expand Up @@ -1460,11 +1460,11 @@ class Manager implements ManagerInterface, InjectionAwareInterface, EventsAwareI
let referencedFields = relation->getReferencedFields();

if typeof fields != "array" {
let conditions[] = "[". referencedFields . "] = :APR0:",
let conditions[] = "[" . referencedModel . "].[". referencedFields . "] = :APR0:",
placeholders["APR0"] = record->readAttribute(fields);
} else {
for refPosition, field in relation->getFields() {
let conditions[] = "[". referencedFields[refPosition] . "] = :APR" . refPosition . ":",
let conditions[] = "[" . referencedModel . "].[". referencedFields[refPosition] . "] = :APR" . refPosition . ":",
placeholders["APR" . refPosition] = record->readAttribute(field);
}
}
Expand Down

0 comments on commit b33bbb3

Please sign in to comment.