Mercurial > libervia-backend
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() |