build_tools: Skip managerEvent combining for AMI action responses
authorMatthew Jordan <mjordan@digium.com>
Thu, 7 Aug 2014 03:04:49 +0000 (03:04 +0000)
committerMatthew Jordan <mjordan@digium.com>
Thu, 7 Aug 2014 03:04:49 +0000 (03:04 +0000)
AMI action responses can (and will) reference AMI events that they return.
These event references and definitions should not be combined with AMI events
raised elsewhere in the code, as they are specifically tied to the AMI action
that raised them.

ASTERISK-24156 #close
Reported by: Rusty Newton

git-svn-id: https://origsvn.digium.com/svn/asterisk/trunk@420289 65c4cc65-6c06-0410-ace0-fbb531ad65f3

build_tools/post_process_documentation.py

index 678328d..5fa1c5b 100644 (file)
@@ -59,7 +59,10 @@ def collapse_event_pair(managerEventOne, managerEventTwo):
 def collapse_manager_events(rootNode, managerEvents):
     events = {}
     for managerEvent in managerEvents:
-        rootNode.removeChild(managerEvent)
+        if (managerEvent.parentNode.nodeName == 'list-elements'
+            or managerEvent.parentNode.nodeName == 'responses'):
+            continue
+        managerEvent.parentNode.removeChild(managerEvent)
         attr = managerEvent.getAttribute('name')
         if attr in events:
             # match, collapse the two managerEvents