I came across this issue when trying to pass a string in the path to a resource's relationship and received a 404, but found that using an integer in the path worked as expected. I dug around and found an implementation of is_numeric() that I believe can be removed.

Comments

theneonlobster’s picture

This patch resolves the issue. I did not test to verify that the relationship argument's type value is being respected, but it wasn't before.

marcingy’s picture

Status: Active » Needs work

The patch submitted removes an isset check which makes no sense given the bug reported.

theneonlobster’s picture

I discovered the bug in 7.x-3.3 and patched 7.x-3.x without verifying that the bug persisted. It appears to have been fixed, however for people using 7.x-3.3, the attached patch will rectify the issue.

theneonlobster’s picture

Status: Needs work » Needs review

Updating status to "needs review"

marcingy’s picture

Status: Needs review » Closed (fixed)

This is fixed in dev.

halth’s picture

For a sake of indexing, this issue was giving me this error: 404 Not found: Could not find the controller.

Hope it can help someone out there!

halth’s picture

Issue summary: View changes

removed a sentence that lacks context