Associations: use class association path instead of association name

[nicolas-besnard]

In the following case:

class RemoteBankAccount
  collection_path 'bankaccounts'
end

class RemoteUser
  has_many :bank_accounts
end

I could be nice to use collection_path instead of the association name.

If you are ok, I can do the PR :)

[edtjones]

Hiya. Can't you do that by specifying the class? has_many :bank_accounts, classname: 'RemoteBankAccount' Unless I have misunderstood? Ed

…

-- Sent from my phone: please forgive the brevity. Ed Jones

On 26 Dec 2016, at 10:54, Nicolas Besnard ***@***.***> wrote: In the following case: class RemoteBankAccount collection_path 'bankaccounts' end class RemoteUser has_many :bank_accounts end I could be nice to use collection_path instead of the association name. If you are ok, I can do the PR :) — You are receiving this because you are subscribed to this thread. Reply to this email directly, view it on GitHub, or mute the thread.

[nicolas-besnard]

When I specify the class, it doesn't use the collection_path (or resource_path) defined on the associated class.

As you can see on the here, the association name is used to find the path

[edtjones]

Hi @nicolas-besnard OK I understand. Makes sense to parameterise that I guess. A PR would be welcome :)

[nicolas-besnard]

I think changing this will break people code. We should use an options for this.

What do you thing ? Should it be the default behavior ?

[edtjones]

I think it should be parameterised / optional @nicolas-besnard . You want to take a swing at a PR and we'll see how we go? Thanks loads for being involved :-)

[zacharywelch]

+1 on using collection_path as a default for has_many. Association paths in general could use some ❤️ to get them in sync with Relation

In the meantime here's a workaround:

class RemoteBankAccount
  collection_path 'bankaccounts'
end

class RemoteUser
  has_many :bank_accounts, path: "/bankaccounts"
end