[asterisk-dev] [Code Review] 4383: res_pjsip_config_wizard: Add ability to auto-create hints.

[George Joseph]

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

(Updated Feb. 4, 2015, 10:31 a.m.)


Review request for Asterisk Developers.


Changes
-------

Updated for Josh's comments...
Check for null application.
Don't disturb existing exact-match entries.


Repository: Asterisk


Description
-------

Looking at the Super Awesome Company sample reminded me that creating hints is just plain gruntwork.  So you can now have the pjsip conifg wizard auto-create them for you.

Specifying 'hint_exten' in the wizard will create 'exten => <hint_exten>,hint/PJSIP/<wizard_id>' in whatever is specified for 'hint_context'.
Specifying 'hint_application' in the wizard will create 'exten => <hint_exten>,1,<hint_application>' in whatever is specified for 'hint_context'.

The default for 'hint_context' is the endpoint's context.
There's no default for 'hint_application'.  If not specified, no app is added.
There's no default for 'hint_exten'.  If not specified, neither the hint itself nor the application will be created.

Some may think this is the slippery slope to users.conf but hints are a basic necessity for phones unlike voicemail, manager, etc that users.conf creates.


Diffs (updated)
-----

  branches/13/res/res_pjsip_config_wizard.c 431571 
  branches/13/configs/samples/pjsip_wizard.conf.sample 431571 

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


Testing
-------

Existing config_wizard testsuite tests pass.
Additional testsuite tests in the works.


Thanks,

George Joseph

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.digium.com/pipermail/asterisk-dev/attachments/20150204/5af52175/attachment.html>