Mercurial > libervia-backend

diff sat/plugins/plugin_comp_ap_gateway/ad_hoc.py @ 4037:524856bd7b19

Find changesets by keywords (author, files, the commit message), revision number or hash, or revset expression.
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 0aa7023dcd08
children
line wrap: on
line diff
--- a/sat/plugins/plugin_comp_ap_gateway/ad_hoc.py	Fri Apr 07 15:18:39 2023 +0200
+++ b/sat/plugins/plugin_comp_ap_gateway/ad_hoc.py	Sat Apr 08 13:54:42 2023 +0200
@@ -38,7 +38,7 @@
         self._c = self.host.plugins["XEP-0050"]
 
     def init(self, client: SatXMPPEntity) -> None:
-        self._c.addAdHocCommand(
+        self._c.add_ad_hoc_command(
             client,
             self.xmpp_jid_node_2_ap_actor,
             "Convert XMPP JID/Node to AP actor",
@@ -82,7 +82,7 @@
         else:
             xmpp_jid = jid.JID(command_form["jid"])
             xmpp_node = command_form.get("node")
-            actor = await self.apg.getAPAccountFromJidAndNode(xmpp_jid, xmpp_node)
+            actor = await self.apg.get_ap_account_from_jid_and_node(xmpp_jid, xmpp_node)
             note = (self._c.NOTE.INFO, actor)
             status = self._c.STATUS.COMPLETED
             payload = None