Improved: Don't need to show files names in UI messages (OFBIZ-12884)

We don't need to show files paths in UI messages. It does not help users, rather
confuse them. Also in log, we give enough information, ie the file and
method used.

Also removes all useless "this." in SimpleMethod class

Conflicts handled by hand in SimpleMethod.java

Author: JacquesLeRoux

framework/minilang/src/main/java/org/apache/ofbiz/minilang/MiniLangRuntimeException.java

@@ -41,7 +41,7 @@ public String getMessage() {
         StringBuilder sb = new StringBuilder(super.getMessage());
         if (this.element != null) {
             SimpleMethod method = this.element.getSimpleMethod();
-            sb.append(" Method = ").append(method.getMethodName()).append(", File = ").append(method.getFromLocation());
+            sb.append(", File#Method = ").append(method.getFileName()).append("#").append(method.getMethodName());
             sb.append(", Element = <").append(this.element.getTagName()).append(">");
             sb.append(", Line ").append(this.element.getLineNumber());
         }

framework/minilang/src/main/java/org/apache/ofbiz/minilang/SimpleMethod.java

@@ -75,7 +75,7 @@
  * (see <a href="http://docs.oracle.com/javase/6/docs/api/java/util/ServiceLoader.html" target="_blank">ServiceLoader</a>)
  * </li>
  * </ul>
- * 
+ *
  * @see <a href="https://cwiki.apache.org/confluence/display/OFBIZ/Mini+Language+-+minilang+-+simple-method+-+Reference">Mini-language Reference</a>
  */
 public final class SimpleMethod extends MiniLangElement {
@@ -584,8 +584,7 @@ public String exec(MethodContext methodContext) throws MiniLangException {
             }
             // rollback here passing beganTransaction to either rollback, or set rollback only
             try {
-                TransactionUtil.rollback(beganTransaction, "Error in simple-method [" + getShortDescription() + "]: "
-                        + summaryErrorStringBuffer, null);
+                TransactionUtil.rollback(beganTransaction, summaryErrorStringBuffer.toString(), null);
             } catch (GenericTransactionException e) {
                 String errMsg = "Error trying to rollback transaction, could not process method: " + e.getMessage();
                 if (methodContext.isTraceOn()) {
@@ -723,14 +722,9 @@ public String getServiceSuccessMessageName() {
     }
 
     public String getShortDescription() {
-        if (fromLocation.contains("file:") && fromLocation.contains("Events.xml")) {
-            return  getFileName() + "#" + getMethodName();
-        } else {
-            return shortDescription + " [" + fromLocation + "#" + methodName + "]";
-        }
+        return shortDescription + " [" + getFileName() + "#" + methodName + "]";
     }
 
-
     @Override
     public SimpleMethod getSimpleMethod() {
         return this;

framework/minilang/src/main/java/org/apache/ofbiz/minilang/ValidationException.java

@@ -39,7 +39,7 @@ public ValidationException(String str, SimpleMethod method, Element element) {
     public String getMessage() {
         StringBuilder sb = new StringBuilder(super.getMessage());
         if (method != null) {
-            sb.append(" Method = ").append(method.getMethodName()).append(", File = ").append(method.getFromLocation());
+            sb.append(", File#Method = ").append(method.getFileName()).append("#").append(method.getMethodName());
         }
         if (element != null) {
             sb.append(", Element = <").append(element.getTagName()).append(">");

framework/minilang/src/main/java/org/apache/ofbiz/minilang/method/envops/Break.java

@@ -56,7 +56,7 @@ public BreakElementException() {
         public String getMessage() {
             StringBuilder sb = new StringBuilder(super.getMessage());
             SimpleMethod method = getSimpleMethod();
-            sb.append(" Method = ").append(method.getMethodName()).append(", File = ").append(method.getFromLocation());
+            sb.append(", File#Method = ").append(method.getFileName()).append("#").append(method.getMethodName());
             sb.append(", Element = <break>, Line ").append(getLineNumber());
             return sb.toString();
         }

framework/minilang/src/main/java/org/apache/ofbiz/minilang/method/envops/Continue.java

@@ -56,7 +56,7 @@ public ContinueElementException() {
         public String getMessage() {
             StringBuilder sb = new StringBuilder(super.getMessage());
             SimpleMethod method = getSimpleMethod();
-            sb.append(" Method = ").append(method.getMethodName()).append(", File = ").append(method.getFromLocation());
+            sb.append(", File#Method = ").append(method.getFileName()).append("#").append(method.getMethodName());
             sb.append(", Element = <continue>, Line ").append(getLineNumber());
             return sb.toString();
         }