🔨 Use simple open in copy engine

Closes #265

.moban.cd/changelog.yml

@@ -2,9 +2,9 @@ name: moban
 organisation: moremoban
 releases:
 - changes:
-  - action: N/A
+  - action: Updated
     details:
-    - "Fill your description"
+    - "`#265`: Use simple `read binary` to read instead of encoding"
   date: unreleased
   version: 0.4.4
 - changes:

CHANGELOG.rst

@@ -4,10 +4,11 @@ Change log
 0.4.4 - unreleased
 --------------------------------------------------------------------------------
 
-N/A
+Updated
 ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
 
-#. Fill your description
+#. `#265 <https://github.com/moremoban/moban/issues/265>`_: Use simple `read
+   binary` to read instead of encoding
 
 0.4.3 - 16.03.2019
 --------------------------------------------------------------------------------

moban/copy/__init__.py

@@ -1,6 +1,5 @@
 from lml.plugin import PluginInfo
 
-import codecs
 
 from moban import utils
 from moban import constants
@@ -30,7 +29,7 @@ def get_template(self, template_file):
         real_file_name = utils.find_file_in_template_dirs(
             template_file, self.template_dirs
         )
-        with codecs.open(real_file_name, encoding="utf-8") as file_handle:
+        with open(real_file_name, "rb") as file_handle:
             return file_handle.read()
 
     def get_template_from_string(self, string):

moban/utils.py

@@ -61,6 +61,8 @@ def file_permissions(afile):
 
 
 def strip_off_trailing_new_lines(content):
+    if isinstance(content, bytes):
+        content = content.decode("utf-8")
     return re.sub(r"(\n\s+)+$", r"\n", content)