add default share_auth and lazy load version

Author: girardinsamuel

inertia/middleware.py

@@ -1,6 +1,6 @@
 from django.http import HttpResponse
 from .share import share
-from .version import asset_version
+from .version import get_version
 
 
 class InertiaMiddleware:
@@ -13,8 +13,8 @@ def __call__(self, request):
         if is_ajax:
             isInertia = request.headers.get("X-Inertia", False)
             assert isInertia, "The client not send X-Inertia Header"
-            inertia_version = asset_version.get_version()
-            inertia_version_header = str(request.headers.get("X-Inertia-Version", "")) 
+            inertia_version = get_version()
+            inertia_version_header = str(request.headers.get("X-Inertia-Version", ""))
             if inertia_version_header != "" and \
                     inertia_version_header != str(inertia_version):
                 response = HttpResponse(status=409)

inertia/share.py

@@ -6,8 +6,8 @@ def share(request, key, value):
     request.session.setdefault("share",{})
     request.session["share"][key]=value
 
-# used in pingcrm ... 
-def share_flash(request, success=False, error=False, errors=False):  
+# used in pingcrm ...
+def share_flash(request, success=False, error=False, errors=False):
     if success:
         request.session["success"]=success
     if error:
@@ -20,3 +20,19 @@ def share_flash(request, success=False, error=False, errors=False):
     share(request, "flash",{'success':success,'error':error})
     if errors:
         share(request, "errors",errors)
+
+
+def share_auth(request):
+    """Default function for globally sharing authentication data. It can be
+    overriden with a custom function defined in SHARE_INERTIA."""
+    auth = {}
+    if request.user.is_authenticated:
+        auth = {
+            "id": request.user.id,
+            "username": request.user.username,
+            "first_name": request.user.first_name,
+            "last_name": request.user.last_name,
+            "email": request.user.email,
+        }
+
+    share(request, "auth", auth)

inertia/version.py

@@ -1,18 +1,9 @@
-# pragma pylint: disable=unused-variable
-# pragma pylint: disable=not-callable
 from django.conf import settings
 
-class Version:
 
-    def set_version(self, version):
-        self.asset_version = version
-
-    def get_version(self):
-        if callable(self.asset_version):
-            return self.asset_version()
-        else:
-            return self.asset_version
-
-
-asset_version = Version()
-asset_version.set_version(getattr(settings,"VERSION",1))
+def get_version():
+    asset_version = settings.get("VERSION", 1)
+    if callable(asset_version):
+        return asset_version()
+    else:
+        return asset_version

inertia/views.py

@@ -9,7 +9,7 @@
 from django.middleware import csrf
 from django.urls import get_callable
 from .share import share
-from .version import asset_version
+from .version import get_version
 
 from django.views.generic import View
 from django.conf import settings
@@ -49,8 +49,8 @@ def render_inertia(request, component_name, props=None, template_name=None):
             "in settings.py or pass template parameter."
         )
 
-    # share custom data if any
-    share_method_path = getattr(settings, "INERTIA_SHARE", None)
+    # share custom data or default authenticated user
+    share_method_path = getattr(settings, "INERTIA_SHARE", "inertia.share.share_auth")
     if share_method_path:
         share_method = get_callable(share_method_path)
         share_method(request)
@@ -69,21 +69,21 @@ def render_inertia(request, component_name, props=None, template_name=None):
             del request.session[key]
 
     # subsequent renders
-    inertia_version = asset_version.get_version()
+    inertia_version = get_version()
     is_version_correct = 'X-Inertia-Version' in request.headers and \
                          request.headers["X-Inertia-Version"] == str(inertia_version)
     if 'X-Inertia' in request.headers:
         response = JsonResponse({
             "component": component_name,
             "props": props,
-            "version": asset_version.get_version(),
+            "version": inertia_version,
             "url": request.get_full_path()
         })
         response['X-Inertia'] = True
         response['Vary'] = 'Accept'
         return response
     context = _build_context(component_name, props,
-                             asset_version.get_version(),
+                             inertia_version,
                              url=request.get_full_path())
     return render(request, inertia_template, context)