Changes between Version 1 and Version 2 of Ticket #14549, comment 7


Ignore:
Timestamp:
Feb 17, 2014, 12:03:32 AM (11 years ago)
Author:
loic84

Legend:

Unmodified
Added
Removed
Modified
  • Ticket #14549, comment 7

    v1 v2  
    44
    55- The `through_` prefix makes it obvious it's an option that is only available to custom `through` relations.
    6 - IMO what is "source" and "target" doesn't always make sense for M2M (i.e. anything other than `symmetrical` and relations to `self`). In that respect, I would document it as `through_fields=('<field1>', '<field2>')` and leave the concept of "source" and "target" out of the equation (with a special note for `symmetrical=False`).
     6- IMO what is "source" and "target" doesn't always make sense for M2M (i.e. anything other than relations to `self` with `symmetrical=False`). In that respect, I would document it as `through_fields=('<field1>', '<field2>')` and leave the concept of "source" and "target" out of the equation (with a special note for `symmetrical=False`).
    77- I prefer being fully explicit, rather than having a fallback mechanism for only one of the FK.
    88
Back to Top