You are viewing a plain text version of this content. The canonical link for it is here.

Posted to notifications@ofbiz.apache.org by GitBox <gi...@apache.org> on 2020/06/04 04:56:47 UTC

[GitHub] [ofbiz-framework] priyasharma1 edited a comment on pull request #185: WIP: [Improved]: Use GroovyBaseScript's logging utility methods instead of using Debug (OFBIZ-11762)

priyasharma1 edited a comment on pull request #185:
URL: https://github.com/apache/ofbiz-framework/pull/185#issuecomment-638601279


   Thanks, Jacques
   
   Actually, in these files, I found the message to be like Debug.logError(e,  "Failure in " + module)
   Thus I did not omit the "module" in the error message.
   IMO module will still point to the current class name, and so it won't cause any loss.
   
   Or, we can alter the error message, if you agree to that.
   Please let me know your views.


----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

For queries about this service, please contact Infrastructure at:
users@infra.apache.org