From 3387d557bc448eb60c67ca631451f62451c0c466 Mon Sep 17 00:00:00 2001
From: Brian Coca <bcoca@ansible.com>
Date: Wed, 1 Jun 2016 09:17:10 -0400
Subject: [PATCH] Revert "Add prefixing and suffixing fuctionality to assemble
 module" (#16084)

---
 lib/ansible/plugins/action/assemble.py         | 18 ++----------------
 .../roles/test_assemble/tasks/main.yml         | 10 ----------
 2 files changed, 2 insertions(+), 26 deletions(-)

diff --git a/lib/ansible/plugins/action/assemble.py b/lib/ansible/plugins/action/assemble.py
index 69f32656e3..5cdfc7cbfb 100644
--- a/lib/ansible/plugins/action/assemble.py
+++ b/lib/ansible/plugins/action/assemble.py
@@ -32,7 +32,7 @@ class ActionModule(ActionBase):
 
     TRANSFERS_FILES = True
 
-    def _assemble_from_fragments(self, src_path, delimiter=None, compiled_regexp=None, ignore_hidden=False, header=None, footer=None):
+    def _assemble_from_fragments(self, src_path, delimiter=None, compiled_regexp=None, ignore_hidden=False):
         ''' assemble a file from a directory of fragments '''
 
         tmpfd, temp_path = tempfile.mkstemp()
@@ -40,11 +40,6 @@ class ActionModule(ActionBase):
         delimit_me = False
         add_newline = False
 
-        if header is not None:
-            if not header.endswith('\n'):
-                header += '\n'
-            tmp.write(header)
-
         for f in sorted(os.listdir(src_path)):
             if compiled_regexp and not compiled_regexp.search(f):
                 continue
@@ -75,13 +70,6 @@ class ActionModule(ActionBase):
             else:
                 add_newline = True
 
-        if footer is not None:
-            if add_newline:  # last fragment did not end with \n
-                footer = '\n' + footer
-            if not footer.endswith('\n'):
-                footer += '\n'
-            tmp.write(footer)
-
         tmp.close()
         return temp_path
 
@@ -99,8 +87,6 @@ class ActionModule(ActionBase):
         src        = self._task.args.get('src', None)
         dest       = self._task.args.get('dest', None)
         delimiter  = self._task.args.get('delimiter', None)
-        header     = self._task.args.get('header', None)
-        footer     = self._task.args.get('footer', None)
         remote_src = self._task.args.get('remote_src', 'yes')
         regexp     = self._task.args.get('regexp', None)
         follow     = self._task.args.get('follow', False)
@@ -135,7 +121,7 @@ class ActionModule(ActionBase):
             return result
 
         # Does all work assembling the file
-        path = self._assemble_from_fragments(src, delimiter, _re, ignore_hidden, header, footer)
+        path = self._assemble_from_fragments(src, delimiter, _re, ignore_hidden)
 
         path_checksum = checksum_s(path)
         dest = self._remote_expand_user(dest)
diff --git a/test/integration/roles/test_assemble/tasks/main.yml b/test/integration/roles/test_assemble/tasks/main.yml
index 9699b10508..d0c1f15e56 100644
--- a/test/integration/roles/test_assemble/tasks/main.yml
+++ b/test/integration/roles/test_assemble/tasks/main.yml
@@ -91,13 +91,3 @@
     - "result.state == 'file'"
     - "result.checksum == '505359f48c65b3904127cf62b912991d4da7ed6d'"
 
-- name: test assemble with a header and a footer
-  assemble: src="{{output_dir}}/src" dest="{{output_dir}}/assembled6" header="prefix" footer="suffix"
-  register: result
-
-- name: assert the fragments were assembled with a header and a footer
-  assert:
-    that:
-    - "result.state == 'file'"
-    - "result.checksum == 'c5cca6452da7b193427ea17583188f7a17df3b60'"
-