Repository navigation
hasOne ignores mapsTo #638
Description
I have a model defined with mapsTo for each property (table is snake_case while model is camelCase). If I use hasOne to associate it to another model, the values of association fields are always null. I have tried the following alternatives:
- Defining association field name as it appears in the model. This leads to an exception due to ORM trying to insert a row with camelCase field.
- Doing help in development #1 but also changing the code in
Utilities.jsfrom:
new_obj[obj].mapsTo = obj;
to
new_obj[obj].mapsTo = new_obj[obj].mapsTo || obj;
This 2nd approach works in my tests but it broke other unit tests.
Before I continue I would like to know if I'm on the right track here. Are associations defined per model property name (camelCase in my case) or per table column name (snake_case in my case)?
Activity
dxg commentedon May 12, 2015
Not sure I understand the question, but maybe neither?
I don't think I ever tested
mapsTowith associations. The functions inutilities.jsare really confusing to work with.Add 'mapsTo' option when defining hasOne #638
dxg commentedon May 12, 2015
I've made some changes to master. They won't necessarily fix your problem, but they include tests.
Can you please run your code against master, and either add a failing test case, give me a code sample, or tell me how to make it fail?
ierceg commentedon May 12, 2015
@dxg Seems to be fixed - I tried two different tests and both passed. Thanks for this!
ierceg commentedon May 12, 2015
@dxg I misspoke - one test is still failing, I just didn't have it correctly configured. I'll add the failing test case.
ierceg commentedon May 13, 2015
@dxg I rechecked, again, and it works alright. It's the mapped name that has to be specified in the
hasOne(I had column name and then it reverts to the behavior of using NULL for the value of the relationship field)ierceg commentedon May 13, 2015
@dxg When do you expect to bump the package version?
dxg commentedon May 14, 2015
Just published 2.1.25
ierceg commentedon May 14, 2015
@dxg Thanks!