Changeset 8361148b in rattail

Timestamp:

07/27/2022 02:02:53 PM (6 months ago)

Author:

Lance Edgar <lance@…>

Branches:

master

Children:

6edc9b1

Parents:

5e978e8

Message:

Avoid more sessions in datasync, when fetching config from DB

ugh, there are probably even more somewhere..?

File:

• rattail/importing/model.py (modified) (3 diffs)

rattail/importing/model.py

@@ -961 +961 @@
         self.auto_create_unknown_group = self.config.getbool(
             'rattail.importing', 'customers.auto_create_unknown_group',
-            default=False)
+            default=False,
+            session=self.session)
 
         # set to False to log debug instead of warning for "unknown" customer group
         self.warn_for_unknown_group = self.config.getbool(
             'rattail.importing', 'customers.warn_for_unknown_group',
-            default=True)
+            default=True,
+            session=self.session)
 
     def cache_query_options(self):
@@ -1719 +1721 @@
         self.auto_create_unknown_brand = self.config.getbool(
             'rattail.importing', 'products.auto_create_unknown_brand',
-            default=True)
+            default=True,
+            session=self.session)
 
         # set to True in order to auto-create record for "unknown" category
         self.auto_create_unknown_category = self.config.getbool(
             'rattail.importing', 'products.auto_create_unknown_category',
-            default=False)
+            default=False,
+            session=self.session)
 
         # set to True in order to auto-create record for "unknown" report code
         self.auto_create_unknown_report_code = self.config.getbool(
             'rattail.importing', 'products.auto_create_unknown_report_code',
-            default=False)
+            default=False,
+            session=self.session)
 
         # set to True in order to log warning instead of debug for "unknown" brand
         self.warn_for_unknown_brand = self.config.getbool(
             'rattail.importing', 'products.warn_for_unknown_brand',
-            default=False)
+            default=False,
+            session=self.session)
 
         # set to False in order to log debug instead of warning for "unknown" category
         self.warn_for_unknown_category = self.config.getbool(
             'rattail.importing', 'products.warn_for_unknown_category',
-            default=True)
+            default=True,
+            session=self.session)
 
         # set to False in order to log debug instead of warning for "unknown" report code
         self.warn_for_unknown_report_code = self.config.getbool(
             'rattail.importing', 'products.warn_for_unknown_report_code',
-            default=True)
+            default=True,
+            session=self.session)
 
     def cache_query(self):
@@ -2786 +2794 @@
         query = super(LabelProfileImporter, self).cache_query()
 
-        if not self.config.getbool('rattail', 'labels.sync_all_profiles', default=False):
+        sync_all = self.config.getbool('rattail', 'labels.sync_all_profiles',
+                                       default=False,
+                                       session=self.session)
+        if not sync_all:
             # only fetch labels from host which are marked as "sync me"
             query = query .filter(self.model_class.sync_me == True)