Skip to content
This repository has been archived by the owner on May 23, 2024. It is now read-only.

Action URI checking isn't quite right #139

Closed
erunion opened this issue Dec 13, 2017 · 0 comments
Closed

Action URI checking isn't quite right #139

erunion opened this issue Dec 13, 2017 · 0 comments
Assignees
@erunion
Labels
bug
Milestone
v4.0: OAI

Comments

@erunion
Copy link
Owner

erunion commented Dec 13, 2017 

     * @api-uri:private:alias {Movies} /movie/+id
     * @api-uriSegment {/movie/+id} id (integer) - Movie ID
     *
     * @api-uri:public {Movies} /movies/+id
     * @api-uriSegment {/movies/+id} id (integer) - Movie ID

With this order, /movie/+id is picked up as the primary URI through Action\Documentation->getUri() eventhough it's actually an alias. getUri should eliminate aliases from contention.
@erunion erunion added this to the v3.0 milestone Dec 13, 2017
@erunion erunion self-assigned this Dec 13, 2017
@erunion erunion removed this from the v3.0 milestone Feb 1, 2018
@erunion erunion added this to the v4.0: OAI milestone Mar 24, 2018
erunion pushed a commit that referenced this issue May 29, 2018
Fixing a bug where an aliased path might be chosen as the primary.
6d0deb1 
#139
This was referenced May 29, 2018
Merged
Merged
@erunion erunion closed this as completed May 29, 2018
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Assignees

@erunion erunion

Labels
bug
Projects
None yet
Milestone
v4.0: OAI
Development

No branches or pull requests
1 participant
@erunion