comparison sat/plugins/plugin_tickets_import_bugzilla.py @ 4037:524856bd7b19

massive refactoring to switch from camelCase to snake_case: historically, Libervia (SàT before) was using camelCase as allowed by PEP8 when using a pre-PEP8 code, to use the same coding style as in Twisted. However, snake_case is more readable and it's better to follow PEP8 best practices, so it has been decided to move on full snake_case. Because Libervia has a huge codebase, this ended with a ugly mix of camelCase and snake_case. To fix that, this patch does a big refactoring by renaming every function and method (including bridge) that are not coming from Twisted or Wokkel, to use fully snake_case. This is a massive change, and may result in some bugs.
author Goffi <goffi@goffi.org>
date Sat, 08 Apr 2023 13:54:42 +0200
parents be6d91572633
children
comparison
equal deleted inserted replaced
4036:c4464d7ae97b 4037:524856bd7b19
123 return (tickets, len(tickets)) 123 return (tickets, len(tickets))
124 124
125 125
126 class BugzillaImport(object): 126 class BugzillaImport(object):
127 def __init__(self, host): 127 def __init__(self, host):
128 log.info(_("Bugilla Import plugin initialization")) 128 log.info(_("Bugilla import plugin initialization"))
129 self.host = host 129 self.host = host
130 host.plugins["TICKETS_IMPORT"].register( 130 host.plugins["TICKETS_IMPORT"].register(
131 "bugzilla", self.Import, SHORT_DESC, LONG_DESC 131 "bugzilla", self.import_, SHORT_DESC, LONG_DESC
132 ) 132 )
133 133
134 def Import(self, client, location, options=None): 134 def import_(self, client, location, options=None):
135 if not os.path.isabs(location): 135 if not os.path.isabs(location):
136 raise exceptions.DataError( 136 raise exceptions.DataError(
137 "An absolute path to XML data need to be given as location" 137 "An absolute path to XML data need to be given as location"
138 ) 138 )
139 bugzilla_parser = BugzillaParser() 139 bugzilla_parser = BugzillaParser()