Clarify syntax of new manager events that reports a status.
authorOlle Johansson <oej@edvina.net>
Tue, 4 Apr 2006 07:59:02 +0000 (07:59 +0000)
committerOlle Johansson <oej@edvina.net>
Tue, 4 Apr 2006 07:59:02 +0000 (07:59 +0000)
We do not longer approve "ThisEventOn" and "ThisEventOff" named events,
they need to be named "ThisEvent" with a status header showing the
current status.
(Approved by Mark)
Old events won't be changed to keep backwards compatibility,
until we have a revision plan for the AMI.

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

doc/CODING-GUIDELINES
doc/manager.txt

index c8afc70..a19fd0a 100644 (file)
@@ -522,6 +522,19 @@ int len = 0;
 if (name && (len = strlen(name) + strlen(prefix) + strlen(postfix) + 3) && (newname = alloca(len)))
        snprintf(newname, len, "%s/%s/%s", prefix, name, postfix);
 
+* Creating new manager events?
+------------------------------
+If you create new AMI events, please read manager.txt. Do not re-use
+existing headers for new purposes, but please re-use existing headers
+for the same type of data.
+
+Manager events that signal a status are required to have one
+event name, with a status header that shows the status.
+The old style, with one event named "ThisEventOn" and another named
+"ThisEventOff", is no longer approved.
+
+Check manager.txt for more information on manager and existing
+headers. Please update this file if you add new headers.
 
 -----------------------------------------------
 Welcome to the Asterisk development community!
index 065d70a..cee0c47 100644 (file)
@@ -296,3 +296,4 @@ Some standard AMI headers:
 
  ** Please try to re-use existing headers to simplify manager message parsing in clients.
 
+Read the CODING-GUIDELINES if you develop new manager commands or events.