fix(panos.updater.SoftwareUpdater.install): Fix for load_config parameter (#447)

FoSix · web-flow · commit 3e2e926199f2 · 2022-06-01T09:50:47.000-07:00
* fix on usage of load_config parameter when updating a device

* fix formatting
diff --git a/panos/updater.py b/panos/updater.py
@@ -120,11 +120,8 @@ def install(self, version, load_config=None, sync=False):
             "Device %s installing version: %s" % (self.pandevice.id, version)
         )
         response = self._op(
-            'request system software install%s version "%s"'
-            % (
-                " load-config " + load_config if load_config is not None else "",
-                version,
-            )
+            'request system software install %s version "%s"'
+            % ('load-config "{0}"'.format(load_config) if load_config else "", version,)
         )
         if sync:
             result = self.pandevice.syncjob(response)

Original file line number	Diff line number	Diff line change
`@@ -120,11 +120,8 @@ def install(self, version, load_config=None, sync=False):`
`120`	`120`	`"Device %s installing version: %s" % (self.pandevice.id, version)`
`121`	`121`	`)`
`122`	`122`	`response = self._op(`
`123`		`- 'request system software install%s version "%s"'`
`124`		`- % (`
`125`		`- " load-config " + load_config if load_config is not None else "",`
`126`		`- version,`
`127`		`- )`
	`123`	`+ 'request system software install %s version "%s"'`
	`124`	`+ % ('load-config "{0}"'.format(load_config) if load_config else "", version,)`
`128`	`125`	`)`
`129`	`126`	`if sync:`
`130`	`127`	`result = self.pandevice.syncjob(response)`