Don't call upstream if nothing found via friendly ID search

[corwinstephen]

Prevent false positives when friendly ID doesn't match

[parndt]

@corwinstephen thanks I added some suggestions; we can convert this from a guard clause to just use the result of exists_by_friendly_id? as that is what we're most interested in here.

[stale[bot]]

This issue has been automatically marked as stale because it has not had recent activity. It will be closed if no further activity occurs. Thank you for your contributions.

[corwinstephen]

Hey sorry to disappear on this. What are you thinking here? Looks like the specs had some trouble getting started.

[parndt]

@corwinstephen it looks to me like there are 16 test failures and they're probably related to us expecting friendly_id to call super