diff options
author | David M. Lee <dlee@digium.com> | 2013-11-07 21:10:31 +0000 |
---|---|---|
committer | David M. Lee <dlee@digium.com> | 2013-11-07 21:10:31 +0000 |
commit | 7d0d1a1efb1d484cce28137f1abc1a6ece61d7e9 (patch) | |
tree | a61c7891a423ff7a70ae5bb3a5880ba4437702fd /res/ari/resource_endpoints.h | |
parent | cdfbc02df164a9bc49e23900fae66e696c4623aa (diff) |
ari: User better nicknames for ARI operations
While working on building client libraries from the Swagger API, I
noticed a problem with the nicknames.
channel.deleteChannel()
channel.answerChannel()
channel.muteChannel()
Etc. We put the object name in the nickname (since we were generating C
code), but it makes OO generators redundant.
This patch makes the nicknames more OO friendly. This resulted in a lot
of name changing within the res_ari_*.so modules, but not much else.
There were a couple of other fixed I made in the process.
* When reversible operations (POST /hold, POST /unhold) were made more
RESTful (POST /hold, DELETE /unhold), the path for the second operation
was left in the API declaration. This worked, but really the two
operations should have been on the same API.
* The POST /unmute operation had still not been REST-ified.
Review: https://reviewboard.asterisk.org/r/2940/
........
Merged revisions 402528 from http://svn.asterisk.org/svn/asterisk/branches/12
git-svn-id: https://origsvn.digium.com/svn/asterisk/trunk@402529 65c4cc65-6c06-0410-ace0-fbb531ad65f3
Diffstat (limited to 'res/ari/resource_endpoints.h')
-rw-r--r-- | res/ari/resource_endpoints.h | 18 |
1 files changed, 9 insertions, 9 deletions
diff --git a/res/ari/resource_endpoints.h b/res/ari/resource_endpoints.h index 5d7aade68..1454a73c9 100644 --- a/res/ari/resource_endpoints.h +++ b/res/ari/resource_endpoints.h @@ -39,8 +39,8 @@ #include "asterisk/ari.h" -/*! \brief Argument struct for ast_ari_get_endpoints() */ -struct ast_get_endpoints_args { +/*! \brief Argument struct for ast_ari_endpoints_list() */ +struct ast_ari_endpoints_list_args { }; /*! * \brief List all endpoints. @@ -49,9 +49,9 @@ struct ast_get_endpoints_args { * \param args Swagger parameters * \param[out] response HTTP response */ -void ast_ari_get_endpoints(struct ast_variable *headers, struct ast_get_endpoints_args *args, struct ast_ari_response *response); -/*! \brief Argument struct for ast_ari_get_endpoints_by_tech() */ -struct ast_get_endpoints_by_tech_args { +void ast_ari_endpoints_list(struct ast_variable *headers, struct ast_ari_endpoints_list_args *args, struct ast_ari_response *response); +/*! \brief Argument struct for ast_ari_endpoints_list_by_tech() */ +struct ast_ari_endpoints_list_by_tech_args { /*! \brief Technology of the endpoints (sip,iax2,...) */ const char *tech; }; @@ -62,9 +62,9 @@ struct ast_get_endpoints_by_tech_args { * \param args Swagger parameters * \param[out] response HTTP response */ -void ast_ari_get_endpoints_by_tech(struct ast_variable *headers, struct ast_get_endpoints_by_tech_args *args, struct ast_ari_response *response); -/*! \brief Argument struct for ast_ari_get_endpoint() */ -struct ast_get_endpoint_args { +void ast_ari_endpoints_list_by_tech(struct ast_variable *headers, struct ast_ari_endpoints_list_by_tech_args *args, struct ast_ari_response *response); +/*! \brief Argument struct for ast_ari_endpoints_get() */ +struct ast_ari_endpoints_get_args { /*! \brief Technology of the endpoint */ const char *tech; /*! \brief ID of the endpoint */ @@ -77,6 +77,6 @@ struct ast_get_endpoint_args { * \param args Swagger parameters * \param[out] response HTTP response */ -void ast_ari_get_endpoint(struct ast_variable *headers, struct ast_get_endpoint_args *args, struct ast_ari_response *response); +void ast_ari_endpoints_get(struct ast_variable *headers, struct ast_ari_endpoints_get_args *args, struct ast_ari_response *response); #endif /* _ASTERISK_RESOURCE_ENDPOINTS_H */ |