fix: fix 'inconsistent-return-statements' for pylint

.pylintrc

@@ -3,6 +3,7 @@
 [MESSAGES CONTROL]
 disable=all
 enable=dangerous-default-value,
+        inconsistent-return-statements,
         redefined-argument-from-local,
         unspecified-encoding,
         unused-import,

src/wuttjamaican/app.py

@@ -229,7 +229,7 @@ class AppHandler:
 
         If ``obj`` is *not* specified, this behaves a bit differently.
         It first will look for a :term:`config setting` named
-        ``wutta.app_dist`` (or similar, dpending on :attr:`appname`).
+        ``wutta.app_dist`` (or similar, depending on :attr:`appname`).
         If there is such a config value, it is returned.  Otherwise
         the "auto-locate" logic described above happens, but using
         ``self`` instead of ``obj``.
@@ -280,8 +280,7 @@ class AppHandler:
             return dist
 
         # fall back to configured dist, if obj lookup failed
-        if obj is not None:
+        return self.config.get(f'{self.appname}.app_dist')
-            return self.config.get(f'{self.appname}.app_dist')
 
     def get_version(self, dist=None, obj=None):
         """
@@ -301,6 +300,7 @@ class AppHandler:
             dist = self.get_distribution(obj=obj)
         if dist:
             return version(dist)
+        return None
 
     def get_model(self):
         """

src/wuttjamaican/auth.py

@@ -93,6 +93,7 @@ class AuthHandler(GenericHandler):
         if user and user.active and user.password:
             if self.check_user_password(user, password):
                 return user
+        return None
 
     def authenticate_user_token(self, session, token):
         """
@@ -122,6 +123,7 @@ class AuthHandler(GenericHandler):
             user = token.user
             if user.active:
                 return user
+        return None
 
     def check_user_password(self, user, password, **kwargs):
         """
@@ -158,7 +160,7 @@ class AuthHandler(GenericHandler):
         model = self.app.model
 
         if not key:
-            return
+            return None
 
         # maybe it is a uuid
         if isinstance(key, _uuid.UUID):
@@ -188,6 +190,7 @@ class AuthHandler(GenericHandler):
                               session=session)
         if key:
             return self.get_role(session, key)
+        return None
 
     def get_user(self, obj, session=None, **kwargs):
         """
@@ -256,6 +259,7 @@ class AuthHandler(GenericHandler):
         person = self.app.get_person(obj)
         if person:
             return person.user
+        return None
 
     def make_person(self, **kwargs):
         """

src/wuttjamaican/conf.py

@@ -466,6 +466,8 @@ class WuttaConfig:
         if default is not UNSPECIFIED:
             return default
 
+        return None
+
     def get_from_db(self, key, session=None, **kwargs):
         """
         Retrieve a config value from database settings table.
@@ -510,6 +512,7 @@ class WuttaConfig:
         value = self.get(*args, **kwargs)
         if value is not None:
             return int(value)
+        return None
 
     def get_list(self, *args, **kwargs):
         """
@@ -524,6 +527,7 @@ class WuttaConfig:
         value = self.get(*args, **kwargs)
         if value is not None:
             return self.parse_list(value)
+        return None
 
     def get_dict(self, prefix):
         """

src/wuttjamaican/db/model/base.py

@@ -214,3 +214,4 @@ class Person(Base):
 
         if self.users:
             return self.users[0]
+        return None

src/wuttjamaican/db/model/batch.py

@@ -263,6 +263,7 @@ class BatchMixin:
         """
         if self.id:
             return f'{self.id:08d}'
+        return None
 
 
 class BatchRowMixin:

src/wuttjamaican/db/util.py

@@ -2,7 +2,7 @@
 ################################################################################
 #
 #  WuttJamaican -- Base package for Wutta Framework
-#  Copyright © 2023-2024 Lance Edgar
+#  Copyright © 2023-2025 Lance Edgar
 #
 #  This file is part of Wutta Framework.
 #
@@ -66,6 +66,7 @@ class ModelBase:
         state = sa.inspect(self)
         if hasattr(state.attrs, key):
             return getattr(self, key)
+        raise KeyError(f"model instance has no attr with key: {key}")
 
 
 class UUID(sa.types.TypeDecorator):

src/wuttjamaican/email.py

@@ -358,6 +358,7 @@ class EmailHandler(GenericHandler):
             if instance:
                 setting = setting(self.config)
             return setting
+        return None
 
     def make_message(self, **kwargs):
         """
@@ -573,6 +574,7 @@ class EmailHandler(GenericHandler):
         if template:
             context = context or {}
             return template.render(**context)
+        return None
 
     def get_auto_html_body(self, key, context=None):
         """
@@ -585,6 +587,7 @@ class EmailHandler(GenericHandler):
         if template:
             context = context or {}
             return template.render(**context)
+        return None
 
     def get_auto_body_template(self, key, mode):
         """ """
@@ -601,6 +604,7 @@ class EmailHandler(GenericHandler):
             return templates.get_template(f'{key}.{mode}.mako')
         except TopLevelLookupException:
             pass
+        return None
 
     def get_notes(self, key):
         """

src/wuttjamaican/install.py

@@ -254,6 +254,7 @@ class InstallHandler(GenericHandler):
             sa.inspect(engine).has_table('whatever')
         except Exception as error:
             return str(error)
+        return None
 
     def make_template_context(self, dbinfo, **kwargs):
         """

src/wuttjamaican/people.py

@@ -2,7 +2,7 @@
 ################################################################################
 #
 #  WuttJamaican -- Base package for Wutta Framework
-#  Copyright © 2023-2024 Lance Edgar
+#  Copyright © 2023-2025 Lance Edgar
 #
 #  This file is part of Wutta Framework.
 #
@@ -88,3 +88,5 @@ class PeopleHandler(GenericHandler):
             user = obj
             if user.person:
                 return user.person
+
+        return None

src/wuttjamaican/problems.py

@@ -308,14 +308,14 @@ class ProblemHandler(GenericHandler):
         if not self.is_enabled(check):
             log.debug("problem check is not enabled: %s", key)
             if not force:
-                return
+                return None
 
         weekday = datetime.date.today().weekday()
         if not self.should_run_for_weekday(check, weekday):
             log.debug("problem check is not scheduled for %s: %s",
                       calendar.day_name[weekday], key)
             if not force:
-                return
+                return None
 
         check_instance = check(self.config)
         problems = self.find_problems(check_instance)

src/wuttjamaican/reports.py

@@ -2,7 +2,7 @@
 ################################################################################
 #
 #  WuttJamaican -- Base package for Wutta Framework
-#  Copyright © 2023-2024 Lance Edgar
+#  Copyright © 2023-2025 Lance Edgar
 #
 #  This file is part of Wutta Framework.
 #
@@ -196,6 +196,7 @@ class ReportHandler(GenericHandler):
             if instance:
                 report = report(self.config)
             return report
+        return None
 
     def make_report_data(self, report, params=None, progress=None, **kwargs):
         """

tests/db/model/test_batch.py

@@ -27,6 +27,10 @@ else:
             batch = MyBatch(id=42, uuid=_uuid.UUID('0675cdac-ffc9-7690-8000-6023de1c8cfd'))
             self.assertEqual(repr(batch), "MyBatch(uuid=UUID('0675cdac-ffc9-7690-8000-6023de1c8cfd'))")
             self.assertEqual(str(batch), "00000042")
+            self.assertEqual(batch.id_str, "00000042")
+
+            batch2 = MyBatch()
+            self.assertIsNone(batch2.id_str)
 
 
     class TestBatchRowMixin(DataTestCase):

tests/db/test_util.py

@@ -22,9 +22,11 @@ else:
         def test_dict_behavior(self):
             setting = Setting()
             self.assertEqual(list(iter(setting)), [('name', None), ('value', None)])
+            self.assertIsNone(setting.name)
             self.assertIsNone(setting['name'])
             setting.name = 'foo'
             self.assertEqual(setting['name'], 'foo')
+            self.assertRaises(KeyError, lambda: setting['notfound'])
 
 
     class TestUUID(TestCase):

tests/test_app.py

@@ -375,6 +375,10 @@ app_title = WuttaTest
         ver = self.app.get_version(obj=query)
         self.assertEqual(ver, version('SQLAlchemy'))
 
+        # random object will not yield a dist nor version
+        ver = self.app.get_version(obj=42)
+        self.assertIsNone(ver)
+
         # can override dist via config
         self.config.setdefault('wuttatest.app_dist', 'python-configuration')
         ver = self.app.get_version()

tests/test_reports.py

@@ -94,6 +94,10 @@ class TestReportHandler(ConfigTestCase):
             self.assertTrue(issubclass(report, mod.Report))
             self.assertIs(report, MockFooReport)
 
+            # not found
+            report = handler.get_report('unknown')
+            self.assertIsNone(report)
+
     def test_make_report_data(self):
         providers = {
             'wuttatest': MagicMock(report_modules=['tests.test_reports']),