Failed withRelated three tables

[samsaga2]

I don't know why Bookshelf is fetching only the first relation.

"use strict";

var Bookshelf = require('bookshelf');

var db = Bookshelf.initialize({
  debug: true,
  client: 'postgres',
  connection: {
    host: '169.254.1.4',
    port: 5432,
    user: 'postgres',
    password: '',
    database: 'canela'
  }
});

var Compta = db.Model.extend({
  tableName: 'comptes'
});

var Client = db.Model.extend({
  tableName: 'fitxes_clients',
  idAttribute: 'id_fitxa',
  compta: function() {
    return this.belongsTo(Compta, 'id_compta');
  }
});

var Fitxa = db.Model.extend({
  tableName: 'fitxes',
  client: function() {
    return this.belongsTo(Client, 'id_fitxa');
  }
});

new Fitxa().fetch({
  withRelated: ['client','client.compta']
})
.then(function(data) {
  console.log(data);
});

But Bookshelf only makes two queries. The client.compta it's never fetched.

{ sql: 'select "fitxes".* from "fitxes" limit 1', bindings: [], __cid: '__cid2' } { sql: 'select "fitxes_clients".* from "fitxes_clients" where "id_fitxa" in (?)', bindings: [ undefined ], __cid: '__cid3' }

And yes, idAttribute on table client really sucks.

[tgriesser]

It looks like it's not retrieving anything from the "client" fetch - bindings: [ undefined ] therefore it wouldn't get to the compta portion of the query... does there need to be an idAttribute set for the Fixta table?

Could you give me an shortened example schema (table keys / values) in the db so I can take a look at reproducing?

[samsaga2]

I was writing a better example and you're right. The bindings: [ undefined ] means my keys are wrong.

But my real problem it's that Bookshelf doesn't support composited keys. Something like:

var Table1 = db.Model.extend({
  tableName: 'table1',
  idAttribute: ['key1', 'key2']
});

var Table2 = db.Model.extend({
  tableName: 'table2',
  test: function() {
    return this.belongsTo(Table1, ['key1_id', 'key2_id']);
  }
});

Thanks for your help.

[tgriesser]

Yeah, I'd like to be able to support composite keys at some point, let me know if you have any ideas on how these could be easily added.

[tgriesser]

Moving to #86.