[asterisk-dev] [Code Review] 2698: ARI - implement allowMultiple for parameters

svnbot reviewboard at asterisk.org
Fri Aug 2 09:36:39 CDT 2013


-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviewboard.asterisk.org/r/2698/
-----------------------------------------------------------

(Updated Aug. 2, 2013, 9:36 a.m.)


Status
------

This change has been marked as submitted.


Review request for Asterisk Developers and jrose.


Changes
-------

Committed in revision 396122


Repository: Asterisk


Description
-------

Swagger allows parameters to be specified as 'allowMultiple', meaning
that the parameter may be specified as a comma separated list of
values.

I had written some of the API docs using that, but promptly forgot
about implementing it. This patch finally fills in that gap.

The codegen template was updated to represent 'allowMultiple' fields
as array/size fields in the _args structs. It also parses the comma
separated list using ast_app_separate_args(), so quoted strings in the
argument will be handled properly.


Diffs
-----

  /trunk/res/ari/resource_asterisk.h 395729 
  /trunk/res/ari/resource_bridges.h 395729 
  /trunk/res/ari/resource_bridges.c 395729 
  /trunk/res/ari/resource_events.h 395729 
  /trunk/res/ari/resource_events.c 395729 
  /trunk/res/ari/resource_sounds.h 395729 
  /trunk/res/res_ari_asterisk.c 395729 
  /trunk/res/res_ari_bridges.c 395729 
  /trunk/res/res_ari_channels.c 395729 
  /trunk/res/res_ari_endpoints.c 395729 
  /trunk/res/res_ari_events.c 395729 
  /trunk/res/res_ari_playback.c 395729 
  /trunk/res/res_ari_recordings.c 395729 
  /trunk/res/res_ari_sounds.c 395729 
  /trunk/rest-api-templates/ari_resource.h.mustache 395729 
  /trunk/rest-api-templates/param_cleanup.mustache PRE-CREATION 
  /trunk/rest-api-templates/param_parsing.mustache 395729 
  /trunk/rest-api-templates/res_ari_resource.c.mustache 395729 
  /trunk/rest-api/api-docs/bridges.json 395729 
  /trunk/rest-api/api-docs/channels.json 395729 
  /trunk/rest-api/api-docs/events.json 395729 
  /trunk/rest-api/api-docs/sounds.json 395729 

Diff: https://reviewboard.asterisk.org/r/2698/diff/


Testing
-------

Lots of variations hitting the events websocket and adding channels to
bridges using comma separated lists.


Thanks,

David Lee

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.digium.com/pipermail/asterisk-dev/attachments/20130802/b5a01478/attachment-0001.htm>


More information about the asterisk-dev mailing list