Project

General

Profile

Bug #6991

API v2 - support unwrapped POST/PUT parameters even though it's not documented

Added by Joseph Magen about 8 years ago. Updated over 6 years ago.

Status:
New
Priority:
Normal
Assignee:
-
Category:
API
Target version:
-
Difficulty:
Triaged:
No
Bugzilla link:
Pull request:
Fixed in Releases:
Found in Releases:

Description

add wrap_parameters for each controller to wrap setter methods that are not db attribute_names


Related issues

Related to Foreman - Bug #6696: API v2 - specify the key in which the parameters would be wrapped rather than passing model nameClosed2014-07-20
Has duplicate Foreman - Bug #8017: *_ids attributes have to be wrapped in an extra hashDuplicate2014-10-21
Has duplicate Foreman - Bug #19335: Adding media via the REST API does not correctly associate org/locationDuplicate2017-04-20

History

#1 Updated by The Foreman Bot about 8 years ago

  • Status changed from Assigned to Ready For Testing
  • Pull request https://github.com/theforeman/foreman/pull/1668 added
  • Pull request deleted ()

#2 Updated by Dominic Cleal about 8 years ago

  • Related to Bug #6696: API v2 - specify the key in which the parameters would be wrapped rather than passing model name added

#3 Updated by Dmitri Dolguikh almost 8 years ago

  • Target version changed from 1.7.5 to 1.7.4

#4 Updated by Dmitri Dolguikh almost 8 years ago

  • Target version changed from 1.7.4 to 1.7.3

#5 Updated by Dominic Cleal almost 8 years ago

  • Target version changed from 1.7.3 to 1.7.2

#6 Updated by Dominic Cleal almost 8 years ago

  • Has duplicate Bug #8017: *_ids attributes have to be wrapped in an extra hash added

#7 Updated by Dominic Cleal over 6 years ago

  • Status changed from Ready For Testing to New
  • Assignee deleted (Joseph Magen)
  • Target version deleted (1.7.2)
  • Pull request deleted (https://github.com/theforeman/foreman/pull/1668)

PR was closed. The manual does document unwrapped parameters, while apipie prefers wrapped. Both should work as it's currently documented, so I'm leaving this open.

#8 Updated by Dominic Cleal over 5 years ago

  • Has duplicate Bug #19335: Adding media via the REST API does not correctly associate org/location added

Also available in: Atom PDF