app_queue.c: Fix error in Queue parameter documentation.

When macro was removed in Asterisk 21, the parameter documentation in
code was not updated to reflect the correct numerization for gosub. It
still stated that it was the seventh parameter, but got shifted to the
sixth due to the removal of macro. This has been updated to correctly
reflect the parameter order, and a note has been added to the XML that
states this was done after the initial commit.

Fixes: #1534
UpgradeNote: As part of Asterisk 21, macros were removed from Asterisk.
This resulted in argument order changing for the Queue dialplan
application since the macro argument was removed. Upgrade notice was
missed when this was done, so this upgrade note has been added to
provide a record of such and a notice to users who may have not upgraded
yet.

diff --git a/apps/app_queue.c b/apps/app_queue.c
index ffb10f2bb252955e7d392f466ef6b555b0652b66..954fe64bf02115b3454ec0fac05ff0ec1f93d6c9 100644 (file)
--- a/apps/app_queue.c
+++ b/apps/app_queue.c
@@ -259,6 +259,10 @@
                                <para>Will run a gosub on the called party's channel (the queue member)
                                once the parties are connected.  The subroutine execution starts in the
                                named context at the s exten and priority 1.</para>
+                               <note><para>Macro was removed in Asterisk 21 which resulted in an
+                               argument ordering change. The upgrade notice was missed for this,
+                               so a note is being made here to provide a record of the change
+                               for users who have not upgraded yet.</para></note>
                        </parameter>
                        <parameter name="rule">
                                <para>Will cause the queue's defaultrule to be overridden by the rule specified.</para>
@@ -7131,7 +7135,7 @@ static void setup_mixmonitor(struct queue_ent *qe, const char *filename)
  * \param[in,out] tries the number of times we have tried calling queue members
  * \param[out] noption set if the call to Queue() has the 'n' option set.
  * \param[in] agi the agi passed as the fifth parameter to the Queue() application
- * \param[in] gosub the gosub passed as the seventh parameter to the Queue() application
+ * \param[in] gosub the gosub passed as the sixth parameter to the Queue() application
  * \param[in] ringing 1 if the 'r' option is set, otherwise 0
  */
 static int try_calling(struct queue_ent *qe, struct ast_flags opts, char **opt_args, char *announceoverride, const char *url, int *tries, int *noption, const char *agi, const char *gosub, int ringing)