Refactor restore jobIds code

Changed constant to more descriptive name,
changed multi line text to use implicit joins,
fixed arg type.

cuegui/cuegui/plugins/MonitorJobsPlugin.py

@@ -48,7 +48,7 @@
 PLUGIN_PROVIDES = "MonitorJobsDockWidget"
 REGEX_EMPTY_STRING = re.compile("^$")
 TIME_DELTA = 3
-JOB_LOAD_LIMIT = 200
+JOB_RESTORE_THRESHOLD_DAYS = 200
 
 class MonitorJobsDockWidget(cuegui.AbstractDockWidget.AbstractDockWidget):
     """Plugin for listing active jobs and managing them."""
@@ -114,16 +114,14 @@ def restoreJobIds(self, jobIds):
 
         :param jobIds: monitored jobs ids and their timestamp from previous working state
                        (loaded from config.ini file)
-        :ptype: list of tuples ex:
-                [("Job.f156be87-987a-48b9-b9da-774cd58674a3", 1612482716.170947),...
+                       ex: [("Job.f156be87-987a-48b9-b9da-774cd58674a3", 1612482716.170947),...
+        :type: list[tuples]
         """
         today = datetime.datetime.now()
-        limit = JOB_LOAD_LIMIT if len(jobIds) > JOB_LOAD_LIMIT else len(jobIds)
-        msg = """
-              Unable to load previously loaded job since
-              it was moved to the historical
-              database:
-              """
+        limit = JOB_RESTORE_THRESHOLD_DAYS if len(jobIds) > \
+                                                JOB_RESTORE_THRESHOLD_DAYS else len(jobIds)
+        msg = ('Unable to load previously loaded job since it was moved '
+                   'to the historical database: {0}')
 
         try:
             for jobId, timestamp in jobIds[:limit]:
@@ -132,7 +130,7 @@ def restoreJobIds(self, jobIds):
                     try:
                         self.jobMonitor.addJob(jobId, timestamp)
                     except opencue.EntityNotFoundException:
-                        logger.info(msg, jobId)
+                        logger.info(msg.format(jobId))
         except ValueError:
             # load older format
             for jobId in jobIds[:limit]: