From 5e47aa2ddb58abb82bbc8e6e378e8848b0a7ead5 Mon Sep 17 00:00:00 2001
From: Felix Fontein <felix@fontein.de>
Date: Fri, 30 Oct 2020 15:51:54 +0100
Subject: [PATCH] ansible-doc: export has_action when --json is used (#72359)

* ansible-doc: export has_action when --json is used.
* Remove docuri and now_data, which were not used resp. ignored in format_plugin_doc and the functions it calls anyway.
* Add function _combine_plugin_doc.

(cherry picked from commit 4fb336cef13e4b6e2b46a4d30e60d1d40cbbbd90)
---
 .../fragments/ansible-doc-has_action.yml      |  2 ++
 lib/ansible/cli/doc.py                        | 28 +++++++++----------
 2 files changed, 16 insertions(+), 14 deletions(-)
 create mode 100644 changelogs/fragments/ansible-doc-has_action.yml

diff --git a/changelogs/fragments/ansible-doc-has_action.yml b/changelogs/fragments/ansible-doc-has_action.yml
new file mode 100644
index 00000000000000..f1ab97eb79d4b0
--- /dev/null
+++ b/changelogs/fragments/ansible-doc-has_action.yml
@@ -0,0 +1,2 @@
+minor_changes:
+- "ansible-doc - provide ``has_action`` field in JSON output for modules. That information is currently only available in the text view (https://github.com/ansible/ansible/pull/72359)."
diff --git a/lib/ansible/cli/doc.py b/lib/ansible/cli/doc.py
index 0004c61295055e..72b6e7c4094f2e 100644
--- a/lib/ansible/cli/doc.py
+++ b/lib/ansible/cli/doc.py
@@ -209,8 +209,7 @@ def run(self):
                     # The doc section existed but was empty
                     continue
 
-                plugin_docs[plugin] = {'doc': doc, 'examples': plainexamples,
-                                       'return': returndocs, 'metadata': metadata}
+                plugin_docs[plugin] = DocCLI._combine_plugin_doc(plugin, plugin_type, doc, plainexamples, returndocs, metadata)
 
             if do_json:
                 # Some changes to how json docs are formatted
@@ -323,23 +322,24 @@ def _get_plugin_doc(plugin, loader, search_paths):
         return doc, plainexamples, returndocs, metadata
 
     @staticmethod
-    def format_plugin_doc(plugin, plugin_type, doc, plainexamples, returndocs, metadata):
-        # assign from other sections
-        doc['plainexamples'] = plainexamples
-        doc['returndocs'] = returndocs
-        doc['metadata'] = metadata
-
+    def _combine_plugin_doc(plugin, plugin_type, doc, plainexamples, returndocs, metadata):
         # generate extra data
         if plugin_type == 'module':
             # is there corresponding action plugin?
             if plugin in action_loader:
-                doc['action'] = True
+                doc['has_action'] = True
             else:
-                doc['action'] = False
+                doc['has_action'] = False
+
+        # return everything as one dictionary
+        return {'doc': doc, 'examples': plainexamples, 'return': returndocs, 'metadata': metadata}
 
-        doc['now_date'] = datetime.date.today().strftime('%Y-%m-%d')
-        if 'docuri' in doc:
-            doc['docuri'] = doc[plugin_type].replace('_', '-')
+    @staticmethod
+    def format_plugin_doc(plugin, plugin_type, doc, plainexamples, returndocs, metadata):
+        # assign from other sections
+        doc['plainexamples'] = plainexamples
+        doc['returndocs'] = returndocs
+        doc['metadata'] = metadata
 
         if context.CLIARGS['show_snippet'] and plugin_type == 'module':
             text = DocCLI.get_snippet_text(doc)
@@ -631,7 +631,7 @@ def get_man_text(doc, plugin_type=None):
         except Exception:
             pass  # FIXME: not suported by plugins
 
-        if doc.pop('action', False):
+        if doc.pop('has_action', False):
             text.append("  * note: %s\n" % "This module has a corresponding action plugin.")
 
         if 'options' in doc and doc['options']: