Mercurial > libervia-backend
comparison sat/core/patches.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 | 691283719bb2 |
| children |
comparison
equal
deleted
inserted
replaced
| 4036:c4464d7ae97b | 4037:524856bd7b19 |
|---|---|
| 69 # hooks at this level should not modify content | 69 # hooks at this level should not modify content |
| 70 # so it's not needed to handle priority as with triggers | 70 # so it's not needed to handle priority as with triggers |
| 71 self._onElementHooks = [] | 71 self._onElementHooks = [] |
| 72 self._sendHooks = [] | 72 self._sendHooks = [] |
| 73 | 73 |
| 74 def addHook(self, hook_type, callback): | 74 def add_hook(self, hook_type, callback): |
| 75 """Add a send or receive hook""" | 75 """Add a send or receive hook""" |
| 76 conflict_msg = f"Hook conflict: can't add {hook_type} hook {callback}" | 76 conflict_msg = f"Hook conflict: can't add {hook_type} hook {callback}" |
| 77 if hook_type == C.STREAM_HOOK_RECEIVE: | 77 if hook_type == C.STREAM_HOOK_RECEIVE: |
| 78 if callback not in self._onElementHooks: | 78 if callback not in self._onElementHooks: |
| 79 self._onElementHooks.append(callback) | 79 self._onElementHooks.append(callback) |