Use plistlib.load() instead of plistlib.readPlist()

Author: homebysix

CHANGELOG.md

@@ -3,6 +3,12 @@
 All notable changes to this project will be documented in this file. This project adheres to [Semantic Versioning](http://semver.org/).
 
 
+## [1.8.0] - 2020-10-08
+
+### Changed
+- Replaced `plistlib.readPlist()` with `plistlib.load()`
+
+
 ## [1.7.0] - 2020-10-06
 
 ### Added

pre_commit_hooks/check_autopkg_recipe_list.py

@@ -7,9 +7,10 @@
 """
 
 import argparse
+import json
 import plistlib
 from xml.parsers.expat import ExpatError
-import json
+
 import ruamel.yaml
 
 yaml = ruamel.yaml.YAML(typ="safe")
@@ -44,7 +45,8 @@ def main(argv=None):
                 ]
         elif filename.endswith(".plist"):
             try:
-                recipe_list = plistlib.readPlist(filename)
+                with open(filename, "rb") as openfile:
+                    recipe_list = plistlib.load(openfile)
             except (ExpatError, ValueError) as err:
                 print("{}: plist parsing error: {}".format(filename, err))
                 retval = 1

pre_commit_hooks/check_autopkg_recipes.py

@@ -383,7 +383,8 @@ def main(argv=None):
     retval = 0
     for filename in args.filenames:
         try:
-            recipe = plistlib.readPlist(filename)
+            with open(filename, "rb") as openfile:
+                recipe = plistlib.load(openfile)
 
         except (ExpatError, ValueError) as err:
             print("{}: plist parsing error: {}".format(filename, err))

pre_commit_hooks/check_jamf_profiles.py

@@ -27,7 +27,8 @@ def main(argv=None):
     retval = 0
     for filename in args.filenames:
         try:
-            profile = plistlib.readPlist(filename)
+            with open(filename, "rb") as openfile:
+                profile = plistlib.load(openfile)
         except (ExpatError, ValueError) as err:
             print("{}: plist parsing error: {}".format(filename, err))
             retval = 1

pre_commit_hooks/check_munki_pkgsinfo.py

@@ -49,7 +49,8 @@ def main(argv=None):
     retval = 0
     for filename in args.filenames:
         try:
-            pkginfo = plistlib.readPlist(filename)
+            with open(filename, "rb") as openfile:
+                pkginfo = plistlib.load(openfile)
         except (ExpatError, ValueError) as err:
             print("{}: plist parsing error: {}".format(filename, err))
             retval = 1
@@ -180,7 +181,7 @@ def main(argv=None):
             if script_type in pkginfo:
                 if all(not pkginfo[script_type].startswith(x + "\n") for x in shebangs):
                     print(
-                        "{}: has a {} that does not start with a valid shebang.".format(
+                        "{}: Has a {} that does not start with a valid shebang.".format(
                             filename, script_type
                         )
                     )

pre_commit_hooks/check_munkipkg_buildinfo.py

@@ -81,7 +81,8 @@ def main(argv=None):
     for filename in args.filenames:
         if filename.endswith(".plist"):
             try:
-                buildinfo = plistlib.readPlist(filename)
+                with open(filename, "rb") as openfile:
+                    buildinfo = plistlib.load(openfile)
             except (ExpatError, ValueError) as err:
                 print("{}: plist parsing error: {}".format(filename, err))
                 retval = 1

pre_commit_hooks/check_plists.py

@@ -27,7 +27,8 @@ def main(argv=None):
     retval = 0
     for filename in args.filenames:
         try:
-            plist = plistlib.readPlist(filename)
+            with open(filename, "rb") as openfile:
+                plist = plistlib.load(openfile)
             # Possible future addition, but disabled for now.
             # if not isinstance(plist, dict):
             #     print("{}: top level of plist should be type dict".format(filename))

pre_commit_hooks/forbid_autopkg_overrides.py

@@ -30,7 +30,8 @@ def main(argv=None):
     retval = 0
     for filename in args.filenames:
         try:
-            recipe = plistlib.readPlist(filename)
+            with open(filename, "rb") as openfile:
+                recipe = plistlib.load(openfile)
             for req_key in required_keys:
                 if req_key not in recipe:
                     print("{}: possible AutoPkg recipe override".format(filename))

pre_commit_hooks/forbid_autopkg_trust_info.py

@@ -28,7 +28,8 @@ def main(argv=None):
     retval = 0
     for filename in args.filenames:
         try:
-            recipe = plistlib.readPlist(filename)
+            with open(filename, "rb") as openfile:
+                recipe = plistlib.load(openfile)
             if "ParentRecipeTrustInfo" in recipe:
                 print("{}: trust info in recipe".format(filename))
                 retval = 1