has many through about openrecord HOT 4 OPEN

Hi @nibarulabs

You'll need to define the target relation. e.g.: this.hasMany('roles', {model: 'Role', through: 'roles_users', relation: 'roles'});.
There is currently no default to use the relation name as the target relation name.

btw:

const role = await Role.find(1);
const user = await User.create({name: 'John'});
user.roles.add(role);
await user.save() // <--- don't forget!

from openrecord.

Hi @PhilWaldmann, thanks for your reply.

Hmm, still getting the same error. I actually played with adding relation last night and was seeing the same behavior.

I've noticed something else too that might be fine, but wanted to double check. I added some console log to print out the role after I look it up and I noticed it adds the has many attributes to the class even though I'm using the through.

role: Role {
  relations: {},
  attributes: {
    id: 1,
    name: 'member',
    created_at: 2020-08-05T22:08:52.630Z,
    updated_at: 2020-08-05T22:08:52.629Z,
    creator_id: 1,
    updater_id: 1,
    rolesUserIds: null,
    userIds: null
  }
...

I have all my models introspecting the data structure from the table and my postgres table definition for roles looks pretty standard:

                                       Table "public.roles"
   Column   |           Type           | Collation | Nullable |              Default              
------------+--------------------------+-----------+----------+-----------------------------------
 id         | bigint                   |           | not null | nextval('roles_id_seq'::regclass)
 name       | character varying(255)   |           |          | 
 created_at | timestamp with time zone |           |          | CURRENT_TIMESTAMP
 updated_at | timestamp with time zone |           |          | CURRENT_TIMESTAMP
 creator_id | bigint                   |           |          | 
 updater_id | bigint                   |           |          | 
...

There are no rolesUserIds or userIds arrays on that table. I haven't reached that part of the OR code, but I'm assuming hasMany is adding it and not removing it when a through is used? Does it matter?

I'm not trying to confuse the issue, but I suspect that having those attributes there and the relations object is empty - maybe there's some issue there?

Feels like I'm close and I'm just missing something small.. I'm going to go through your test code to see how you're testing things for through to see if I can spot something there.

from openrecord.

Ok, I found a workaround.

In my user.js I added the following function:

async addRole(role) {
    const RoleUser = require('./role-user');
    const sql = `INSERT INTO roles_users (user_id,role_id) values (${this.id},${role.id})`;
    console.log("sql:", sql)
    const result = await RoleUser.raw(sql);
    await this.roles;
}

This now allows me to do something like:

const role = await Role.find(1);
const user = await User.create({name: 'John'});
user.addRole(role);

I get that it's not ideal and not really 'correct', but it gets the association working for me, albeit with more db hits (I don't need to over optimize at this point and can fix later).

If there are any suggestions as to what else I can try to get the user.roles.add(role) syntax working, then I can try that when it becomes available. For now, this allows me to keep moving forward.

from openrecord.

Yeah, that's not the prettiest solution.
I'll try to rebuild your situation in a test case to see where the problem is. I hope I've time to do so in the next few days.

from openrecord.