r11710 breaks recursive ManyToManyField

[knutin@…]

Revision r11710(#10109) breaks recursive ManyToManyField, ie. ManyToManyField('self').

This appears to be caused by a change in the column names for intermediate tables(_get_m2m_attr() maybe).

Previous versions, for example 11709, generates column names like this: from_foo_id, to_foo_id.

[Alex Gaynor]

What names does it generate now? I thought I was pretty careful to make sure nothing had it's name changed.

[Knut Nesheim]

Output from sqlall before r11710:

CREATE TABLE "transport_transportorder_related_orders" (
    "id" serial NOT NULL PRIMARY KEY,
    "from_transportorder_id" integer NOT NULL REFERENCES "transport_transportorder" ("id") DEFERRABLE INITIALLY DEFERRED,
    "to_transportorder_id" integer NOT NULL REFERENCES "transport_transportorder" ("id") DEFERRABLE INITIALLY DEFERRED,
    UNIQUE ("from_transportorder_id", "to_transportorder_id")
)
;

After:

CREATE TABLE "transport_transportorder_related_orders" (
    "id" serial NOT NULL PRIMARY KEY,
    "from_transportorder_id" integer NOT NULL,
    "transportorder_id" integer NOT NULL,
    UNIQUE ("from_transportorder_id", "transportorder_id")
)
;

[Alex Gaynor]

Can you confirm whether this fixes the issue for you?

[Russell Keith-Magee]

(In [11713]) Fixed #12143 -- Corrected the naming of the 'to' column in recursive m2m models. Thanks to knutin@… for the report, and Alex Gaynor for the patch.