(ofbiz-framework) 01/03: Fixed: Prevents to uselessly clutter the logs up with SetTimeZoneFromBrowser errors (OFBIZ-13061)

[jleroux]

This is an automated email from the ASF dual-hosted git repository.

jleroux pushed a commit to branch release18.12
in repository https://gitbox.apache.org/repos/asf/ofbiz-framework.git

commit 183a9a222c694c6dff53e43c6194641744d160bb
Author: Jacques Le Roux <jacques.le.r...@les7arts.com>
AuthorDate: Mon Apr 22 10:40:57 2024 +0200

    Fixed: Prevents to uselessly clutter the logs up with 
SetTimeZoneFromBrowser errors (OFBIZ-13061)
    
    Currently the demo logs, and I guess users production logs, are cluttered up
    with very common and unique SetTimeZoneFromBrowser error.
    We can easily prevent that by not returning an error from
    SetTimeZoneFromBrowser.groovy, and so the never ending
    <<Request SetTimeZoneFromBrowser caused an error with the following message:
    User authorization is required for this service: SetTimeZoneFromBrowser>>
    from RequestHandler.java
---
 framework/common/groovyScripts/SetTimeZoneFromBrowser.groovy | 3 +--
 1 file changed, 1 insertion(+), 2 deletions(-)

diff --git a/framework/common/groovyScripts/SetTimeZoneFromBrowser.groovy 
b/framework/common/groovyScripts/SetTimeZoneFromBrowser.groovy
index 21f9607916..00296dd56a 100644
--- a/framework/common/groovyScripts/SetTimeZoneFromBrowser.groovy
+++ b/framework/common/groovyScripts/SetTimeZoneFromBrowser.groovy
@@ -27,7 +27,6 @@ public Map SetTimeZoneFromBrowser() {
             userLogin.store()
             return ServiceUtil.returnSuccess()
         }
-    } else {
-        return ServiceUtil.returnError()
     }
+    // Do nothing if no userLogin to prevents to uselessly clutter the logs up 
with very common SetTimeZoneFromBrowser errors
 }