[Asterisk-code-review] res_geolocation: Add two new options to GEOLOC_PROFILE (asterisk[16])

George Joseph asteriskteam at digium.com
Wed Aug 31 08:03:25 CDT 2022


Hello Friendly Automation, 

I'd like you to reexamine a change. Please visit

    https://gerrit.asterisk.org/c/asterisk/+/18989

to look at the new patch set (#2).

Change subject: res_geolocation: Add two new options to GEOLOC_PROFILE
......................................................................

res_geolocation: Add two new options to GEOLOC_PROFILE

Added an 'a' option to the GEOLOC_PROFILE function to allow
variable lists like location_info_refinement to be appended
to instead of replacing the entire list.

Added an 'r' option to the GEOLOC_PROFILE function to resolve all
variables before a read operation and after a Set operation.

Added a few missing parameters to the ones allowed for writing
with GEOLOC_PROFILE.

Fixed a bug where calling GEOLOC_PROFILE to read a parameter
might actually update the profile object.

Cleaned up XML documentation a bit.

ASTERISK-30190

Change-Id: I75f541db43345509a2e86225bfa4cf8e242e5b6c
---
M doc/CHANGES-staging/res_geolocation.txt
M include/asterisk/res_geolocation.h
M res/res_geolocation/geoloc_dialplan.c
M res/res_geolocation/geoloc_doc.xml
M res/res_geolocation/geoloc_eprofile.c
M res/res_geolocation/geoloc_private.h
M res/res_pjsip_geolocation.c
7 files changed, 287 insertions(+), 37 deletions(-)


  git pull ssh://gerrit.asterisk.org:29418/asterisk refs/changes/89/18989/2
-- 
To view, visit https://gerrit.asterisk.org/c/asterisk/+/18989
To unsubscribe, or for help writing mail filters, visit https://gerrit.asterisk.org/settings

Gerrit-Project: asterisk
Gerrit-Branch: 16
Gerrit-Change-Id: I75f541db43345509a2e86225bfa4cf8e242e5b6c
Gerrit-Change-Number: 18989
Gerrit-PatchSet: 2
Gerrit-Owner: George Joseph <gjoseph at digium.com>
Gerrit-Reviewer: Friendly Automation
Gerrit-MessageType: newpatchset
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.digium.com/pipermail/asterisk-code-review/attachments/20220831/e1e91bc0/attachment.html>


More information about the asterisk-code-review mailing list