Changes between Version 1 and Version 2 of Ticket #29466, comment 2


Ignore:
Timestamp:
Jun 10, 2018, 11:25:06 AM (6 years ago)
Author:
Alexandr Tatarinov

Legend:

Unmodified
Added
Removed
Modified
  • Ticket #29466, comment 2

    v1 v2  
    1 I will assume that this is intended behavior. According to #24215, abstract models does not register pending lookups into Apps registry. I  have tried to implement it, but found an issue with recursive relations in abstract model - they cannot be resolved (because abstract models is not registered), but it is valid to have them, because they can be resolved in concrete children classes. So for abstract models lazy-referenced relations should not be resolved. I would not change the triage stage as I am not sure about this.
     1I will assume that this is intended behavior. According to #24215, abstract models does not register pending lookups into Apps registry. I  have tried to implement it, but found an issue with recursive relations in abstract model - they cannot be resolved (because abstract models is not registered), but it is valid to have them, because they can be resolved in concrete children classes. So for abstract models lazy-referenced relations should not be resolved. I am not sure about this, need someone more proficient to take a look.
Back to Top