bredikhin / barrels

Simple DB Fixtures for Sails.js
MIT License
85 stars 34 forks source link

1.1.1 breaks associations #10

Closed RWOverdijk closed 10 years ago

RWOverdijk commented 10 years ago

Hey,

No idea what happened here, but 1.1.1 breaks associations. Travis started complaining and I noticed that my associations contained value null. (Really simple oneToOne association, with model definition on both sides).

Sidenote, your versioning is wrong. You went from 1.0.1 to 1.1.1. Doesn't add up. :) Unless you combined a bugfix and functionality, in which case you've been a bad boy :p Resource: http://semver.org/

bredikhin commented 10 years ago

I really need some code to understand what's happening.

Sidenote, let's just say 1.0.0 was never released. ;) But in general you're kind of right, I'm pretty liberal with Semver usage.

RWOverdijk commented 10 years ago

The travis build is: https://travis-ci.org/Islive-io/fortress/jobs/38035586 but it doesn't say much to you.

These are the models, nothing special really:

bredikhin commented 10 years ago

Did you check out this part of readme: https://github.com/bredikhin/barrels#associations? ;)

RWOverdijk commented 10 years ago

Actually, I did not. And that's weird, I'll stick with 1.0.1 then. :)

bredikhin commented 10 years ago

FYI, automatic associations can be disabled now by passing false as second parameter to populate.