[IMP] bi_sql_editor manual field check performance

The `table_columns()` slows down modules loading significantly up to 2-3x.
In most cases `model._name.startswith(self._model_prefix)` returns `False`, so better to check it first.

bi_sql_editor/models/bi_sql_view.py

@@ -704,9 +704,9 @@ class BiSQLView(models.Model):
     def check_manual_fields(self, model):
         # check the fields we need are defined on self, to stop it going
         # early on install / startup - particularly problematic during upgrade
-        if "group_operator" in table_columns(
-            self.env.cr, "bi_sql_view_field"
-        ) and model._name.startswith(self._model_prefix):
+        if model._name.startswith(
+            self._model_prefix
+        ) and "group_operator" in table_columns(self.env.cr, "bi_sql_view_field"):
             # Use SQL instead of ORM, as ORM might not be fully initialised -
             # we have no control over the order that fields are defined!
             # We are not concerned about user security rules.

bi_sql_editor/models/bi_sql_view_field.py

@@ -180,7 +180,7 @@ class BiSQLViewField(models.Model):
                     "many2one_model_id": many2one_model_id,
                 }
             )
-        return super().create(vals)
+        return super().create(vals_list)
 
     # Custom Section
     @api.model