----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviewboard.asterisk.org/r/3650/ -----------------------------------------------------------
(Updated June 26, 2014, 4:12 p.m.) Review request for Asterisk Developers, Matt Jordan and Mark Michelson. Changes ------- Changes DialplanAddExtension to DialplanExtensionAdd and DialplanRemoveExtension to DialplanExtensionRemove so that they will be grouped together alphabetically. Repository: Asterisk Description ------- Adds 'DialplanAddExtension' and 'DialplanRemoveExtension' manager commands that work in mostly the same way as their CLI command equivalents. The following header arguments are used for each: Action: DialplanAddExtension Context - which context should be used Extension - name of the extension being created, if '/' is included, the portion after the '/' is a CID match for that extension. Priority - priority being added Application - name of the application to be used at this priority ApplicationData - not required (if not included results in no args), forms the arguments to the application Replace - not required (if not included, same as 'no'). If set to a truth value, replace existing extensions/priorities rather than failing if one exists where we are adding already. Action: DialplanRemoveExtension Context - which context is being removed from Extension - Which extension is being removed or having a priority removed from, if '/' is included, the portion after the '/' is a CID match for that extension. Priority - not required, if included then just a single priority is removed from the extension instead of the whole extension. A change to the pbx extension adding code was necessary in order for ast_add_extension to report an error when attempting to add an extension without replacing it when it already exists. The particular section in question previously had some developer comments questioning why the return values were what they were in the first place. I didn't observe any problematic behavior occuring as a result of the change, but it is in pbx.c, so I guess it could end up being a bit of a minefield. Diffs (updated) ----- /trunk/pbx/pbx_config.c 416234 /trunk/main/pbx.c 416234 /trunk/CHANGES 416234 Diff: https://reviewboard.asterisk.org/r/3650/diff/ Testing ------- Tested add extension with/without appdata Tested add extension with/without '/' in extension and made sure the rest of the field was used as a CID value and that it worked the same as the CLI command equivalent Tested remove extension with/without priority Tested remove extension with/without '/' in extension and made sure that if CID was included that it deleted the CID including extension. Thanks, Jonathan Rose
-- _____________________________________________________________________ -- Bandwidth and Colocation Provided by http://www.api-digital.com -- asterisk-dev mailing list To UNSUBSCRIBE or update options visit: http://lists.digium.com/mailman/listinfo/asterisk-dev