[Yanel-commits] rev 60684 - public/yanel/trunk/src/webapp/src/java/org/wyona/yanel/servlet

michi at wyona.com michi at wyona.com
Tue Sep 13 17:00:55 CEST 2011


Author: michi
Date: 2011-09-13 17:00:54 +0200 (Tue, 13 Sep 2011)
New Revision: 60684

Modified:
   public/yanel/trunk/src/webapp/src/java/org/wyona/yanel/servlet/YanelServlet.java
Log:
check whether access logging is applicable refactoring started

Modified: public/yanel/trunk/src/webapp/src/java/org/wyona/yanel/servlet/YanelServlet.java
===================================================================
--- public/yanel/trunk/src/webapp/src/java/org/wyona/yanel/servlet/YanelServlet.java	2011-09-13 15:00:14 UTC (rev 60683)
+++ public/yanel/trunk/src/webapp/src/java/org/wyona/yanel/servlet/YanelServlet.java	2011-09-13 15:00:54 UTC (rev 60684)
@@ -2039,6 +2039,7 @@
 
         // INFO: Check if viewable resource has already created a response
         if (!view.isResponse()) {
+            // TODO: Replace with logAccessIsApplicable(view.getMimeType())
             if(logAccessEnabled) {
                 if (view.getMimeType() != null) {
                     if (isMimeTypeOk(view.getMimeType())) {
@@ -2924,4 +2925,19 @@
 
         return viewId;
     }
+
+    /**
+     * Check whether access logging makes sense
+     * @param mimeType Content type of requested resource
+     */
+    private boolean logAccessIsApplicable(String mimeType) {
+        if(logAccessEnabled) {
+            if (mimeType != null) {
+                if (isMimeTypeOk(mimeType)) {
+                    return true;
+                }
+            }
+        }
+        return false;
+    }
 }



More information about the Yanel-commits mailing list