[Asterisk-code-review] manager: Update ModuleCheck documentation. (asterisk[20])

Friendly Automation asteriskteam at digium.com
Tue Nov 8 08:16:55 CST 2022


Friendly Automation has submitted this change. ( https://gerrit.asterisk.org/c/asterisk/+/19542 )

Change subject: manager: Update ModuleCheck documentation.
......................................................................

manager: Update ModuleCheck documentation.

The ModuleCheck XML documentation falsely
claims that the module's version number is returned.
This has not been the case since 14, since the version
number is not available anymore, but the documentation
was not changed at the time. It is now updated to
reflect this.

ASTERISK-30285 #close

Change-Id: Idde2d1205a11f2623fa1ddab192faa3dc4081e91
---
M main/manager.c
1 file changed, 20 insertions(+), 1 deletion(-)

Approvals:
  George Joseph: Looks good to me, approved
  Friendly Automation: Approved for Submit




diff --git a/main/manager.c b/main/manager.c
index 90333a0..496e5c1 100644
--- a/main/manager.c
+++ b/main/manager.c
@@ -1109,7 +1109,8 @@
 		</syntax>
 		<description>
 			<para>Checks if Asterisk module is loaded. Will return Success/Failure.
-			For success returns, the module revision number is included.</para>
+			An empty Version header is also returned (which doesn't contain
+			the module revision number).</para>
 		</description>
 		<see-also>
 			<ref type="manager">ModuleLoad</ref>

-- 
To view, visit https://gerrit.asterisk.org/c/asterisk/+/19542
To unsubscribe, or for help writing mail filters, visit https://gerrit.asterisk.org/settings

Gerrit-Project: asterisk
Gerrit-Branch: 20
Gerrit-Change-Id: Idde2d1205a11f2623fa1ddab192faa3dc4081e91
Gerrit-Change-Number: 19542
Gerrit-PatchSet: 2
Gerrit-Owner: N A <asterisk at phreaknet.org>
Gerrit-Reviewer: Friendly Automation
Gerrit-Reviewer: George Joseph <gjoseph at digium.com>
Gerrit-MessageType: merged
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.digium.com/pipermail/asterisk-code-review/attachments/20221108/14998a3f/attachment-0001.html>


More information about the asterisk-code-review mailing list