Dynamic Agents in Queues not working if a comma is in the user displayname

Dynamic Agents in Queues not working if a comma is in the user displayname.

In the context "[macro-toggle-add-agent]" in extensions_additional.conf there is a bug that causes users containing a comma in the display name to be incorrectly added to queues.

Here is an incorrect entry and a correct one (config fixt):
 

LastName (Local/555@from-queue/n from FirstName,hint:555@ext-local) (ringinuse disabled) (dynamic) (Invalid) has taken no calls yet
  
LastName, FirstName (Local/555@from-queue/n from hint:555@ext-local) (ringinuse enabled) (dynamic) (Not in use) has taken no calls yet

In these two lines (5326 & 5327) in extensions_additional.conf the ${QUEUEUSERCIDNAME} must be put in quotation marks at the end ("${QUEUEUSERCIDNAME}")
 

exten => s,n,ExecIf($["${DB(AMPUSER/${QUEUEUSER}/queues/qnostate)}" != "ignorestate"]?AddQueueMember(${QUEUENO},Local/${QUEUEUSER}@from-queue/n,${DB(QPENALTY/${QUEUENO}/agents/${QUEUEUSER})},,"${QUEUEUSERCIDNAME}",hint:${QUEUEUSER}@ext-local))}}
exten => s,n,ExecIf($["${DB(AMPUSER/${QUEUEUSER}/queues/qnostate)}" = "ignorestate"]?AddQueueMember(${QUEUENO},Local/${QUEUEUSER}@from-queue/n,${DB(QPENALTY/${QUEUENO}/agents/${QUEUEUSER})},,"${QUEUEUSERCIDNAME}"))

After this customization, adding Dynamic Queues agents works correctly. Of course, each time the configuration is changed via the GUI, this adjustment is overwritten.

A temporary workaround is to add the corrected [macro-toggle-add-agent] context to extensions_override_freepbx.conf.

The bug is not new, as someone has already come across this problem by 2015.

https://community.freepbx.org/t/difficult-issue-queue-members-invalid-tearing-my-little-bit-of-hair-out/32313/6