Mercurial Repository: p/roundup/code: roundup/admin.py comparison

comparison roundup/admin.py @ 6959:3211745e8d7c

flake8 fixes.

author	John Rouillard <rouilj@ieee.org>
date	Mon, 12 Sep 2022 21:52:52 -0400
parents	f924af12ef50
children	d3d20e145cea

comparison

equal deleted inserted replaced

-:e54a2db40a9e
+:3211745e8d7c
 """
 from __future__ import print_function
 __docformat__ = 'restructuredtext'
-import csv, getopt, getpass, operator, os, re, shutil, sys
+import csv
+import getopt
+import getpass
+import operator
+import os
+import re
+import shutil
+import sys
 from roundup import date, hyperdb, init, password, token
 from roundup import __version__ as roundup_version
 import roundup.instance
 from roundup.configuration import (CoreConfig, NoConfigError,
 h = _(command.__doc__).split('\n')[0]
 commands.append(' '+h[7:])
 commands.sort()
 commands.append(_(
 """Commands may be abbreviated as long as the abbreviation
-matches only one command, e.g. l == li == lis == list."""))
+matches only one command, e.g. l == li == lis == list."""))  # noqa: E122
 sys.stdout.write('\n'.join(commands) + '\n\n')
-def help_commands_html(self, indent_re=re.compile(r'^(\s+)\S+')):
+indent_re = re.compile(r'^(\s+)\S+')
+def help_commands_html(self, indent_re=indent_re):
 """ Produce an HTML command list.
 """
 commands = sorted(iter(self.commands.values()),
 key=operator.attrgetter('__name__'))
 for command in commands:
 print("""
 <tr><td valign=top><strong>%(name)s</strong></td>
 <td><tt>%(usage)s</tt><p>
 <pre>""" % locals())
 indent = indent_re.match(h[3])
-if indent: indent = len(indent.group(1))
+if indent: indent = len(indent.group(1))  # noqa: E701
 for line in h[3:]:
 if indent:
 print(line[indent:])
 else:
 print(line)
 """))
 for name, command in list(self.commands.items()):
 print(_('%s:') % name)
 print('   ', _(command.__doc__))
-def do_help(self, args, nl_re=re.compile('[\r\n]'),
+nl_re = re.compile('[\r\n]')
-indent_re=re.compile(r'^(\s+)\S+')):
+# indent_re defined above
+def do_help(self, args, nl_re=nl_re, indent_re=indent_re):
 ''"""Usage: help topic
 Give help about topic.
 commands  -- list commands
 <command> -- help specific to a command
 # display the help for each match, removing the docstring indent
 for _name, help in cmd_docs:
 lines = nl_re.split(_(help.__doc__))
 print(lines[0])
 indent = indent_re.match(lines[1])
-if indent: indent = len(indent.group(1))
+if indent: indent = len(indent.group(1))  # noqa: E701
 for line in lines[1:]:
 if indent:
 print(line[indent:])
 else:
 print(line)
 #    (2 dirs up)
 #
 # we're interested in where the directory containing "share" is
 debug = False
 templates = {}
-if debug: print(__file__)
+if debug: print(__file__)  # noqa: E701
 for N in 2, 4, 5, 6:
 path = __file__
 # move up N elements in the path
 for _i in range(N):
 path = os.path.dirname(path)
 tdir = os.path.join(path, 'share', 'roundup', 'templates')
-if debug or trace_search: print(tdir)
+if debug or trace_search: print(tdir)  # noqa: E701
 if os.path.isdir(tdir):
 templates = init.listTemplates(tdir)
-if debug: print("  Found templates breaking loop")
+if debug: print(" Found templates breaking loop")  # noqa: E701
 break
 # search for data files parallel to the roundup
 # install dir. E.G. a wheel install
 #  use roundup.__path__ and go up a level then use sys.prefix
 for _N in 1, 2:
 path = os.path.dirname(path)
 # path is /usr/local/lib/python3.10/site-packages
 tdir = os.path.join(path, sys.prefix[1:], 'share',
 'roundup', 'templates')
-if debug or trace_search: print(tdir)
+if debug or trace_search: print(tdir)  # noqa: E701
 if os.path.isdir(tdir):
 templates.update(init.listTemplates(tdir))
 try:
 # sigh pip 3.10 in virtual env finds another place to bury them.
 # why local and sys.base_prefix are in path I do not know.
 # path is /usr/local/lib/python3.10/site-packages
 tdir = os.path.join(path, sys.base_prefix[1:], 'local', 'share',
 'roundup', 'templates')
-if debug or trace_search: print(tdir)
+if debug or trace_search: print(tdir)  # noqa: E701
 if os.path.isdir(tdir):
 templates.update(init.listTemplates(tdir))
 # path is /usr/local/lib/python3.10/site-packages
 tdir = os.path.join(path, sys.base_prefix[1:], 'share',
 'roundup', 'templates')
-if debug or trace_search: print(tdir)
+if debug or trace_search: print(tdir)  # noqa: E701
 if os.path.isdir(tdir):
 templates.update(init.listTemplates(tdir))
 except AttributeError:
 pass  # sys.base_prefix doesn't work under python2
 # Try subdirs of the current dir
 templates.update(init.listTemplates(os.getcwd()))
-if debug or trace_search: print(os.getcwd() + '/*')
+if debug or trace_search: print(os.getcwd() + '/*')  # noqa: E701
 # Finally, try the current directory as a template
 template = init.loadTemplateInfo(os.getcwd())
-if debug or trace_search: print(os.getcwd())
+if debug or trace_search: print(os.getcwd())  # noqa: E701
 if template:
-if debug: print("  Found template %s"%template['name'])
+if debug: print("  Found template %s" %   # noqa: E701
+template['name'])
 templates[template['name']] = template
 return templates
 def help_initopts(self):
 os.path.join(tracker_home, 'config.py')])):
 if not self.force:
 ok = my_input(_(
 """WARNING: There appears to be a tracker in "%(tracker_home)s"!
 If you re-install it, you will lose all the data!
-Erase it? Y/N: """) % locals())
+Erase it? Y/N: """) % locals())  # noqa: E122
 if ok.strip().lower() != 'y':
 return 0
 # clear it out so the install isn't confused
 shutil.rmtree(tracker_home)
 ... see the documentation on customizing for more information.
 You MUST run the "roundup-admin initialise" command once you've performed
 the above steps.
 ---------------------------------------------------------------------------
-""") % {
+""") % {'database_config_file': os.path.join(tracker_home, 'schema.py'),
-'database_config_file': os.path.join(tracker_home, 'schema.py'),
+'database_init_file': os.path.join(tracker_home, 'initial_data.py')}) \
-'database_init_file': os.path.join(tracker_home, 'initial_data.py'),
+# noqa: E122
-})
 return 0
 def _get_choice(self, list_name, prompt, options, argument, default=None):
 if default is None:
 default = options[0]  # just pick the first one
 if tracker.exists():
 if not self.force:
 ok = my_input(_(
 """WARNING: The database is already initialised!
 If you re-initialise it, you will lose all the data!
-Erase it? Y/N: """))
+Erase it? Y/N: """))  # noqa: E122
 if ok.strip().lower() != 'y':
 return 0
 # nuke it
 tracker.nuke()
 # print
 properties = cl.getprops()
 property = properties[propname]
 if not (isinstance(property, hyperdb.Multilink) or
 isinstance(property, hyperdb.Link)):
-raise UsageError(_('property %s is not of type'
+raise UsageError(_(
+'property %s is not of type'
 ' Multilink or Link so -d flag does not '
 'apply.') % propname)
 propclassname = self.db.getclass(property.classname).classname
 id = cl.get(nodeid, propname)
 for i in id:
 if self.print_designator:
 properties = cl.getprops()
 property = properties[propname]
 if not (isinstance(property, hyperdb.Multilink) or
 isinstance(property, hyperdb.Link)):
-raise UsageError(_('property %s is not of type'
+raise UsageError(_(
+'property %s is not of type'
 ' Multilink or Link so -d flag does not '
 'apply.') % propname)
 propclassname = self.db.getclass(property.classname).classname
 id = cl.get(nodeid, propname)
 for i in id:
 lastprop = pn  # get current component
 # get classname for this link
 try:
 curclassname = curclass.getprops()[pn].classname
 except KeyError:
-raise UsageError(_("Class %(curclassname)s has "
+raise UsageError(_(
-"no property %(pn)s in %(propname)s." % locals()))
+"Class %(curclassname)s has "
+"no property %(pn)s in %(propname)s." %
+locals()))
 # get class object
 curclass = self.get_class(curclassname)
 except AttributeError:
 # curclass.getprops()[pn].classname raises this
 # when we are at a non link/multilink property
 props = {}
 properties = cl.getprops(protected=0)
 if len(args) == 1:
 # ask for the properties
 for key in properties:
-if key == 'id': continue
+if key == 'id': continue  # noqa: E701
 value = properties[key]
 name = value.__class__.__name__
 if isinstance(value, hyperdb.Password):
 again = None
 while value != again:
 %
 {'propname': key.capitalize()})
 again = getpass.getpass(_('   %(propname)s (Again): ')
 %
 {'propname': key.capitalize()})
-if value != again: print(_('Sorry, try again...'))
+if value != again:
+print(_('Sorry, try again...'))
 if value:
 props[key] = value
 else:
 value = my_input(_('%(propname)s (%(proptype)s): ') % {
 'propname': key.capitalize(), 'proptype': name})
 # convert types
 for propname in props:
 try:
 props[propname] = hyperdb.rawToHyperdb(self.db, cl, None,
-propname, props[propname])
+propname,
+props[propname])
 except hyperdb.HyperdbValueError as message:
 raise UsageError(message)
 # check for the key property
 propname = cl.getkey()
 templates = self.listTemplates(trace_search=trace_search)
 for name in sorted(list(templates.keys())):
 templates[name]['description'] = textwrap.fill(
-"\n".join([ line.lstrip() for line in
+"\n".join([line.lstrip() for line in
 templates[name]['description'].split("\n")]),
 70,
 subsequent_indent="      "
 )
 print("""
 Name: %(name)s
 Path: %(path)s
 Desc: %(description)s
-"""%templates[name])
+""" % templates[name])
 def do_table(self, args):
 ''"""Usage: table classname [property[,property]*]
 List the instances of a class in tabular form.
 # on imports to rdbms.
 all_nodes = cl.getnodeids()
 classkey = cl.getkey()
 if classkey:  # False sorts before True, so negate is_retired
-keysort = lambda i: (cl.get(i, classkey),
+keysort = lambda i: (cl.get(i, classkey),  # noqa: E731
 not cl.is_retired(i))
 all_nodes.sort(key=keysort)
 # if there is no classkey no need to sort
 for nodeid in all_nodes:
 pack_before = date.Date(value)
 self.db.pack(pack_before)
 self.db_uncommitted = True
 return 0
-def do_reindex(self, args, desre=re.compile('([A-Za-z]+)([0-9]+)')):
+designator_re = re.compile('([A-Za-z]+)([0-9]+)')
+def do_reindex(self, args, desre=designator_re):
 ''"""Usage: reindex [classname|designator]*
 Re-generate a tracker's search indexes.
 This will re-generate the search indexes for a tracker.
 This will typically happen automatically.
 sys.stdout.write(_('Role "%(name)s":\n') % role.__dict__)
 for permission in role.permissions:
 d = permission.__dict__
 if permission.klass:
 if permission.properties:
-sys.stdout.write(_(' %(description)s (%(name)s for "%(klass)s"' +
+sys.stdout.write(_(
-': %(properties)s only)\n') % d)
+' %(description)s (%(name)s for "%(klass)s"' +
+': %(properties)s only)\n') % d)
 # verify that properties exist; report bad props
 bad_props = []
 cl = self.db.getclass(permission.klass)
 class_props = cl.getprops(protected=True)
 for p in permission.properties:
 if p in class_props:
 continue
 else:
 bad_props.append(p)
 if bad_props:
-sys.stdout.write(_('\n  **Invalid properties for %(class)s: %(props)s\n\n') % {"class": permission.klass, "props": bad_props})
+sys.stdout.write(_(
+'\n  **Invalid properties for %(class)s: '
+'%(props)s\n\n') % {
+"class": permission.klass,
+"props": bad_props})
 return 1
 else:
 sys.stdout.write(_(' %(description)s (%(name)s for '
 '"%(klass)s" only)\n') % d)
 else:
 using anydbm).
 It's safe to run this even if it's not required, so just get
 into the habit.
 """
-if getattr(self.db, 'db_version_updated'):
+if self.db.db_version_updated:
 print(_('Tracker updated'))
 self.db_uncommitted = True
 else:
 print(_('No migration action required'))
 return 0
 return 1
 except NoConfigError as message:  # noqa: F841
 self.tracker_home = ''
 print(_("Error: Couldn't open tracker: %(message)s") % locals())
 return 1
-except ParsingOptionError as message: # message used via locals
+# message used via locals
+except ParsingOptionError as message:  # noqa: F841
 print("%(message)s" % locals())
 return 1
 # only open the database once!
 if not self.db:
 self.db = tracker.open(self.name)
 # dont use tracker.config["TRACKER_LANGUAGE"] here as the
 # cli operator likely wants to have i18n as set in the
 # environment.
 # This is needed to fetch the locale's of the tracker's home dir.
-self.db.i18n = get_translation (tracker_home = tracker.tracker_home)
+self.db.i18n = get_translation(tracker_home=tracker.tracker_home)
 self.db.tx_Source = 'cli'
 # do the command
 ret = 0
 try:
 command = my_input(_('roundup> '))
 except EOFError:
 print(_('exit...'))
 break
-if not command: continue
+if not command: continue  # noqa: E701
 try:
 args = token.token_split(command)
 except ValueError:
 continue        # Ignore invalid quoted token
-if not args: continue
+if not args: continue  # noqa: E701
-if args[0] in ('quit', 'exit'): break
+if args[0] in ('quit', 'exit'): break   # noqa: E701
 self.run_command(args)
 # exit.. check for transactions
 if self.db and self.db_uncommitted:
 commit = my_input(_('There are unsaved changes. Commit them (y/N)? '))
 try:
 if not args:
 self.interactive()
 else:
 ret = self.run_command(args)
-if self.db: self.db.commit()
+if self.db: self.db.commit()  # noqa: E701
 return ret
 finally:
 if self.db:
 self.db.close()

Mercurial > p > roundup > code

comparison roundup/admin.py @ 6959:3211745e8d7c