Mercurial > libervia-backend
comparison sat_frontends/jp/cmd_uri.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 | 4b842c1fb686 |
comparison
equal
deleted
inserted
replaced
4036:c4464d7ae97b | 4037:524856bd7b19 |
---|---|
41 self.parser.add_argument( | 41 self.parser.add_argument( |
42 "uri", help=_("XMPP URI to parse") | 42 "uri", help=_("XMPP URI to parse") |
43 ) | 43 ) |
44 | 44 |
45 async def start(self): | 45 async def start(self): |
46 await self.output(uri.parseXMPPUri(self.args.uri)) | 46 await self.output(uri.parse_xmpp_uri(self.args.uri)) |
47 self.host.quit() | 47 self.host.quit() |
48 | 48 |
49 | 49 |
50 class Build(base.CommandBase): | 50 class Build(base.CommandBase): |
51 def __init__(self, host): | 51 def __init__(self, host): |
66 help=_("URI fields"), | 66 help=_("URI fields"), |
67 ) | 67 ) |
68 | 68 |
69 async def start(self): | 69 async def start(self): |
70 fields = dict(self.args.fields) if self.args.fields else {} | 70 fields = dict(self.args.fields) if self.args.fields else {} |
71 self.disp(uri.buildXMPPUri(self.args.type, path=self.args.path, **fields)) | 71 self.disp(uri.build_xmpp_uri(self.args.type, path=self.args.path, **fields)) |
72 self.host.quit() | 72 self.host.quit() |
73 | 73 |
74 | 74 |
75 class Uri(base.CommandBase): | 75 class Uri(base.CommandBase): |
76 subcommands = (Parse, Build) | 76 subcommands = (Parse, Build) |