When Many-to-Many Through Associations release?

The docs said it is coming soon. while I wait it is released is the alternative way to attack addition attributes?

I am able to get this mostly working (not fully tested). It's a pretty manual job, and finnicky! For now, don't bother with attribute methods or lifecycle callbacks, as they'll get overwritten. I have an issue about that here: #707. Here's an example. I would love for someone to let me know if there's any easier way! In particular, I'm confused by the via attribute on the through table's foreign keys, and something might not be quite right. I suppose a lookup might happen, as the through table never references the other tables' foreign keys; thus, there's no clear connection between the humanFriends and dogFriends attributes. Nonetheless, it's working!

var Dogs= {
        identity: 'dogs',     
        attributes: {
            humanFriends: {
                collection: 'humans',
                through: 'friendships'
            } 
        }
},

var Friendships = {
        identity: 'friendships',
        attributes: {
            someOtherAttr: 'string',
            dog: {
                columnName: 'dog',
                type: 'integer',
                foreignKey: true,
                references: 'dogs',
                on: 'id',
                onKey: 'id',
                via: 'human'
            },
            human: {
                columnName: 'human',
                type: 'integer',
                foreignKey: true,
                references: 'humans',
                on: 'id',
                onKey: 'id',
                via: 'dog'
            }
        }
}

var Humans = {
        identity: 'humans',
        attributes: {
            dogFriends: {
                collection: 'dogs',
                through: 'friendships'
            } 
        }
}

Not working for me... TypeError: Cannot read property of 'via' of undefined :(

@mewben

@mewben, do you get that error right when waterline is initialized / when your sails lift (if you're using sails)? And which version of waterline are you using? I'd love to find a reproducible way to hack through-associations into working correctly!

I'm using sails v0.10.5... sails lifts okay though.. the error triggers when using .add() in the association and then .save() to commit...

@mewben

@mewben using that exact example I gave you above, this is working for me. I'm on waterline 0.10.12 using sails-disk. Perhaps if you follow that error's stack trace, we can figure out if it's simply the version of waterline that you're running.

I'm using the waterline v0.10.9 which sails v0.10.5 is using.. I guess I'll have to wait for sails to bump their waterline dependency then...

@devinivy

I've tried a similar code with v0.10.12 and it worked for me, thanks @devinivy.

@devinivy

The issue title did not correspond to the question in the issue body here.. Is there a date for the 'official' associations-attributes implementation? Or is it just the docs that are missing, and @devinivy has the right way to do it?

I'm not sure about an official date/release– sadly it doesn't seem to be in the 0.11 milestone: https://github.com/balderdashy/waterline/milestones/0.11 . My method works, but it's undocumented. If you peek through the waterline-schema codebase you'll see some references to the through keyword. I expect when this is officially supported that the model configuration on the junction-collection will be simpler than my example above.

@NiclasLindqvist

@NiclasLindqvist, the waterline-adapter-tests give a hint as to what may be the right way to describe many-to-many through associations in the future. Check hasManyThrough.venue.fixture.js (join table) and hasManyThrough.team.fixture.js. This approach works for joins (relevant test) but it seems to fail when creating an association. Using @devinivy's method does work for both creating associations and joins which makes me think that many-to-many through associations will be officially supported soon (fingers crossed).

@devinivy

@devinivy what statement are you using to populate your join model once humans and dogs are populated? Also, is declaring the variables necessary?

@kirkaleze

@kirkaleze declaring the variables is merely for illustrative purposes. The populate query would then look quite normal:
DogCollection.find().populate('humanFriends').exec(function(err, dogs) {});

But you should also be able to query the junction table:
FriendshipsCollection.find().exec(function(err, friendships) {});

@elennaro

It would also be good to be able to store data in the join table (association) while saving. Quoting @elennaro (#957):

title: extra fields on many to many relationships join table
It's a widely used approach when we save extra data with relation.
Like quantity for sold products in order-product relation.
Like expiration on user-post-ban. etc. etc.

Yes, indeedy! Info on the join table should also be included appear when populating records during a find.

Sorry for a noob question, but is there any manual way to overwrite find for humans somehow to get both Dogs and identity field from friendship?

@cuongcua90

Thanks for posting, @cuongcua90. I'm a repo bot-- nice to meet you!

It has been 30 days since there have been any updates or new comments on this page. If this issue has been resolved, feel free to disregard the rest of this message. On the other hand, if you are still waiting on a patch, please:

review our contribution guide to make sure this submission meets our criteria (only verified bugs with documented features, please; no questions, commentary, or bug reports about undocumented features or unofficial plugins)
create a new issue with the latest information, including updated version details with error messages, failing tests, and a link back to the original issue. This allows GitHub to automatically create a back-reference for future visitors arriving from search engines.

Thanks so much for your help!

Reopening, as this is in a feature request in an active milestone. We can close once it's PRed into the roadmap.

@devinivy

@devinivy Any ETA on when this will be available? Any patches we can integrate today?

@mukk85

@mukk85 this is essentially usable– the configuration for it is just ugly and it's considered experimental. @atiertant might have a patch that you can integrate?

@devinivy

@devinivy got some work on it,got something working a little,need more test...
there is an other problem:

  {
    identity: 'Taxi',
    connection: 'foo',
    tableName: 'taxi_table',
    attributes: {
      "taxiId": {
        "type": "integer",
        "primaryKey": true
      },
      "taxiMatricule": {
        "type": "string"
      },
      "drivers": {
        "collection": "Driver",
        "through": "ride",
        "via": "taxi"
      }
    }
  }

  {
    identity: 'Ride',
    connection: 'foo',
    tableName: 'ride_table',
    attributes: {
      "rideId": {
        "type": "integer",
        "primaryKey": true
      },
      "taxi": {
        "model": "Taxi"
      },
      "new_driver": {
        "model": "Driver"
      },
      "last_driver": {
        "model": "Driver"
      },
    }
  }

like this this not possible to define if relation on taxi should use new_driver or last_driver on ride ...
maybe spec could had forward key to specify when this ambiguous like this :

      "drivers": {
        "collection": "Driver",
        "through": "ride",
        "via": "taxi",
        "forward": "new_driver"
      }

@particlebanana

@particlebanana merges patch to correct many bugs on this assocations type but not this one.
@particlebanana what do you think about adding optional forward key ?

@cuongcua90

Thanks for posting, @cuongcua90. I'm a repo bot-- nice to meet you!

It has been 30 days since there have been any updates or new comments on this page. If this issue has been resolved, feel free to disregard the rest of this message. On the other hand, if you are still waiting on a patch, please:

review our contribution guide to make sure this submission meets our criteria (only verified bugs with documented features, please; no questions, commentary, or bug reports about undocumented features or unofficial plugins)
create a new issue with the latest information, including updated version details with error messages, failing tests, and a link back to the original issue. This allows GitHub to automatically create a back-reference for future visitors arriving from search engines.

Thanks so much for your help!

Re-opening until this is added to the roadmap

@particlebanana

@particlebanana the real problem is we don't know what must be done... update doc? correct a bug ? add some tests,...just add a line in roadmap to say "something like this should work later"

Ok this is on the Roadmap. Waterline is taking the contribution guide from Sails. So basically what needs to happen is the following:

First, write out a high-level summary of the feature you are proposing as a concise description (3-5 sentences) focused around a convincing real-world use case where the app you are building or maintaining for your job, your clients, your company, your non-profit work, or your independent hobby project would be made easier by this feature or change.

Next, describe in clear prose with relevant links to code files exactly why it would be difficult or impossible to implement the feature without changing Waterline core. If this is not the case, and this feature could be implemented as an adapter, then please reconsider writing your proposal (it is unlikely the core team will be able to accept it).

Finally, if you have time, take a first pass at proposing a spec for this feature (its configuration, usage, and how it would be implemented). If you do not have time to write out a first draft of a thorough specification, please make that point in your feature request, and clarify that it would be up to other contributors with the same or a similar use case to finish this proposal.

This has a somewhat defined spec with the forward key and we are transitioning over to the new contribution guide for features so I just added it.

So TLDR: We have a loose spec of how to do M:M through with extra columns and it has been added to the roadmap. It could be scheduled for 0.11 or beyond depending on time constraints.

Navigation Menu

Repository navigation

When Many-to-Many Through Associations release? #5291

8 remaining items

Metadata

Assignees

Labels

Type

Projects

Milestone

Relationships

Development

When Many-to-Many Through Associations release? #5291

Description

Activity

devinivy commented on Oct 29, 2014

mewben commented on Oct 30, 2014

devinivy commented on Oct 30, 2014

mewben commented on Oct 31, 2014

devinivy commented on Oct 31, 2014

mewben commented on Oct 31, 2014

dmarcelino commented on Nov 1, 2014

NiclasLindqvist commented on Nov 21, 2014

devinivy commented on Nov 21, 2014

dmarcelino commented on Nov 21, 2014

kirkaleze commented on Mar 31, 2015

devinivy commented on Mar 31, 2015

dmarcelino commented on Apr 20, 2015

devinivy commented on Apr 20, 2015

elennaro commented on Apr 20, 2015

8 remaining items

sailsbot commented on Oct 23, 2015

devinivy commented on Oct 23, 2015

mukk85 commented on Nov 16, 2015

devinivy commented on Nov 16, 2015

atiertant commented on Nov 16, 2015

atiertant commented on Nov 25, 2015

sailsbot commented on Dec 26, 2015

particlebanana commented on Dec 26, 2015

atiertant commented on Dec 28, 2015

particlebanana commented on Jan 16, 2016

Metadata

Metadata

Assignees

Labels

Type

Projects

Milestone

Relationships

Development

Issue actions