fix error in migrate groups (freeze for large console output)

This commit is contained in:
Mario Voigt 2020-09-13 00:26:22 +02:00
parent dd8d0c57a7
commit 3b283abb1c

View File

@ -8,11 +8,12 @@ This extension parses the selection and will put all elements into one single gr
Author: Mario Voigt / FabLab Chemnitz
Mail: mario.voigt@stadtfabrikanten.org
Date: 13.08.2020
Last Patch: 31.08.2020
Last Patch: 13.09.2020
License: GNU GPL v3
"""
import inkex
import os
from lxml import etree
class MigrateGroups(inkex.Effect):
@ -158,13 +159,34 @@ class MigrateGroups(inkex.Effect):
#inkex.utils.debug(len(self.allDrops))
#inkex.utils.debug(self.allDrops)
migrate_log = "migrategroups.log"
# Clean up possibly previously generated log file
if os.path.exists(migrate_log):
try:
os.remove(migrate_log)
except OSError as e:
inkex.utils.debug("Error while deleting previously generated log file " + migrate_log)
# show a list with items to delete. For ungroup mode it does not apply because we are not going to remove anything
if self.options.operationmode == "filter_only" or self.options.operationmode == "ungroup_and_filter":
if self.options.showdroplist:
self.msg(str(len(self.allDrops)) + " elements were removed during nodes while migration:")
inkex.utils.debug(str(len(self.allDrops)) + " elements were removed during nodes while migration:")
if len(self.allDrops) > 100: #if we print too much to the output stream we will freeze InkScape forever wihtout any visual error message. So we write to file instead
migrate_log_file = open('migrategroups.log', 'w')
else:
migrate_log_file = None
for i in self.allDrops:
if i.get('id') is not None:
self.msg(i.tag.replace("{http://www.w3.org/2000/svg}","svg:") + " id:" + i.get('id'))
migrateString = i.tag.replace("{http://www.w3.org/2000/svg}","svg:") + " id:" + i.get('id')
if migrate_log_file is None:
inkex.utils.debug(migrateString)
else:
migrate_log_file.write(migrateString + "\n")
if migrate_log_file is not None:
migrate_log_file.close()
inkex.utils.debug("Detailed output was dumped into file " + os.path.join(os.getcwd(), migrate_log))
# remove all groups from the selection and form a new single group of it by copying with old IDs.
if self.options.operationmode == "ungroup_only" or self.options.operationmode == "ungroup_and_filter":