Dev: Refactor: Introduce custom sort order enable/disable functions

Added `enable_custom_sort_order` and `disable_custom_sort_order`
functions to encapsulate the logic for modifying `rl_sort_completion_matches`.

crmsh/command.py

@@ -535,10 +535,7 @@ def complete(self, context, args):
         '''
         ret = []
         if self.completer is not None:
-            if sort_completion_inst is not None and sort_completion_inst.value == 0:
-                # Restore the original value of rl_sort_completion_matches
-                # before calling the completer again
-                sort_completion_inst.value = orig_sort_completion_value
+            disable_custom_sort_order()
             specs = inspect.getfullargspec(self.completer)
             if 'context' in specs.args:
                 ret = self.completer([self.name] + args, context)
@@ -566,6 +563,18 @@ def _check_args(fn, expected):
                          ": Expected method with signature " + repr(expected))
 
 
+def enable_custom_sort_order():
+    if sort_completion_inst is not None:
+        sort_completion_inst.value = 0
+
+
+def disable_custom_sort_order():
+    if sort_completion_inst is not None and sort_completion_inst.value == 0:
+        # Restore the original value of rl_sort_completion_matches
+        # before calling the completer again
+        sort_completion_inst.value = orig_sort_completion_value
+
+
 readline_path = ctypes.util.find_library('readline')
 sort_completion_inst = None
 if readline_path:

crmsh/ui_configure.py

@@ -251,9 +251,7 @@ def _prim_params_completer(agent, args):
         return []
     elif '=' in completing:
         return []
-    if command.sort_completion_inst is not None:
-        # Set the value to 0 to use the costum sort order
-        command.sort_completion_inst.value = 0
+    command.enable_custom_sort_order()
     return utils.filter_keys(agent.params(), args)