Project

General

Profile

Bug #13245

Apipie json returns duplicated methods

Added by David Caro almost 4 years ago. Updated over 1 year ago.

Status:
Closed
Priority:
Normal
Assignee:
Category:
API
Target version:
Difficulty:
trivial
Triaged:
Bugzilla link:
Fixed in Releases:
Found in Releases:

Description

When retrieving the apipie json file to get the api definition, it returns a duplicated method (exactly the same method for the same resource with the same params overt the same http method):

For the full json file see https://gist.githubusercontent.com/fatmcgav/018a26acdf0aeb99b655/raw/e3f92d8c600a27893662336f6f1bdc5d3e9283c3/1.9.2-v2.json

The extract: {
"short_description": "List storage_domains for a compute resource",
"deprecated": null,
"http_method": "GET",
"api_url": "/api/compute_resources/:id/available_storage_domains"
}, {
"short_description": "List storage domains for a compute resource",
"deprecated": null,
"http_method": "GET",
"api_url": "/api/compute_resources/:id/available_storage_domains"
},

It's not a big issue, but probably an indicator that something fishy is going on.

1.9.2-v2.json 1.9.2-v2.json 957 KB David Caro, 01/15/2016 10:55 AM

Related issues

Related to Foreman - Feature #6756: expose vmware resource pools and folders through APIClosed2014-07-23

Associated revisions

Revision adc07e24 (diff)
Added by Shlomi Zadok almost 4 years ago

Fixes #13245 - Remove duplicate apipie description from compute_resources

History

#1 Updated by Dominic Cleal almost 4 years ago

  • Related to Feature #6756: expose vmware resource pools and folders through API added

#2 Updated by Dominic Cleal almost 4 years ago

  • Category set to API
  • Difficulty set to trivial

There are two "api :GET" lines for exactly the same path in app/controllers/api/v2/compute_resources_controller.rb. The first (without N_()) should be deleted.

#3 Updated by The Foreman Bot almost 4 years ago

  • Status changed from New to Ready For Testing
  • Assignee set to Shlomi Zadok
  • Pull request https://github.com/theforeman/foreman/pull/3162 added

#4 Updated by Shlomi Zadok almost 4 years ago

  • Status changed from Ready For Testing to Closed
  • % Done changed from 0 to 100

#5 Updated by Dominic Cleal almost 4 years ago

  • Legacy Backlogs Release (now unused) set to 71

Also available in: Atom PDF