Introspecting a model with a custom path throws an exception about her HOT 10 CLOSED

Does you User model has a company_id attribute?

from her.

No it doesn't, it is just basic information, e.g.:

{
  "id": 54,
  "name": "Luca"
}

In my actual use case (as opposed to this example), the child can belong to multiple parents, so it doesn't make sense to have a single parent_id attribute. Maybe a better example would be Groups which have Users, and Users can belong to multiple Groups.

from her.

Ah I see. Her is indeed calling the request path for each User resource.

In your actual use case, what URL would be called if you call the #save method on a user? PUT /users/:id (because the URL can’t depend on the parent since a user can belong to many parents)? In that case, you’d have to use the resource_path method:

class User
  collection_path "/companies/:company_id/users"
  resource_path "/users/:id"
end

from her.

Sorry for the delay, we've been busy busy recently...

Back to the issue, so the actual model in our case is read-only, so we don't need to worry about saving it. However if we did the the example you provided would be correct.

I think for now we'll just add some custom stuff to stop the #inspect method from doing this, in our case it's pretty simple to know what the URL is, so I don't think we need it there.

from her.

I think we could just catch any exceptions when calling request_path here.

class User
  collection_path "/companies/:company_id/users"
end

user1 = User.find(1) # { "id": 1, "name": "Tobias Fünke", "company_id": 2 }
user1.inspect # => #<User(/companies/2/users/1) name="Tobias Fünke">

user2 = User.find(2) # { "id": 2, "name": "Maeby Fünke" }
user2.inspect # => #<User(<unknown path>) name="Maeby Fünke">

That makes sense, right?

from her.

That sounds good to me :)

from her.

This commit should fix it. If I take my previous example again:

class User
  collection_path "/companies/:company_id/users"
end

user1 = User.find(1) # { "id": 1, "name": "Tobias Fünke", "company_id": 2 }
user1.inspect # => #<User(/companies/2/users/1) name="Tobias Fünke">

user2 = User.find(2) # { "id": 2, "name": "Maeby Fünke" }
user2.inspect # => #<User(<unknown path, missing `company_id`>) name="Maeby Fünke">

from her.

Great! It's been annoying for a long time ;)

@lucaspiller Can you verify and close the issue?

from her.

@lucaspiller You should not be experiencing the issue with newer versions of her (0.6+). Please upgrade, test again and close this issue 😄 Thanks!

from her.

Works perfect, thanks!

from her.