You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@ofbiz.apache.org by ad...@apache.org on 2012/07/03 19:08:24 UTC

svn commit: r1356836 [1/2] - in /ofbiz/trunk/framework/minilang/src/org/ofbiz/minilang: ./ artifact/ method/ method/callops/ method/conditional/ method/entityops/ method/envops/ method/eventops/ method/ifops/ method/otherops/ method/serviceops/

Author: adrianc
Date: Tue Jul  3 17:08:16 2012
New Revision: 1356836

URL: http://svn.apache.org/viewvc?rev=1356836&view=rev
Log:
Mini-language JavaDocs.

Modified:
    ofbiz/trunk/framework/minilang/src/org/ofbiz/minilang/MiniLangElement.java
    ofbiz/trunk/framework/minilang/src/org/ofbiz/minilang/MiniLangUtil.java
    ofbiz/trunk/framework/minilang/src/org/ofbiz/minilang/MiniLangValidate.java
    ofbiz/trunk/framework/minilang/src/org/ofbiz/minilang/SimpleServiceEngine.java
    ofbiz/trunk/framework/minilang/src/org/ofbiz/minilang/artifact/ArtifactInfoContext.java
    ofbiz/trunk/framework/minilang/src/org/ofbiz/minilang/method/FieldObject.java
    ofbiz/trunk/framework/minilang/src/org/ofbiz/minilang/method/MessageElement.java
    ofbiz/trunk/framework/minilang/src/org/ofbiz/minilang/method/callops/CallBsh.java
    ofbiz/trunk/framework/minilang/src/org/ofbiz/minilang/method/callops/CallClassMethod.java
    ofbiz/trunk/framework/minilang/src/org/ofbiz/minilang/method/callops/CallObjectMethod.java
    ofbiz/trunk/framework/minilang/src/org/ofbiz/minilang/method/callops/CallService.java
    ofbiz/trunk/framework/minilang/src/org/ofbiz/minilang/method/callops/CallServiceAsynch.java
    ofbiz/trunk/framework/minilang/src/org/ofbiz/minilang/method/callops/CallSimpleMapProcessor.java
    ofbiz/trunk/framework/minilang/src/org/ofbiz/minilang/method/callops/CallSimpleMethod.java
    ofbiz/trunk/framework/minilang/src/org/ofbiz/minilang/method/callops/Return.java
    ofbiz/trunk/framework/minilang/src/org/ofbiz/minilang/method/callops/SetServiceFields.java
    ofbiz/trunk/framework/minilang/src/org/ofbiz/minilang/method/conditional/CombinedCondition.java
    ofbiz/trunk/framework/minilang/src/org/ofbiz/minilang/method/conditional/Compare.java
    ofbiz/trunk/framework/minilang/src/org/ofbiz/minilang/method/conditional/CompareCondition.java
    ofbiz/trunk/framework/minilang/src/org/ofbiz/minilang/method/conditional/CompareFieldCondition.java
    ofbiz/trunk/framework/minilang/src/org/ofbiz/minilang/method/conditional/ElseIf.java
    ofbiz/trunk/framework/minilang/src/org/ofbiz/minilang/method/conditional/HasPermissionCondition.java
    ofbiz/trunk/framework/minilang/src/org/ofbiz/minilang/method/conditional/RegexpCondition.java
    ofbiz/trunk/framework/minilang/src/org/ofbiz/minilang/method/conditional/ValidateMethodCondition.java
    ofbiz/trunk/framework/minilang/src/org/ofbiz/minilang/method/entityops/ClearCacheLine.java
    ofbiz/trunk/framework/minilang/src/org/ofbiz/minilang/method/entityops/ClearEntityCaches.java
    ofbiz/trunk/framework/minilang/src/org/ofbiz/minilang/method/entityops/CloneValue.java
    ofbiz/trunk/framework/minilang/src/org/ofbiz/minilang/method/entityops/CreateValue.java
    ofbiz/trunk/framework/minilang/src/org/ofbiz/minilang/method/entityops/EntityAnd.java
    ofbiz/trunk/framework/minilang/src/org/ofbiz/minilang/method/entityops/EntityCondition.java
    ofbiz/trunk/framework/minilang/src/org/ofbiz/minilang/method/entityops/EntityCount.java
    ofbiz/trunk/framework/minilang/src/org/ofbiz/minilang/method/entityops/EntityData.java
    ofbiz/trunk/framework/minilang/src/org/ofbiz/minilang/method/entityops/EntityOne.java
    ofbiz/trunk/framework/minilang/src/org/ofbiz/minilang/method/entityops/FilterListByAnd.java
    ofbiz/trunk/framework/minilang/src/org/ofbiz/minilang/method/entityops/FilterListByDate.java
    ofbiz/trunk/framework/minilang/src/org/ofbiz/minilang/method/entityops/GetRelated.java
    ofbiz/trunk/framework/minilang/src/org/ofbiz/minilang/method/entityops/GetRelatedOne.java
    ofbiz/trunk/framework/minilang/src/org/ofbiz/minilang/method/entityops/MakeNextSeqId.java
    ofbiz/trunk/framework/minilang/src/org/ofbiz/minilang/method/entityops/MakeValue.java
    ofbiz/trunk/framework/minilang/src/org/ofbiz/minilang/method/entityops/OrderValueList.java
    ofbiz/trunk/framework/minilang/src/org/ofbiz/minilang/method/entityops/RefreshValue.java
    ofbiz/trunk/framework/minilang/src/org/ofbiz/minilang/method/entityops/RemoveByAnd.java
    ofbiz/trunk/framework/minilang/src/org/ofbiz/minilang/method/entityops/RemoveList.java
    ofbiz/trunk/framework/minilang/src/org/ofbiz/minilang/method/entityops/RemoveRelated.java
    ofbiz/trunk/framework/minilang/src/org/ofbiz/minilang/method/entityops/RemoveValue.java
    ofbiz/trunk/framework/minilang/src/org/ofbiz/minilang/method/entityops/SequencedIdToEnv.java
    ofbiz/trunk/framework/minilang/src/org/ofbiz/minilang/method/entityops/SetCurrentUserLogin.java
    ofbiz/trunk/framework/minilang/src/org/ofbiz/minilang/method/entityops/SetNonpkFields.java
    ofbiz/trunk/framework/minilang/src/org/ofbiz/minilang/method/entityops/SetPkFields.java
    ofbiz/trunk/framework/minilang/src/org/ofbiz/minilang/method/entityops/StoreList.java
    ofbiz/trunk/framework/minilang/src/org/ofbiz/minilang/method/entityops/StoreValue.java
    ofbiz/trunk/framework/minilang/src/org/ofbiz/minilang/method/entityops/TransactionBegin.java
    ofbiz/trunk/framework/minilang/src/org/ofbiz/minilang/method/entityops/TransactionCommit.java
    ofbiz/trunk/framework/minilang/src/org/ofbiz/minilang/method/entityops/TransactionRollback.java
    ofbiz/trunk/framework/minilang/src/org/ofbiz/minilang/method/envops/AddError.java
    ofbiz/trunk/framework/minilang/src/org/ofbiz/minilang/method/envops/Break.java
    ofbiz/trunk/framework/minilang/src/org/ofbiz/minilang/method/envops/CheckId.java
    ofbiz/trunk/framework/minilang/src/org/ofbiz/minilang/method/envops/ClearField.java
    ofbiz/trunk/framework/minilang/src/org/ofbiz/minilang/method/envops/Continue.java
    ofbiz/trunk/framework/minilang/src/org/ofbiz/minilang/method/envops/CreateObject.java
    ofbiz/trunk/framework/minilang/src/org/ofbiz/minilang/method/envops/FieldToList.java
    ofbiz/trunk/framework/minilang/src/org/ofbiz/minilang/method/envops/FirstFromList.java
    ofbiz/trunk/framework/minilang/src/org/ofbiz/minilang/method/envops/Iterate.java
    ofbiz/trunk/framework/minilang/src/org/ofbiz/minilang/method/envops/IterateMap.java
    ofbiz/trunk/framework/minilang/src/org/ofbiz/minilang/method/envops/ListToList.java
    ofbiz/trunk/framework/minilang/src/org/ofbiz/minilang/method/envops/Loop.java
    ofbiz/trunk/framework/minilang/src/org/ofbiz/minilang/method/envops/MapToMap.java
    ofbiz/trunk/framework/minilang/src/org/ofbiz/minilang/method/envops/Now.java
    ofbiz/trunk/framework/minilang/src/org/ofbiz/minilang/method/envops/OrderMapList.java
    ofbiz/trunk/framework/minilang/src/org/ofbiz/minilang/method/envops/SetCalendar.java
    ofbiz/trunk/framework/minilang/src/org/ofbiz/minilang/method/envops/SetOperation.java
    ofbiz/trunk/framework/minilang/src/org/ofbiz/minilang/method/envops/StringAppend.java
    ofbiz/trunk/framework/minilang/src/org/ofbiz/minilang/method/envops/StringToList.java
    ofbiz/trunk/framework/minilang/src/org/ofbiz/minilang/method/envops/ToString.java
    ofbiz/trunk/framework/minilang/src/org/ofbiz/minilang/method/envops/While.java
    ofbiz/trunk/framework/minilang/src/org/ofbiz/minilang/method/eventops/FieldToRequest.java
    ofbiz/trunk/framework/minilang/src/org/ofbiz/minilang/method/eventops/FieldToSession.java
    ofbiz/trunk/framework/minilang/src/org/ofbiz/minilang/method/eventops/RequestParametersToList.java
    ofbiz/trunk/framework/minilang/src/org/ofbiz/minilang/method/eventops/RequestToField.java
    ofbiz/trunk/framework/minilang/src/org/ofbiz/minilang/method/eventops/SessionToField.java
    ofbiz/trunk/framework/minilang/src/org/ofbiz/minilang/method/eventops/WebappPropertyToField.java
    ofbiz/trunk/framework/minilang/src/org/ofbiz/minilang/method/ifops/CheckPermission.java
    ofbiz/trunk/framework/minilang/src/org/ofbiz/minilang/method/ifops/IfInstanceOf.java
    ofbiz/trunk/framework/minilang/src/org/ofbiz/minilang/method/ifops/IfNotEmpty.java
    ofbiz/trunk/framework/minilang/src/org/ofbiz/minilang/method/otherops/Log.java
    ofbiz/trunk/framework/minilang/src/org/ofbiz/minilang/method/otherops/PropertyToField.java
    ofbiz/trunk/framework/minilang/src/org/ofbiz/minilang/method/otherops/Trace.java
    ofbiz/trunk/framework/minilang/src/org/ofbiz/minilang/method/serviceops/FieldToResult.java

Modified: ofbiz/trunk/framework/minilang/src/org/ofbiz/minilang/MiniLangElement.java
URL: http://svn.apache.org/viewvc/ofbiz/trunk/framework/minilang/src/org/ofbiz/minilang/MiniLangElement.java?rev=1356836&r1=1356835&r2=1356836&view=diff
==============================================================================
--- ofbiz/trunk/framework/minilang/src/org/ofbiz/minilang/MiniLangElement.java (original)
+++ ofbiz/trunk/framework/minilang/src/org/ofbiz/minilang/MiniLangElement.java Tue Jul  3 17:08:16 2012
@@ -41,21 +41,42 @@ public class MiniLangElement {
         this.tagName = element.getTagName().intern();
     }
 
+    /**
+     * Updates <code>aic</code> with this element's artifact information.
+     * @param aic The artifact information context
+     */
     public void gatherArtifactInfo(ArtifactInfoContext aic) {
     }
 
+    /**
+     * Returns the source code line number for this element.
+     * @return The source code line number for this element
+     */
     public String getLineNumber() {
         return this.lineNumber == null ? "unknown" : this.lineNumber.toString();
     }
 
+    /**
+     * Returns the containing {@link  SimpleMethod} object.
+     * @return The containing {@link  SimpleMethod} object
+     */
     public SimpleMethod getSimpleMethod() {
         return this.simpleMethod;
     }
 
+    /**
+     * Returns this element's tag name.
+     * @return This element's tag name
+     */
     public String getTagName() {
         return this.tagName;
     }
 
+    /**
+     * Logs a trace message.
+     * @param methodContext
+     * @param messages
+     */
     public void outputTraceMessage(MethodContext methodContext, String... messages) {
         String lineSep = System.getProperty("line.separator");
         StringBuilder buf = new StringBuilder(getSimpleMethod().getFileName());

Modified: ofbiz/trunk/framework/minilang/src/org/ofbiz/minilang/MiniLangUtil.java
URL: http://svn.apache.org/viewvc/ofbiz/trunk/framework/minilang/src/org/ofbiz/minilang/MiniLangUtil.java?rev=1356836&r1=1356835&r2=1356836&view=diff
==============================================================================
--- ofbiz/trunk/framework/minilang/src/org/ofbiz/minilang/MiniLangUtil.java (original)
+++ ofbiz/trunk/framework/minilang/src/org/ofbiz/minilang/MiniLangUtil.java Tue Jul  3 17:08:16 2012
@@ -50,7 +50,6 @@ import org.ofbiz.minilang.method.MethodO
 import org.ofbiz.minilang.method.MethodOperation;
 import org.w3c.dom.Document;
 import org.w3c.dom.Element;
-import org.w3c.dom.NamedNodeMap;
 import org.w3c.dom.Node;
 
 /**
@@ -70,6 +69,11 @@ public final class MiniLangUtil {
         SCRIPT_PREFIXES = Collections.unmodifiableSet(scriptPrefixes);
     }
 
+    /**
+     * Returns <code>true</code> if <code>str</code> contains a script.
+     * @param str The string to test
+     * @return <code>true</code> if <code>str</code> contains a script
+     */
     public static boolean containsScript(String str) {
         if (str.length() > 0) {
             for (String scriptPrefix : SCRIPT_PREFIXES) {
@@ -81,10 +85,27 @@ public final class MiniLangUtil {
         return false;
     }
 
+    /**
+     * Returns <code>true</code> if the Mini-language script engine has been configured to
+     * auto-correct version changes.
+     * @return <code>true</code> if the Mini-language script engine has been configured to
+     * auto-correct version changes
+     */
     public static boolean autoCorrectOn() {
         return "true".equals(UtilProperties.getPropertyValue("minilang.properties", "autocorrect"));
     }
 
+    /**
+     * Calls an object method.
+     * @param operation A reference to the <code>MethodOperation</code> calling this method
+     * @param methodContext
+     * @param parameters
+     * @param methodClass
+     * @param methodObject
+     * @param methodName
+     * @param retFieldFma
+     * @throws MiniLangRuntimeException
+     */
     public static void callMethod(MethodOperation operation, MethodContext methodContext, List<MethodObject<?>> parameters, Class<?> methodClass, Object methodObject, String methodName, FlexibleMapAccessor<Object> retFieldFma) throws MiniLangRuntimeException {
         Object[] args = null;
         Class<?>[] parameterTypes = null;
@@ -115,6 +136,16 @@ public final class MiniLangUtil {
         }
     }
 
+    /**
+     * Returns <code>obj</code> converted to the type specified in <code>targetClass</code>.
+     * @param obj
+     * @param targetClass
+     * @param locale
+     * @param timeZone
+     * @param format
+     * @return The converted object
+     * @throws Exception
+     */
     @SuppressWarnings("unchecked")
     public static Object convertType(Object obj, Class<?> targetClass, Locale locale, TimeZone timeZone, String format) throws Exception {
         if (obj == null || obj == GenericEntity.NULL_FIELD) {
@@ -154,6 +185,10 @@ public final class MiniLangUtil {
         return converter.convert(obj);
     }
 
+    /**
+     * Flags a Mini-language XML document as corrected.
+     * @param element
+     */
     public static void flagDocumentAsCorrected(Element element) {
         Document doc = element.getOwnerDocument();
         if (doc != null) {
@@ -161,6 +196,16 @@ public final class MiniLangUtil {
         }
     }
 
+    /**
+     * Returns a <code>object</code>-derived <code>Class</code> that is
+     * suitable for use with the conversion framework. If the argument is
+     * <code>null</code> or a <code>String</code>, the method returns the
+     * {@link PlainString} class.
+     * 
+     * @param object
+     * @return A <code>Class</code> that is
+     * suitable for use with the conversion framework
+     */
     public static Class<?> getObjectClassForConversion(Object object) {
         if (object == null || object instanceof String) {
             return PlainString.class;
@@ -177,6 +222,14 @@ public final class MiniLangUtil {
         }
     }
 
+    /**
+     * Returns <code>true</code> if <code>attributeValue</code> is a
+     * constant value (it does not contain an expression).
+     * 
+     * @param attributeValue The value to test
+     * @return <code>true</code> if <code>attributeValue</code> is a
+     * constant value
+     */
     public static boolean isConstantAttribute(String attributeValue) {
         if (attributeValue.length() > 0) {
             return !FlexibleStringExpander.containsExpression(FlexibleStringExpander.getInstance(attributeValue));
@@ -184,6 +237,15 @@ public final class MiniLangUtil {
         return true;
     }
 
+    /**
+     * Returns <code>true</code> if <code>attributeValue</code> is a
+     * constant value (it does not contain an expression) or a constant
+     * plus expression value.
+     * 
+     * @param attributeValue The value to test
+     * @return <code>true</code> if <code>attributeValue</code> is a
+     * constant value or a constant plus expression value
+     */
     public static boolean isConstantPlusExpressionAttribute(String attributeValue) {
         if (attributeValue.length() > 0) {
             if (attributeValue.startsWith("${") && attributeValue.endsWith("}")) {
@@ -200,29 +262,23 @@ public final class MiniLangUtil {
         return true;
     }
 
+    /**
+     * Returns <code>true</code> if <code>document</code> contains corrections.
+     * 
+     * @param document The document to test
+     * @return  <code>true</code> if <code>document</code> contains corrections
+     */
     public static boolean isDocumentAutoCorrected(Document document) {
         return "true".equals(document.getUserData("autoCorrected"));
     }
 
-    public static void removeInvalidAttributes(Element element, String... validAttributeNames) {
-        Set<String> validNames = new HashSet<String>();
-        for (String name : validAttributeNames) {
-            validNames.add(name);
-        }
-        boolean elementModified = false;
-        NamedNodeMap nnm = element.getAttributes();
-        for (int i = 0; i < nnm.getLength(); i++) {
-            String attributeName = nnm.item(i).getNodeName();
-            if (!validNames.contains(attributeName)) {
-                element.removeAttribute(attributeName);
-                elementModified = true;
-            }
-        }
-        if (elementModified) {
-            flagDocumentAsCorrected(element);
-        }
-    }
-
+    /**
+     * Writes a Mini-language <code>Document</code> to disk. The XML file is styled by the
+     * config/MiniLang.xslt style sheet.
+     * 
+     * @param xmlURL
+     * @param document
+     */
     public static void writeMiniLangDocument(URL xmlURL, Document document) {
         URL styleSheetURL = null;
         InputStream styleSheetInStream = null;

Modified: ofbiz/trunk/framework/minilang/src/org/ofbiz/minilang/MiniLangValidate.java
URL: http://svn.apache.org/viewvc/ofbiz/trunk/framework/minilang/src/org/ofbiz/minilang/MiniLangValidate.java?rev=1356836&r1=1356835&r2=1356836&view=diff
==============================================================================
--- ofbiz/trunk/framework/minilang/src/org/ofbiz/minilang/MiniLangValidate.java (original)
+++ ofbiz/trunk/framework/minilang/src/org/ofbiz/minilang/MiniLangValidate.java Tue Jul  3 17:08:16 2012
@@ -63,7 +63,7 @@ public final class MiniLangValidate {
      * 
      * @param attributeValue
      * @param defaultValue
-     * @return
+     * @return <code>attributeValue</code> if it is not empty, else returns <code>defaultValue</code>
      */
     public static String checkAttribute(String attributeValue, String defaultValue) {
         return attributeValue.isEmpty() ? defaultValue : attributeValue;

Modified: ofbiz/trunk/framework/minilang/src/org/ofbiz/minilang/SimpleServiceEngine.java
URL: http://svn.apache.org/viewvc/ofbiz/trunk/framework/minilang/src/org/ofbiz/minilang/SimpleServiceEngine.java?rev=1356836&r1=1356835&r2=1356836&view=diff
==============================================================================
--- ofbiz/trunk/framework/minilang/src/org/ofbiz/minilang/SimpleServiceEngine.java (original)
+++ ofbiz/trunk/framework/minilang/src/org/ofbiz/minilang/SimpleServiceEngine.java Tue Jul  3 17:08:16 2012
@@ -27,7 +27,7 @@ import org.ofbiz.service.ServiceDispatch
 import org.ofbiz.service.engine.GenericAsyncEngine;
 
 /**
- * Standard Java Static Method Service Engine
+ * Mini-language Service Engine.
  */
 public final class SimpleServiceEngine extends GenericAsyncEngine {
 

Modified: ofbiz/trunk/framework/minilang/src/org/ofbiz/minilang/artifact/ArtifactInfoContext.java
URL: http://svn.apache.org/viewvc/ofbiz/trunk/framework/minilang/src/org/ofbiz/minilang/artifact/ArtifactInfoContext.java?rev=1356836&r1=1356835&r2=1356836&view=diff
==============================================================================
--- ofbiz/trunk/framework/minilang/src/org/ofbiz/minilang/artifact/ArtifactInfoContext.java (original)
+++ ofbiz/trunk/framework/minilang/src/org/ofbiz/minilang/artifact/ArtifactInfoContext.java Tue Jul  3 17:08:16 2012
@@ -32,30 +32,55 @@ public final class ArtifactInfoContext {
     private final Set<String> serviceNameSet = FastSet.newInstance();
     private final Set<String> simpleMethodNameSet = FastSet.newInstance();
 
+    /**
+     * Adds an entity name to this context.
+     * @param name The entity name to add to this context
+     */
     public void addEntityName(String name) {
         if (name != null) {
             this.entityNameSet.add(name);
         }
     }
 
+    /**
+     * Adds a service name to this context.
+     * @param name The service name to add to this context
+     */
     public void addServiceName(String name) {
         if (name != null) {
             this.serviceNameSet.add(name);
         }
     }
 
+    /**
+     * Adds a visited <code>SimpleMethod</code> to this context.
+     * @param method the <code>SimpleMethod</code> to add to this context
+     */
     public void addSimpleMethod(SimpleMethod method) {
         this.simpleMethodNameSet.add(method.getLocationAndName());
     }
 
+    /**
+     * Returns the entity names in this context.
+     * @return The entity names in this context
+     */
     public Set<String> getEntityNames() {
         return this.entityNameSet;
     }
 
+    /**
+     * Returns the service names in this context.
+     * @return The service names in this context
+     */
     public Set<String> getServiceNames() {
         return this.serviceNameSet;
     }
 
+    /**
+     * Returns <code>true</code> if this context has visited <code>method</code>.
+     * @param method The <code>SimpleMethod</code> to test
+     * @return <code>true</code> if this context has visited <code>method</code>
+     */
     public boolean hasVisited(SimpleMethod method) {
         return simpleMethodNameSet.contains(method.getLocationAndName());
     }

Modified: ofbiz/trunk/framework/minilang/src/org/ofbiz/minilang/method/FieldObject.java
URL: http://svn.apache.org/viewvc/ofbiz/trunk/framework/minilang/src/org/ofbiz/minilang/method/FieldObject.java?rev=1356836&r1=1356835&r2=1356836&view=diff
==============================================================================
--- ofbiz/trunk/framework/minilang/src/org/ofbiz/minilang/method/FieldObject.java (original)
+++ ofbiz/trunk/framework/minilang/src/org/ofbiz/minilang/method/FieldObject.java Tue Jul  3 17:08:16 2012
@@ -25,7 +25,9 @@ import org.ofbiz.minilang.SimpleMethod;
 import org.w3c.dom.Element;
 
 /**
- * Specifies an environment field to be passed as an argument to a method call.
+ * Implements the &lt;field&gt; element.
+ * 
+ * @see <a href="https://cwiki.apache.org/OFBADMIN/mini-language-reference.html#Mini-languageReference-{{%3Cfield%3E}}">Mini-language Reference</a>
  */
 public final class FieldObject<T> extends MethodObject<T> {
 

Modified: ofbiz/trunk/framework/minilang/src/org/ofbiz/minilang/method/MessageElement.java
URL: http://svn.apache.org/viewvc/ofbiz/trunk/framework/minilang/src/org/ofbiz/minilang/method/MessageElement.java?rev=1356836&r1=1356835&r2=1356836&view=diff
==============================================================================
--- ofbiz/trunk/framework/minilang/src/org/ofbiz/minilang/method/MessageElement.java (original)
+++ ofbiz/trunk/framework/minilang/src/org/ofbiz/minilang/method/MessageElement.java Tue Jul  3 17:08:16 2012
@@ -29,6 +29,8 @@ import org.w3c.dom.Element;
 
 /**
  * Implements the &lt;fail-message&gt; and &lt;fail-property&gt; elements.
+ * 
+ * @see <a href="https://cwiki.apache.org/OFBADMIN/mini-language-reference.html#Mini-languageReference-{{<failmessage>}}">Mini-language Reference</a>
  */
 public final class MessageElement extends MiniLangElement {
 

Modified: ofbiz/trunk/framework/minilang/src/org/ofbiz/minilang/method/callops/CallBsh.java
URL: http://svn.apache.org/viewvc/ofbiz/trunk/framework/minilang/src/org/ofbiz/minilang/method/callops/CallBsh.java?rev=1356836&r1=1356835&r2=1356836&view=diff
==============================================================================
--- ofbiz/trunk/framework/minilang/src/org/ofbiz/minilang/method/callops/CallBsh.java (original)
+++ ofbiz/trunk/framework/minilang/src/org/ofbiz/minilang/method/callops/CallBsh.java Tue Jul  3 17:08:16 2012
@@ -41,7 +41,9 @@ import bsh.EvalError;
 import bsh.Interpreter;
 
 /**
- * Executes a BSH script.
+ * Implements the &lt;call-bsh&gt; element.
+ * 
+ * @see <a href="https://cwiki.apache.org/OFBADMIN/mini-language-reference.html#Mini-languageReference-{{%3Ccallbsh%3E}}">Mini-language Reference</a>
  */
 public final class CallBsh extends MethodOperation {
 
@@ -147,11 +149,16 @@ public final class CallBsh extends Metho
         return sb.toString();
     }
 
+    /**
+     * A factory for the &lt;call-bsh&gt; element.
+     */
     public static final class CallBshFactory implements Factory<CallBsh> {
+        @Override
         public CallBsh createMethodOperation(Element element, SimpleMethod simpleMethod) throws MiniLangException {
             return new CallBsh(element, simpleMethod);
         }
 
+        @Override
         public String getName() {
             return "call-bsh";
         }

Modified: ofbiz/trunk/framework/minilang/src/org/ofbiz/minilang/method/callops/CallClassMethod.java
URL: http://svn.apache.org/viewvc/ofbiz/trunk/framework/minilang/src/org/ofbiz/minilang/method/callops/CallClassMethod.java?rev=1356836&r1=1356835&r2=1356836&view=diff
==============================================================================
--- ofbiz/trunk/framework/minilang/src/org/ofbiz/minilang/method/callops/CallClassMethod.java (original)
+++ ofbiz/trunk/framework/minilang/src/org/ofbiz/minilang/method/callops/CallClassMethod.java Tue Jul  3 17:08:16 2012
@@ -26,7 +26,6 @@ import java.util.List;
 import org.ofbiz.base.util.ObjectType;
 import org.ofbiz.base.util.UtilXml;
 import org.ofbiz.base.util.collections.FlexibleMapAccessor;
-import org.ofbiz.base.util.string.FlexibleStringExpander;
 import org.ofbiz.minilang.MiniLangException;
 import org.ofbiz.minilang.MiniLangUtil;
 import org.ofbiz.minilang.MiniLangValidate;
@@ -39,7 +38,9 @@ import org.ofbiz.minilang.method.StringO
 import org.w3c.dom.Element;
 
 /**
- * Calls a static method on a Java class.
+ * Implements the &lt;call-class-method&gt; element.
+ * 
+ * @see <a href="https://cwiki.apache.org/OFBADMIN/mini-language-reference.html#Mini-languageReference-{{%3Ccallclassmethod%3E}}">Mini-language Reference</a>
  */
 public final class CallClassMethod extends MethodOperation {
 
@@ -109,11 +110,16 @@ public final class CallClassMethod exten
         return sb.toString();
     }
 
+    /**
+     * A factory for the &lt;call-class-method&gt; element.
+     */
     public static final class CallClassMethodFactory implements Factory<CallClassMethod> {
+        @Override
         public CallClassMethod createMethodOperation(Element element, SimpleMethod simpleMethod) throws MiniLangException {
             return new CallClassMethod(element, simpleMethod);
         }
 
+        @Override
         public String getName() {
             return "call-class-method";
         }

Modified: ofbiz/trunk/framework/minilang/src/org/ofbiz/minilang/method/callops/CallObjectMethod.java
URL: http://svn.apache.org/viewvc/ofbiz/trunk/framework/minilang/src/org/ofbiz/minilang/method/callops/CallObjectMethod.java?rev=1356836&r1=1356835&r2=1356836&view=diff
==============================================================================
--- ofbiz/trunk/framework/minilang/src/org/ofbiz/minilang/method/callops/CallObjectMethod.java (original)
+++ ofbiz/trunk/framework/minilang/src/org/ofbiz/minilang/method/callops/CallObjectMethod.java Tue Jul  3 17:08:16 2012
@@ -24,7 +24,6 @@ import java.util.List;
 
 import org.ofbiz.base.util.UtilXml;
 import org.ofbiz.base.util.collections.FlexibleMapAccessor;
-import org.ofbiz.base.util.string.FlexibleStringExpander;
 import org.ofbiz.minilang.MiniLangException;
 import org.ofbiz.minilang.MiniLangRuntimeException;
 import org.ofbiz.minilang.MiniLangUtil;
@@ -38,7 +37,9 @@ import org.ofbiz.minilang.method.StringO
 import org.w3c.dom.Element;
 
 /**
- * Calls a Java object method using the given fields as parameters
+ * Implements the &lt;call-object-method&gt; element.
+ * 
+ * @see <a href="https://cwiki.apache.org/OFBADMIN/mini-language-reference.html#Mini-languageReference-{{%3Ccallobjectmethod%3E}}">Mini-language Reference</a>
  */
 public final class CallObjectMethod extends MethodOperation {
 
@@ -104,11 +105,16 @@ public final class CallObjectMethod exte
         return sb.toString();
     }
 
+    /**
+     * A factory for the &lt;call-object-method&gt; element.
+     */
     public static final class CallObjectMethodFactory implements Factory<CallObjectMethod> {
+        @Override
         public CallObjectMethod createMethodOperation(Element element, SimpleMethod simpleMethod) throws MiniLangException {
             return new CallObjectMethod(element, simpleMethod);
         }
 
+        @Override
         public String getName() {
             return "call-object-method";
         }

Modified: ofbiz/trunk/framework/minilang/src/org/ofbiz/minilang/method/callops/CallService.java
URL: http://svn.apache.org/viewvc/ofbiz/trunk/framework/minilang/src/org/ofbiz/minilang/method/callops/CallService.java?rev=1356836&r1=1356835&r2=1356836&view=diff
==============================================================================
--- ofbiz/trunk/framework/minilang/src/org/ofbiz/minilang/method/callops/CallService.java (original)
+++ ofbiz/trunk/framework/minilang/src/org/ofbiz/minilang/method/callops/CallService.java Tue Jul  3 17:08:16 2012
@@ -48,7 +48,9 @@ import org.ofbiz.service.ServiceUtil;
 import org.w3c.dom.Element;
 
 /**
- * Calls a service using the given parameters
+ * Implements the &lt;call-service&gt; element.
+ * 
+ * @see <a href="https://cwiki.apache.org/OFBADMIN/mini-language-reference.html#Mini-languageReference-{{%3Ccallservice%3E}}">Mini-language Reference</a>
  */
 public final class CallService extends MethodOperation {
 
@@ -379,11 +381,16 @@ public final class CallService extends M
         return sb.toString();
     }
 
+    /**
+     * A factory for the &lt;call-service&gt; element.
+     */
     public static final class CallServiceFactory implements Factory<CallService> {
+        @Override
         public CallService createMethodOperation(Element element, SimpleMethod simpleMethod) throws MiniLangException {
             return new CallService(element, simpleMethod);
         }
 
+        @Override
         public String getName() {
             return "call-service";
         }

Modified: ofbiz/trunk/framework/minilang/src/org/ofbiz/minilang/method/callops/CallServiceAsynch.java
URL: http://svn.apache.org/viewvc/ofbiz/trunk/framework/minilang/src/org/ofbiz/minilang/method/callops/CallServiceAsynch.java?rev=1356836&r1=1356835&r2=1356836&view=diff
==============================================================================
--- ofbiz/trunk/framework/minilang/src/org/ofbiz/minilang/method/callops/CallServiceAsynch.java (original)
+++ ofbiz/trunk/framework/minilang/src/org/ofbiz/minilang/method/callops/CallServiceAsynch.java Tue Jul  3 17:08:16 2012
@@ -37,7 +37,9 @@ import org.ofbiz.service.GenericServiceE
 import org.w3c.dom.Element;
 
 /**
- * Calls a service using the given parameters
+ * Implements the &lt;call-service-asynch&gt; element.
+ * 
+ * @see <a href="https://cwiki.apache.org/OFBADMIN/mini-language-reference.html#Mini-languageReference-{{%3Ccallserviceasynch%3E}}">Mini-language Reference</a>
  */
 public final class CallServiceAsynch extends MethodOperation {
 
@@ -126,11 +128,16 @@ public final class CallServiceAsynch ext
         return sb.toString();
     }
 
+    /**
+     * A factory for the &lt;call-service-asynch&gt; element.
+     */
     public static final class CallServiceAsynchFactory implements Factory<CallServiceAsynch> {
+        @Override
         public CallServiceAsynch createMethodOperation(Element element, SimpleMethod simpleMethod) throws MiniLangException {
             return new CallServiceAsynch(element, simpleMethod);
         }
 
+        @Override
         public String getName() {
             return "call-service-asynch";
         }

Modified: ofbiz/trunk/framework/minilang/src/org/ofbiz/minilang/method/callops/CallSimpleMapProcessor.java
URL: http://svn.apache.org/viewvc/ofbiz/trunk/framework/minilang/src/org/ofbiz/minilang/method/callops/CallSimpleMapProcessor.java?rev=1356836&r1=1356835&r2=1356836&view=diff
==============================================================================
--- ofbiz/trunk/framework/minilang/src/org/ofbiz/minilang/method/callops/CallSimpleMapProcessor.java (original)
+++ ofbiz/trunk/framework/minilang/src/org/ofbiz/minilang/method/callops/CallSimpleMapProcessor.java Tue Jul  3 17:08:16 2012
@@ -36,7 +36,9 @@ import org.ofbiz.minilang.operation.MapP
 import org.w3c.dom.Element;
 
 /**
- * An event operation that calls a simple map processor inlined or from a separate file
+ * Implements the &lt;call-map-processor&gt; element.
+ * 
+ * @see <a href="https://cwiki.apache.org/OFBADMIN/mini-language-reference.html#Mini-languageReference-{{%3Ccallmapprocessor%3E}}">Mini-language Reference</a>
  */
 public final class CallSimpleMapProcessor extends MethodOperation {
 
@@ -122,11 +124,16 @@ public final class CallSimpleMapProcesso
         return sb.toString();
     }
 
+    /**
+     * A factory for the &lt;call-map-processor&gt; element.
+     */
     public static final class CallSimpleMapProcessorFactory implements Factory<CallSimpleMapProcessor> {
+        @Override
         public CallSimpleMapProcessor createMethodOperation(Element element, SimpleMethod simpleMethod) throws MiniLangException {
             return new CallSimpleMapProcessor(element, simpleMethod);
         }
 
+        @Override
         public String getName() {
             return "call-map-processor";
         }

Modified: ofbiz/trunk/framework/minilang/src/org/ofbiz/minilang/method/callops/CallSimpleMethod.java
URL: http://svn.apache.org/viewvc/ofbiz/trunk/framework/minilang/src/org/ofbiz/minilang/method/callops/CallSimpleMethod.java?rev=1356836&r1=1356835&r2=1356836&view=diff
==============================================================================
--- ofbiz/trunk/framework/minilang/src/org/ofbiz/minilang/method/callops/CallSimpleMethod.java (original)
+++ ofbiz/trunk/framework/minilang/src/org/ofbiz/minilang/method/callops/CallSimpleMethod.java Tue Jul  3 17:08:16 2012
@@ -42,7 +42,9 @@ import org.ofbiz.minilang.method.MethodO
 import org.w3c.dom.Element;
 
 /**
- * Invokes a Mini-language simple method.
+ * Implements the &lt;call-simple-method&gt; element.
+ * 
+ * @see <a href="https://cwiki.apache.org/OFBADMIN/mini-language-reference.html#Mini-languageReference-{{%3Ccallsimplemethod%3E}}">Mini-language Reference</a>
  */
 public final class CallSimpleMethod extends MethodOperation {
 
@@ -192,11 +194,16 @@ public final class CallSimpleMethod exte
         return sb.toString();
     }
 
+    /**
+     * A factory for the &lt;call-simple-method&gt; element.
+     */
     public static final class CallSimpleMethodFactory implements Factory<CallSimpleMethod> {
+        @Override
         public CallSimpleMethod createMethodOperation(Element element, SimpleMethod simpleMethod) throws MiniLangException {
             return new CallSimpleMethod(element, simpleMethod);
         }
 
+        @Override
         public String getName() {
             return "call-simple-method";
         }

Modified: ofbiz/trunk/framework/minilang/src/org/ofbiz/minilang/method/callops/Return.java
URL: http://svn.apache.org/viewvc/ofbiz/trunk/framework/minilang/src/org/ofbiz/minilang/method/callops/Return.java?rev=1356836&r1=1356835&r2=1356836&view=diff
==============================================================================
--- ofbiz/trunk/framework/minilang/src/org/ofbiz/minilang/method/callops/Return.java (original)
+++ ofbiz/trunk/framework/minilang/src/org/ofbiz/minilang/method/callops/Return.java Tue Jul  3 17:08:16 2012
@@ -27,7 +27,9 @@ import org.ofbiz.minilang.method.MethodO
 import org.w3c.dom.Element;
 
 /**
- * An event operation that returns the given response code
+ * Implements the &lt;return&gt; element.
+ * 
+ * @see <a href="https://cwiki.apache.org/OFBADMIN/mini-language-reference.html#Mini-languageReference-{{%3Creturn%3E}}">Mini-language Reference</a>
  */
 public final class Return extends MethodOperation {
 
@@ -39,12 +41,15 @@ public final class Return extends Method
             MiniLangValidate.attributeNames(simpleMethod, element, "response-code");
             MiniLangValidate.noChildElements(simpleMethod, element);
         }
-        responseCodeFse = FlexibleStringExpander.getInstance(MiniLangValidate.checkAttribute(element.getAttribute("response-code"), "success"));
+        responseCodeFse = FlexibleStringExpander.getInstance(element.getAttribute("response-code"));
     }
 
     @Override
     public boolean exec(MethodContext methodContext) throws MiniLangException {
         String responseCode = responseCodeFse.expandString(methodContext.getEnvMap());
+        if (responseCode.isEmpty()) {
+            responseCode = simpleMethod.getDefaultSuccessCode();
+        }
         if (methodContext.getMethodType() == MethodContext.EVENT) {
             methodContext.putEnv(simpleMethod.getEventResponseCodeName(), responseCode);
         } else {
@@ -63,11 +68,16 @@ public final class Return extends Method
         return sb.toString();
     }
 
+    /**
+     * A factory for the &lt;return&gt; element.
+     */
     public static final class ReturnFactory implements Factory<Return> {
+        @Override
         public Return createMethodOperation(Element element, SimpleMethod simpleMethod) throws MiniLangException {
             return new Return(element, simpleMethod);
         }
 
+        @Override
         public String getName() {
             return "return";
         }

Modified: ofbiz/trunk/framework/minilang/src/org/ofbiz/minilang/method/callops/SetServiceFields.java
URL: http://svn.apache.org/viewvc/ofbiz/trunk/framework/minilang/src/org/ofbiz/minilang/method/callops/SetServiceFields.java?rev=1356836&r1=1356835&r2=1356836&view=diff
==============================================================================
--- ofbiz/trunk/framework/minilang/src/org/ofbiz/minilang/method/callops/SetServiceFields.java (original)
+++ ofbiz/trunk/framework/minilang/src/org/ofbiz/minilang/method/callops/SetServiceFields.java Tue Jul  3 17:08:16 2012
@@ -40,7 +40,9 @@ import org.ofbiz.service.ModelService;
 import org.w3c.dom.Element;
 
 /**
- * Sets all Service parameters/attributes in the to-map using the map as a source.
+ * Implements the &lt;set-service-fields&gt; element.
+ * 
+ * @see <a href="https://cwiki.apache.org/OFBADMIN/mini-language-reference.html#Mini-languageReference-{{%3Csetservicefields%3E}}">Mini-language Reference</a>
  */
 public final class SetServiceFields extends MethodOperation {
 
@@ -80,6 +82,13 @@ public final class SetServiceFields exte
 
     @Override
     public boolean exec(MethodContext methodContext) throws MiniLangException {
+        Map<String, ? extends Object> fromMap = mapFma.get(methodContext.getEnvMap());
+        if (fromMap == null) {
+            if (Debug.verboseOn()) {
+                Debug.logVerbose("The from map in set-service-field was not found with name: " + mapFma, module);
+            }
+            return true;
+        }
         String serviceName = serviceNameFse.expandString(methodContext.getEnvMap());
         ModelService modelService = null;
         try {
@@ -92,13 +101,6 @@ public final class SetServiceFields exte
             toMap = FastMap.newInstance();
             toMapFma.put(methodContext.getEnvMap(), toMap);
         }
-        Map<String, ? extends Object> fromMap = mapFma.get(methodContext.getEnvMap());
-        if (fromMap == null) {
-            if (Debug.verboseOn()) {
-                Debug.logVerbose("The from map in set-service-field was not found with name: " + mapFma, module);
-            }
-            return true;
-        }
         List<Object> errorMessages = FastList.newInstance();
         Map<String, Object> validAttributes = modelService.makeValid(fromMap, "IN", true, errorMessages, methodContext.getTimeZone(), methodContext.getLocale());
         if (errorMessages.size() > 0) {
@@ -132,11 +134,16 @@ public final class SetServiceFields exte
         return sb.toString();
     }
 
+    /**
+     * A factory for the &lt;set-service-fields&gt; element.
+     */
     public static final class SetServiceFieldsFactory implements Factory<SetServiceFields> {
+        @Override
         public SetServiceFields createMethodOperation(Element element, SimpleMethod simpleMethod) throws MiniLangException {
             return new SetServiceFields(element, simpleMethod);
         }
 
+        @Override
         public String getName() {
             return "set-service-fields";
         }

Modified: ofbiz/trunk/framework/minilang/src/org/ofbiz/minilang/method/conditional/CombinedCondition.java
URL: http://svn.apache.org/viewvc/ofbiz/trunk/framework/minilang/src/org/ofbiz/minilang/method/conditional/CombinedCondition.java?rev=1356836&r1=1356835&r2=1356836&view=diff
==============================================================================
--- ofbiz/trunk/framework/minilang/src/org/ofbiz/minilang/method/conditional/CombinedCondition.java (original)
+++ ofbiz/trunk/framework/minilang/src/org/ofbiz/minilang/method/conditional/CombinedCondition.java Tue Jul  3 17:08:16 2012
@@ -32,6 +32,8 @@ import org.w3c.dom.Element;
 
 /**
  * Implements the &lt;and&gt;, &lt;or&gt;, &lt;not&gt;, and &lt;xor&gt; elements.
+ * 
+ * @see <a href="https://cwiki.apache.org/OFBADMIN/mini-language-reference.html#Mini-languageReference-Conditional%2FLoopingStatements">Mini-language Reference</a>
  */
 public abstract class CombinedCondition extends MiniLangElement implements Conditional {
 

Modified: ofbiz/trunk/framework/minilang/src/org/ofbiz/minilang/method/conditional/Compare.java
URL: http://svn.apache.org/viewvc/ofbiz/trunk/framework/minilang/src/org/ofbiz/minilang/method/conditional/Compare.java?rev=1356836&r1=1356835&r2=1356836&view=diff
==============================================================================
--- ofbiz/trunk/framework/minilang/src/org/ofbiz/minilang/method/conditional/Compare.java (original)
+++ ofbiz/trunk/framework/minilang/src/org/ofbiz/minilang/method/conditional/Compare.java Tue Jul  3 17:08:16 2012
@@ -79,7 +79,7 @@ public abstract class Compare {
      * Returns a <code>Compare</code> instance for the specified operator.
      * 
      * @param operator
-     * @return
+     * @return A <code>Compare</code> instance for the specified operator
      */
     public static Compare getInstance(String operator) {
         Assert.notNull("operator", operator);
@@ -95,7 +95,7 @@ public abstract class Compare {
      * @param format Optional format to be used in object type conversions
      * @param locale Optional locale to be used in object type conversions
      * @param timeZone Optional time zone to be used in object type conversions
-     * @return
+     * @return The result of this comparison
      * @throws Exception
      */
     public abstract boolean doCompare(Object lValue, Object rValue, Class<?> type, Locale locale, TimeZone timeZone, String format) throws Exception;

Modified: ofbiz/trunk/framework/minilang/src/org/ofbiz/minilang/method/conditional/CompareCondition.java
URL: http://svn.apache.org/viewvc/ofbiz/trunk/framework/minilang/src/org/ofbiz/minilang/method/conditional/CompareCondition.java?rev=1356836&r1=1356835&r2=1356836&view=diff
==============================================================================
--- ofbiz/trunk/framework/minilang/src/org/ofbiz/minilang/method/conditional/CompareCondition.java (original)
+++ ofbiz/trunk/framework/minilang/src/org/ofbiz/minilang/method/conditional/CompareCondition.java Tue Jul  3 17:08:16 2012
@@ -38,6 +38,8 @@ import org.w3c.dom.Element;
 
 /**
  * Implements the &lt;if-compare&gt; element.
+ * 
+ * @see <a href="https://cwiki.apache.org/OFBADMIN/mini-language-reference.html#Mini-languageReference-{{%3Cifcompare%3E}}">Mini-language Reference</a>
  */
 public final class CompareCondition extends MethodOperation implements Conditional {
 
@@ -58,7 +60,7 @@ public final class CompareCondition exte
             MiniLangValidate.attributeNames(simpleMethod, element, "field", "format", "operator", "type", "value");
             MiniLangValidate.requiredAttributes(simpleMethod, element, "field", "operator", "value");
             MiniLangValidate.constantAttributes(simpleMethod, element, "operator", "type");
-            MiniLangValidate.constantPlusExpressionAttributes(simpleMethod, element, "value", "format");
+            MiniLangValidate.constantPlusExpressionAttributes(simpleMethod, element, "value");
             MiniLangValidate.expressionAttributes(simpleMethod, element, "field");
         }
         this.fieldFma = FlexibleMapAccessor.getInstance(element.getAttribute("field"));
@@ -101,7 +103,7 @@ public final class CompareCondition exte
     @Override
     public boolean checkCondition(MethodContext methodContext) throws MiniLangException {
         if (this.compare == null) {
-            throw new MiniLangRuntimeException("Invalid operator " + this.operator, this);
+            throw new MiniLangRuntimeException("Invalid operator \"" + this.operator + "\"", this);
         }
         Object fieldVal = fieldFma.get(methodContext.getEnvMap());
         Class<?> targetClass = this.targetClass;

Modified: ofbiz/trunk/framework/minilang/src/org/ofbiz/minilang/method/conditional/CompareFieldCondition.java
URL: http://svn.apache.org/viewvc/ofbiz/trunk/framework/minilang/src/org/ofbiz/minilang/method/conditional/CompareFieldCondition.java?rev=1356836&r1=1356835&r2=1356836&view=diff
==============================================================================
--- ofbiz/trunk/framework/minilang/src/org/ofbiz/minilang/method/conditional/CompareFieldCondition.java (original)
+++ ofbiz/trunk/framework/minilang/src/org/ofbiz/minilang/method/conditional/CompareFieldCondition.java Tue Jul  3 17:08:16 2012
@@ -38,6 +38,8 @@ import org.w3c.dom.Element;
 
 /**
  * Implements the &lt;if-compare-field&gt; element.
+ * 
+ * @see <a href="https://cwiki.apache.org/OFBADMIN/mini-language-reference.html#Mini-languageReference-{{%3Cifcomparefield%3E}}">Mini-language Reference</a>
  */
 public final class CompareFieldCondition extends MethodOperation implements Conditional {
 

Modified: ofbiz/trunk/framework/minilang/src/org/ofbiz/minilang/method/conditional/ElseIf.java
URL: http://svn.apache.org/viewvc/ofbiz/trunk/framework/minilang/src/org/ofbiz/minilang/method/conditional/ElseIf.java?rev=1356836&r1=1356835&r2=1356836&view=diff
==============================================================================
--- ofbiz/trunk/framework/minilang/src/org/ofbiz/minilang/method/conditional/ElseIf.java (original)
+++ ofbiz/trunk/framework/minilang/src/org/ofbiz/minilang/method/conditional/ElseIf.java Tue Jul  3 17:08:16 2012
@@ -33,6 +33,8 @@ import org.w3c.dom.Element;
 
 /**
  * Implements the &lt;else-if&gt; element.
+ * 
+ * @see <a href="https://cwiki.apache.org/OFBADMIN/mini-language-reference.html#Mini-languageReference-{{%3Celseif%3E}}">Mini-language Reference</a>
  */
 public final class ElseIf extends MiniLangElement {
 

Modified: ofbiz/trunk/framework/minilang/src/org/ofbiz/minilang/method/conditional/HasPermissionCondition.java
URL: http://svn.apache.org/viewvc/ofbiz/trunk/framework/minilang/src/org/ofbiz/minilang/method/conditional/HasPermissionCondition.java?rev=1356836&r1=1356835&r2=1356836&view=diff
==============================================================================
--- ofbiz/trunk/framework/minilang/src/org/ofbiz/minilang/method/conditional/HasPermissionCondition.java (original)
+++ ofbiz/trunk/framework/minilang/src/org/ofbiz/minilang/method/conditional/HasPermissionCondition.java Tue Jul  3 17:08:16 2012
@@ -37,6 +37,8 @@ import org.w3c.dom.Element;
 
 /**
  * Implements the &lt;if-has-permission&gt; element.
+ * 
+ * @see <a href="https://cwiki.apache.org/OFBADMIN/mini-language-reference.html#Mini-languageReference-{{%3Cifhaspermission%3E}}">Mini-language Reference</a>
  */
 public final class HasPermissionCondition extends MethodOperation implements Conditional {
 

Modified: ofbiz/trunk/framework/minilang/src/org/ofbiz/minilang/method/conditional/RegexpCondition.java
URL: http://svn.apache.org/viewvc/ofbiz/trunk/framework/minilang/src/org/ofbiz/minilang/method/conditional/RegexpCondition.java?rev=1356836&r1=1356835&r2=1356836&view=diff
==============================================================================
--- ofbiz/trunk/framework/minilang/src/org/ofbiz/minilang/method/conditional/RegexpCondition.java (original)
+++ ofbiz/trunk/framework/minilang/src/org/ofbiz/minilang/method/conditional/RegexpCondition.java Tue Jul  3 17:08:16 2012
@@ -38,6 +38,8 @@ import org.w3c.dom.Element;
 
 /**
  * Implements the &lt;if-regexp&gt; element.
+ * 
+ * @see <a href="https://cwiki.apache.org/OFBADMIN/mini-language-reference.html#Mini-languageReference-{{%3Cifregexp%3E}}">Mini-language Reference</a>
  */
 public class RegexpCondition extends MethodOperation implements Conditional {
 
@@ -55,7 +57,6 @@ public class RegexpCondition extends Met
         if (MiniLangValidate.validationOn()) {
             MiniLangValidate.attributeNames(simpleMethod, element, "field", "expr");
             MiniLangValidate.requiredAttributes(simpleMethod, element, "field", "expr");
-            MiniLangValidate.constantPlusExpressionAttributes(simpleMethod, element, "expr");
             MiniLangValidate.expressionAttributes(simpleMethod, element, "field");
         }
         this.fieldFma = FlexibleMapAccessor.getInstance(element.getAttribute("field"));

Modified: ofbiz/trunk/framework/minilang/src/org/ofbiz/minilang/method/conditional/ValidateMethodCondition.java
URL: http://svn.apache.org/viewvc/ofbiz/trunk/framework/minilang/src/org/ofbiz/minilang/method/conditional/ValidateMethodCondition.java?rev=1356836&r1=1356835&r2=1356836&view=diff
==============================================================================
--- ofbiz/trunk/framework/minilang/src/org/ofbiz/minilang/method/conditional/ValidateMethodCondition.java (original)
+++ ofbiz/trunk/framework/minilang/src/org/ofbiz/minilang/method/conditional/ValidateMethodCondition.java Tue Jul  3 17:08:16 2012
@@ -36,6 +36,8 @@ import org.w3c.dom.Element;
 
 /**
  * Implements the &lt;if-validate-method&gt; element.
+ * 
+ * @see <a href="https://cwiki.apache.org/OFBADMIN/mini-language-reference.html#Mini-languageReference-{{%3Cifvalidatemethod%3E}}">Mini-language Reference</a>
  */
 public final class ValidateMethodCondition extends MethodOperation implements Conditional {
 

Modified: ofbiz/trunk/framework/minilang/src/org/ofbiz/minilang/method/entityops/ClearCacheLine.java
URL: http://svn.apache.org/viewvc/ofbiz/trunk/framework/minilang/src/org/ofbiz/minilang/method/entityops/ClearCacheLine.java?rev=1356836&r1=1356835&r2=1356836&view=diff
==============================================================================
--- ofbiz/trunk/framework/minilang/src/org/ofbiz/minilang/method/entityops/ClearCacheLine.java (original)
+++ ofbiz/trunk/framework/minilang/src/org/ofbiz/minilang/method/entityops/ClearCacheLine.java Tue Jul  3 17:08:16 2012
@@ -31,6 +31,8 @@ import org.w3c.dom.Element;
 
 /**
  * Implements the &lt;clear-cache-line&gt; element.
+ * 
+ * @see <a href="https://cwiki.apache.org/OFBADMIN/mini-language-reference.html#Mini-languageReference-{{%3Cclearcacheline%3E}}">Mini-language Reference</a>
  */
 public final class ClearCacheLine extends MethodOperation {
 
@@ -76,10 +78,12 @@ public final class ClearCacheLine extend
      * A factory for the &lt;clear-cache-line&gt; element.
      */
     public static final class ClearCacheLineFactory implements Factory<ClearCacheLine> {
+        @Override
         public ClearCacheLine createMethodOperation(Element element, SimpleMethod simpleMethod) throws MiniLangException {
             return new ClearCacheLine(element, simpleMethod);
         }
 
+        @Override
         public String getName() {
             return "clear-cache-line";
         }

Modified: ofbiz/trunk/framework/minilang/src/org/ofbiz/minilang/method/entityops/ClearEntityCaches.java
URL: http://svn.apache.org/viewvc/ofbiz/trunk/framework/minilang/src/org/ofbiz/minilang/method/entityops/ClearEntityCaches.java?rev=1356836&r1=1356835&r2=1356836&view=diff
==============================================================================
--- ofbiz/trunk/framework/minilang/src/org/ofbiz/minilang/method/entityops/ClearEntityCaches.java (original)
+++ ofbiz/trunk/framework/minilang/src/org/ofbiz/minilang/method/entityops/ClearEntityCaches.java Tue Jul  3 17:08:16 2012
@@ -28,6 +28,8 @@ import org.w3c.dom.NamedNodeMap;
 
 /**
  * Implements the &lt;clear-entity-caches&gt; element.
+ * 
+ * @see <a href="https://cwiki.apache.org/OFBADMIN/mini-language-reference.html#Mini-languageReference-{{%3Cclearentitycaches%3E}}">Mini-language Reference</a>
  */
 public final class ClearEntityCaches extends MethodOperation {
 
@@ -58,10 +60,12 @@ public final class ClearEntityCaches ext
      * A factory for the &lt;clear-entity-caches&gt; element.
      */
     public static final class ClearEntityCachesFactory implements Factory<ClearEntityCaches> {
+        @Override
         public ClearEntityCaches createMethodOperation(Element element, SimpleMethod simpleMethod) throws MiniLangException {
             return new ClearEntityCaches(element, simpleMethod);
         }
 
+        @Override
         public String getName() {
             return "clear-entity-caches";
         }

Modified: ofbiz/trunk/framework/minilang/src/org/ofbiz/minilang/method/entityops/CloneValue.java
URL: http://svn.apache.org/viewvc/ofbiz/trunk/framework/minilang/src/org/ofbiz/minilang/method/entityops/CloneValue.java?rev=1356836&r1=1356835&r2=1356836&view=diff
==============================================================================
--- ofbiz/trunk/framework/minilang/src/org/ofbiz/minilang/method/entityops/CloneValue.java (original)
+++ ofbiz/trunk/framework/minilang/src/org/ofbiz/minilang/method/entityops/CloneValue.java Tue Jul  3 17:08:16 2012
@@ -29,6 +29,8 @@ import org.w3c.dom.Element;
 
 /**
  * Implements the &lt;clone-value&gt; element.
+ * 
+ * @see <a href="https://cwiki.apache.org/OFBADMIN/mini-language-reference.html#Mini-languageReference-{{%3Cclonevalue%3E}}">Mini-language Reference</a>
  */
 public final class CloneValue extends MethodOperation {
 
@@ -69,10 +71,12 @@ public final class CloneValue extends Me
      * A factory for the &lt;clone-value&gt; element.
      */
     public static final class CloneValueFactory implements Factory<CloneValue> {
+        @Override
         public CloneValue createMethodOperation(Element element, SimpleMethod simpleMethod) throws MiniLangException {
             return new CloneValue(element, simpleMethod);
         }
 
+        @Override
         public String getName() {
             return "clone-value";
         }

Modified: ofbiz/trunk/framework/minilang/src/org/ofbiz/minilang/method/entityops/CreateValue.java
URL: http://svn.apache.org/viewvc/ofbiz/trunk/framework/minilang/src/org/ofbiz/minilang/method/entityops/CreateValue.java?rev=1356836&r1=1356835&r2=1356836&view=diff
==============================================================================
--- ofbiz/trunk/framework/minilang/src/org/ofbiz/minilang/method/entityops/CreateValue.java (original)
+++ ofbiz/trunk/framework/minilang/src/org/ofbiz/minilang/method/entityops/CreateValue.java Tue Jul  3 17:08:16 2012
@@ -31,6 +31,8 @@ import org.w3c.dom.Element;
 
 /**
  * Implements the &lt;create-value&gt; element.
+ * 
+ * @see <a href="https://cwiki.apache.org/OFBADMIN/mini-language-reference.html#Mini-languageReference-{{%3Ccreatevalue%3E}}">Mini-language Reference</a>
  */
 public final class CreateValue extends MethodOperation {
 
@@ -96,10 +98,12 @@ public final class CreateValue extends M
      * A factory for the &lt;create-value&gt; element.
      */
     public static final class CreateValueFactory implements Factory<CreateValue> {
+        @Override
         public CreateValue createMethodOperation(Element element, SimpleMethod simpleMethod) throws MiniLangException {
             return new CreateValue(element, simpleMethod);
         }
 
+        @Override
         public String getName() {
             return "create-value";
         }

Modified: ofbiz/trunk/framework/minilang/src/org/ofbiz/minilang/method/entityops/EntityAnd.java
URL: http://svn.apache.org/viewvc/ofbiz/trunk/framework/minilang/src/org/ofbiz/minilang/method/entityops/EntityAnd.java?rev=1356836&r1=1356835&r2=1356836&view=diff
==============================================================================
--- ofbiz/trunk/framework/minilang/src/org/ofbiz/minilang/method/entityops/EntityAnd.java (original)
+++ ofbiz/trunk/framework/minilang/src/org/ofbiz/minilang/method/entityops/EntityAnd.java Tue Jul  3 17:08:16 2012
@@ -32,6 +32,8 @@ import org.w3c.dom.Element;
 
 /**
  * Implements the &lt;entity-and&gt; element.
+ * 
+ * @see <a href="https://cwiki.apache.org/OFBADMIN/mini-language-reference.html#Mini-languageReference-{{%3Centityand%3E}}">Mini-language Reference</a>
  */
 public final class EntityAnd extends MethodOperation {
 
@@ -81,10 +83,12 @@ public final class EntityAnd extends Met
      * A factory for the &lt;entity-and&gt; element.
      */
     public static final class EntityAndFactory implements Factory<EntityAnd> {
+        @Override
         public EntityAnd createMethodOperation(Element element, SimpleMethod simpleMethod) throws MiniLangException {
             return new EntityAnd(element, simpleMethod);
         }
 
+        @Override
         public String getName() {
             return "entity-and";
         }

Modified: ofbiz/trunk/framework/minilang/src/org/ofbiz/minilang/method/entityops/EntityCondition.java
URL: http://svn.apache.org/viewvc/ofbiz/trunk/framework/minilang/src/org/ofbiz/minilang/method/entityops/EntityCondition.java?rev=1356836&r1=1356835&r2=1356836&view=diff
==============================================================================
--- ofbiz/trunk/framework/minilang/src/org/ofbiz/minilang/method/entityops/EntityCondition.java (original)
+++ ofbiz/trunk/framework/minilang/src/org/ofbiz/minilang/method/entityops/EntityCondition.java Tue Jul  3 17:08:16 2012
@@ -31,7 +31,9 @@ import org.ofbiz.minilang.method.MethodO
 import org.w3c.dom.Element;
 
 /**
- * Implements the &lt;entity-and&gt; element.
+ * Implements the &lt;entity-condition&gt; element.
+ * 
+ * @see <a href="https://cwiki.apache.org/OFBADMIN/mini-language-reference.html#Mini-languageReference-{{%3Centitycondition%3E}}">Mini-language Reference</a>
  */
 public final class EntityCondition extends MethodOperation {
 
@@ -78,10 +80,12 @@ public final class EntityCondition exten
     }
 
     public static final class EntityConditionFactory implements Factory<EntityCondition> {
+        @Override
         public EntityCondition createMethodOperation(Element element, SimpleMethod simpleMethod) throws MiniLangException {
             return new EntityCondition(element, simpleMethod);
         }
 
+        @Override
         public String getName() {
             return "entity-condition";
         }

Modified: ofbiz/trunk/framework/minilang/src/org/ofbiz/minilang/method/entityops/EntityCount.java
URL: http://svn.apache.org/viewvc/ofbiz/trunk/framework/minilang/src/org/ofbiz/minilang/method/entityops/EntityCount.java?rev=1356836&r1=1356835&r2=1356836&view=diff
==============================================================================
--- ofbiz/trunk/framework/minilang/src/org/ofbiz/minilang/method/entityops/EntityCount.java (original)
+++ ofbiz/trunk/framework/minilang/src/org/ofbiz/minilang/method/entityops/EntityCount.java Tue Jul  3 17:08:16 2012
@@ -42,6 +42,8 @@ import org.w3c.dom.Element;
 
 /**
  * Implements the &lt;entity-count&gt; element.
+ * 
+ * @see <a href="https://cwiki.apache.org/OFBADMIN/mini-language-reference.html#Mini-languageReference-{{%3Centitycount%3E}}">Mini-language Reference</a>
  */
 public final class EntityCount extends MethodOperation {
 

Modified: ofbiz/trunk/framework/minilang/src/org/ofbiz/minilang/method/entityops/EntityData.java
URL: http://svn.apache.org/viewvc/ofbiz/trunk/framework/minilang/src/org/ofbiz/minilang/method/entityops/EntityData.java?rev=1356836&r1=1356835&r2=1356836&view=diff
==============================================================================
--- ofbiz/trunk/framework/minilang/src/org/ofbiz/minilang/method/entityops/EntityData.java (original)
+++ ofbiz/trunk/framework/minilang/src/org/ofbiz/minilang/method/entityops/EntityData.java Tue Jul  3 17:08:16 2012
@@ -42,6 +42,8 @@ import org.w3c.dom.Element;
 
 /**
  * Implements the &lt;entity-data&gt; element.
+ * 
+ * @see <a href="https://cwiki.apache.org/OFBADMIN/mini-language-reference.html#Mini-languageReference-{{%3Centitydata%3E}}">Mini-language Reference</a>
  */
 public final class EntityData extends MethodOperation {
 
@@ -63,7 +65,11 @@ public final class EntityData extends Me
         }
         locationFse = FlexibleStringExpander.getInstance(element.getAttribute("location"));
         delegatorNameFse = FlexibleStringExpander.getInstance(element.getAttribute("delegator-name"));
+        mode = MiniLangValidate.checkAttribute(element.getAttribute("mode"), "load");
         String timeoutAttribute = element.getAttribute("timeout");
+        if (!"load".equals(mode) && !timeoutAttribute.isEmpty()) {
+            MiniLangValidate.handleError("timeout attribute is valid only when mode=\"load\".", simpleMethod, element);
+        }
         int timeout = -1;
         if (!timeoutAttribute.isEmpty()) {
             try {
@@ -74,7 +80,6 @@ public final class EntityData extends Me
         }
         this.timeout = timeout;
         errorListFma = FlexibleMapAccessor.getInstance(MiniLangValidate.checkAttribute(element.getAttribute("error-list-name"), "error_list"));
-        mode = MiniLangValidate.checkAttribute(element.getAttribute("mode"), "load");
     }
 
     @Override

Modified: ofbiz/trunk/framework/minilang/src/org/ofbiz/minilang/method/entityops/EntityOne.java
URL: http://svn.apache.org/viewvc/ofbiz/trunk/framework/minilang/src/org/ofbiz/minilang/method/entityops/EntityOne.java?rev=1356836&r1=1356835&r2=1356836&view=diff
==============================================================================
--- ofbiz/trunk/framework/minilang/src/org/ofbiz/minilang/method/entityops/EntityOne.java (original)
+++ ofbiz/trunk/framework/minilang/src/org/ofbiz/minilang/method/entityops/EntityOne.java Tue Jul  3 17:08:16 2012
@@ -32,6 +32,8 @@ import org.w3c.dom.Element;
 
 /**
  * Implements the &lt;entity-one&gt; element.
+ * 
+ * @see <a href="https://cwiki.apache.org/OFBADMIN/mini-language-reference.html#Mini-languageReference-{{%3Centityone%3E}}">Mini-language Reference</a>
  */
 public final class EntityOne extends MethodOperation {
 

Modified: ofbiz/trunk/framework/minilang/src/org/ofbiz/minilang/method/entityops/FilterListByAnd.java
URL: http://svn.apache.org/viewvc/ofbiz/trunk/framework/minilang/src/org/ofbiz/minilang/method/entityops/FilterListByAnd.java?rev=1356836&r1=1356835&r2=1356836&view=diff
==============================================================================
--- ofbiz/trunk/framework/minilang/src/org/ofbiz/minilang/method/entityops/FilterListByAnd.java (original)
+++ ofbiz/trunk/framework/minilang/src/org/ofbiz/minilang/method/entityops/FilterListByAnd.java Tue Jul  3 17:08:16 2012
@@ -33,6 +33,8 @@ import org.w3c.dom.Element;
 
 /**
  * Implements the &lt;filter-list-by-and&gt; element.
+ * 
+ * @see <a href="https://cwiki.apache.org/OFBADMIN/mini-language-reference.html#Mini-languageReference-{{%3Cfilterlistbyand%3E}}">Mini-language Reference</a>
  */
 public final class FilterListByAnd extends MethodOperation {
 

Modified: ofbiz/trunk/framework/minilang/src/org/ofbiz/minilang/method/entityops/FilterListByDate.java
URL: http://svn.apache.org/viewvc/ofbiz/trunk/framework/minilang/src/org/ofbiz/minilang/method/entityops/FilterListByDate.java?rev=1356836&r1=1356835&r2=1356836&view=diff
==============================================================================
--- ofbiz/trunk/framework/minilang/src/org/ofbiz/minilang/method/entityops/FilterListByDate.java (original)
+++ ofbiz/trunk/framework/minilang/src/org/ofbiz/minilang/method/entityops/FilterListByDate.java Tue Jul  3 17:08:16 2012
@@ -34,6 +34,8 @@ import org.w3c.dom.Element;
 
 /**
  * Implements the &lt;filter-list-by-date&gt; element.
+ * 
+ * @see <a href="https://cwiki.apache.org/OFBADMIN/mini-language-reference.html#Mini-languageReference-{{%3Cfilterlistbydate%3E}}">Mini-language Reference</a>
  */
 public final class FilterListByDate extends MethodOperation {
 

Modified: ofbiz/trunk/framework/minilang/src/org/ofbiz/minilang/method/entityops/GetRelated.java
URL: http://svn.apache.org/viewvc/ofbiz/trunk/framework/minilang/src/org/ofbiz/minilang/method/entityops/GetRelated.java?rev=1356836&r1=1356835&r2=1356836&view=diff
==============================================================================
--- ofbiz/trunk/framework/minilang/src/org/ofbiz/minilang/method/entityops/GetRelated.java (original)
+++ ofbiz/trunk/framework/minilang/src/org/ofbiz/minilang/method/entityops/GetRelated.java Tue Jul  3 17:08:16 2012
@@ -37,6 +37,8 @@ import org.w3c.dom.Element;
 
 /**
  * Implements the &lt;get-related&gt; element.
+ * 
+ * @see <a href="https://cwiki.apache.org/OFBADMIN/mini-language-reference.html#Mini-languageReference-{{%3Cgetrelated%3E}}">Mini-language Reference</a>
  */
 public final class GetRelated extends MethodOperation {
 

Modified: ofbiz/trunk/framework/minilang/src/org/ofbiz/minilang/method/entityops/GetRelatedOne.java
URL: http://svn.apache.org/viewvc/ofbiz/trunk/framework/minilang/src/org/ofbiz/minilang/method/entityops/GetRelatedOne.java?rev=1356836&r1=1356835&r2=1356836&view=diff
==============================================================================
--- ofbiz/trunk/framework/minilang/src/org/ofbiz/minilang/method/entityops/GetRelatedOne.java (original)
+++ ofbiz/trunk/framework/minilang/src/org/ofbiz/minilang/method/entityops/GetRelatedOne.java Tue Jul  3 17:08:16 2012
@@ -34,6 +34,8 @@ import org.w3c.dom.Element;
 
 /**
  * Implements the &lt;get-related-one&gt; element.
+ * 
+ * @see <a href="https://cwiki.apache.org/OFBADMIN/mini-language-reference.html#Mini-languageReference-{{%3Cgetrelatedone%3E}}">Mini-language Reference</a>
  */
 public final class GetRelatedOne extends MethodOperation {
 

Modified: ofbiz/trunk/framework/minilang/src/org/ofbiz/minilang/method/entityops/MakeNextSeqId.java
URL: http://svn.apache.org/viewvc/ofbiz/trunk/framework/minilang/src/org/ofbiz/minilang/method/entityops/MakeNextSeqId.java?rev=1356836&r1=1356835&r2=1356836&view=diff
==============================================================================
--- ofbiz/trunk/framework/minilang/src/org/ofbiz/minilang/method/entityops/MakeNextSeqId.java (original)
+++ ofbiz/trunk/framework/minilang/src/org/ofbiz/minilang/method/entityops/MakeNextSeqId.java Tue Jul  3 17:08:16 2012
@@ -31,6 +31,8 @@ import org.w3c.dom.Element;
 
 /**
  * Implements the &lt;make-next-seq-id&gt; element.
+ * 
+ * @see <a href="https://cwiki.apache.org/OFBADMIN/mini-language-reference.html#Mini-languageReference-{{%3Cmakenextseqid%3E}}">Mini-language Reference</a>
  */
 public final class MakeNextSeqId extends MethodOperation {
 

Modified: ofbiz/trunk/framework/minilang/src/org/ofbiz/minilang/method/entityops/MakeValue.java
URL: http://svn.apache.org/viewvc/ofbiz/trunk/framework/minilang/src/org/ofbiz/minilang/method/entityops/MakeValue.java?rev=1356836&r1=1356835&r2=1356836&view=diff
==============================================================================
--- ofbiz/trunk/framework/minilang/src/org/ofbiz/minilang/method/entityops/MakeValue.java (original)
+++ ofbiz/trunk/framework/minilang/src/org/ofbiz/minilang/method/entityops/MakeValue.java Tue Jul  3 17:08:16 2012
@@ -34,6 +34,8 @@ import org.w3c.dom.Element;
 
 /**
  * Implements the &lt;make-value&gt; element.
+ * 
+ * @see <a href="https://cwiki.apache.org/OFBADMIN/mini-language-reference.html#Mini-languageReference-{{%3Cmakevalue%3E}}">Mini-language Reference</a>
  */
 public final class MakeValue extends MethodOperation {
 
@@ -58,7 +60,7 @@ public final class MakeValue extends Met
     public boolean exec(MethodContext methodContext) throws MiniLangException {
         String entityName = entityNameFse.expandString(methodContext.getEnvMap());
         if (entityName.isEmpty()) {
-            throw new MiniLangRuntimeException("Entity name not found: " + entityNameFse, this);
+            throw new MiniLangRuntimeException("Entity name not found.", this);
         }
         valueFma.put(methodContext.getEnvMap(), methodContext.getDelegator().makeValidValue(entityName, mapFma.get(methodContext.getEnvMap())));
         return true;

Modified: ofbiz/trunk/framework/minilang/src/org/ofbiz/minilang/method/entityops/OrderValueList.java
URL: http://svn.apache.org/viewvc/ofbiz/trunk/framework/minilang/src/org/ofbiz/minilang/method/entityops/OrderValueList.java?rev=1356836&r1=1356835&r2=1356836&view=diff
==============================================================================
--- ofbiz/trunk/framework/minilang/src/org/ofbiz/minilang/method/entityops/OrderValueList.java (original)
+++ ofbiz/trunk/framework/minilang/src/org/ofbiz/minilang/method/entityops/OrderValueList.java Tue Jul  3 17:08:16 2012
@@ -32,6 +32,8 @@ import org.w3c.dom.Element;
 
 /**
  * Implements the &lt;order-value-list&gt; element.
+ * 
+ * @see <a href="https://cwiki.apache.org/OFBADMIN/mini-language-reference.html#Mini-languageReference-{{%3Cordervaluelist%3E}}">Mini-language Reference</a>
  */
 public final class OrderValueList extends MethodOperation {
 

Modified: ofbiz/trunk/framework/minilang/src/org/ofbiz/minilang/method/entityops/RefreshValue.java
URL: http://svn.apache.org/viewvc/ofbiz/trunk/framework/minilang/src/org/ofbiz/minilang/method/entityops/RefreshValue.java?rev=1356836&r1=1356835&r2=1356836&view=diff
==============================================================================
--- ofbiz/trunk/framework/minilang/src/org/ofbiz/minilang/method/entityops/RefreshValue.java (original)
+++ ofbiz/trunk/framework/minilang/src/org/ofbiz/minilang/method/entityops/RefreshValue.java Tue Jul  3 17:08:16 2012
@@ -33,6 +33,8 @@ import org.w3c.dom.Element;
 
 /**
  * Implements the &lt;refresh-value&gt; element.
+ * 
+ * @see <a href="https://cwiki.apache.org/OFBADMIN/mini-language-reference.html#Mini-languageReference-{{%3Crefreshvalue%3E}}">Mini-language Reference</a>
  */
 public final class RefreshValue extends MethodOperation {
 

Modified: ofbiz/trunk/framework/minilang/src/org/ofbiz/minilang/method/entityops/RemoveByAnd.java
URL: http://svn.apache.org/viewvc/ofbiz/trunk/framework/minilang/src/org/ofbiz/minilang/method/entityops/RemoveByAnd.java?rev=1356836&r1=1356835&r2=1356836&view=diff
==============================================================================
--- ofbiz/trunk/framework/minilang/src/org/ofbiz/minilang/method/entityops/RemoveByAnd.java (original)
+++ ofbiz/trunk/framework/minilang/src/org/ofbiz/minilang/method/entityops/RemoveByAnd.java Tue Jul  3 17:08:16 2012
@@ -34,6 +34,8 @@ import org.w3c.dom.Element;
 
 /**
  * Implements the &lt;remove-by-and&gt; element.
+ * 
+ * @see <a href="https://cwiki.apache.org/OFBADMIN/mini-language-reference.html#Mini-languageReference-{{%3Cremovebyand%3E}}">Mini-language Reference</a>
  */
 public final class RemoveByAnd extends MethodOperation {
 

Modified: ofbiz/trunk/framework/minilang/src/org/ofbiz/minilang/method/entityops/RemoveList.java
URL: http://svn.apache.org/viewvc/ofbiz/trunk/framework/minilang/src/org/ofbiz/minilang/method/entityops/RemoveList.java?rev=1356836&r1=1356835&r2=1356836&view=diff
==============================================================================
--- ofbiz/trunk/framework/minilang/src/org/ofbiz/minilang/method/entityops/RemoveList.java (original)
+++ ofbiz/trunk/framework/minilang/src/org/ofbiz/minilang/method/entityops/RemoveList.java Tue Jul  3 17:08:16 2012
@@ -35,6 +35,8 @@ import org.w3c.dom.Element;
 
 /**
  * Implements the &lt;remove-list&gt; element.
+ * 
+ * @see <a href="https://cwiki.apache.org/OFBADMIN/mini-language-reference.html#Mini-languageReference-{{%3Cremovelist%3E}}">Mini-language Reference</a>
  */
 public final class RemoveList extends MethodOperation {
 

Modified: ofbiz/trunk/framework/minilang/src/org/ofbiz/minilang/method/entityops/RemoveRelated.java
URL: http://svn.apache.org/viewvc/ofbiz/trunk/framework/minilang/src/org/ofbiz/minilang/method/entityops/RemoveRelated.java?rev=1356836&r1=1356835&r2=1356836&view=diff
==============================================================================
--- ofbiz/trunk/framework/minilang/src/org/ofbiz/minilang/method/entityops/RemoveRelated.java (original)
+++ ofbiz/trunk/framework/minilang/src/org/ofbiz/minilang/method/entityops/RemoveRelated.java Tue Jul  3 17:08:16 2012
@@ -34,6 +34,8 @@ import org.w3c.dom.Element;
 
 /**
  * Implements the &lt;remove-related&gt; element.
+ * 
+ * @see <a href="https://cwiki.apache.org/OFBADMIN/mini-language-reference.html#Mini-languageReference-{{%3Cremoverelated%3E}}">Mini-language Reference</a>
  */
 public final class RemoveRelated extends MethodOperation {
 

Modified: ofbiz/trunk/framework/minilang/src/org/ofbiz/minilang/method/entityops/RemoveValue.java
URL: http://svn.apache.org/viewvc/ofbiz/trunk/framework/minilang/src/org/ofbiz/minilang/method/entityops/RemoveValue.java?rev=1356836&r1=1356835&r2=1356836&view=diff
==============================================================================
--- ofbiz/trunk/framework/minilang/src/org/ofbiz/minilang/method/entityops/RemoveValue.java (original)
+++ ofbiz/trunk/framework/minilang/src/org/ofbiz/minilang/method/entityops/RemoveValue.java Tue Jul  3 17:08:16 2012
@@ -33,6 +33,8 @@ import org.w3c.dom.Element;
 
 /**
  * Implements the &lt;remove-value&gt; element.
+ * 
+ * @see <a href="https://cwiki.apache.org/OFBADMIN/mini-language-reference.html#Mini-languageReference-{{%3Cremovevalue%3E}}">Mini-language Reference</a>
  */
 public final class RemoveValue extends MethodOperation {
 

Modified: ofbiz/trunk/framework/minilang/src/org/ofbiz/minilang/method/entityops/SequencedIdToEnv.java
URL: http://svn.apache.org/viewvc/ofbiz/trunk/framework/minilang/src/org/ofbiz/minilang/method/entityops/SequencedIdToEnv.java?rev=1356836&r1=1356835&r2=1356836&view=diff
==============================================================================
--- ofbiz/trunk/framework/minilang/src/org/ofbiz/minilang/method/entityops/SequencedIdToEnv.java (original)
+++ ofbiz/trunk/framework/minilang/src/org/ofbiz/minilang/method/entityops/SequencedIdToEnv.java Tue Jul  3 17:08:16 2012
@@ -29,6 +29,8 @@ import org.w3c.dom.Element;
 
 /**
  * Implements the &lt;sequenced-id&gt; element.
+ * 
+ * @see <a href="https://cwiki.apache.org/OFBADMIN/mini-language-reference.html#Mini-languageReference-{{%3Csequencedid%3E}}">Mini-language Reference</a>
  */
 public final class SequencedIdToEnv extends MethodOperation {
 

Modified: ofbiz/trunk/framework/minilang/src/org/ofbiz/minilang/method/entityops/SetCurrentUserLogin.java
URL: http://svn.apache.org/viewvc/ofbiz/trunk/framework/minilang/src/org/ofbiz/minilang/method/entityops/SetCurrentUserLogin.java?rev=1356836&r1=1356835&r2=1356836&view=diff
==============================================================================
--- ofbiz/trunk/framework/minilang/src/org/ofbiz/minilang/method/entityops/SetCurrentUserLogin.java (original)
+++ ofbiz/trunk/framework/minilang/src/org/ofbiz/minilang/method/entityops/SetCurrentUserLogin.java Tue Jul  3 17:08:16 2012
@@ -30,6 +30,8 @@ import org.w3c.dom.Element;
 
 /**
  * Implements the &lt;set-current-user-login&gt; element.
+ * 
+ * @see <a href="https://cwiki.apache.org/OFBADMIN/mini-language-reference.html#Mini-languageReference-{{%3Csetcurrentuserlogin%3E}}">Mini-language Reference</a>
  */
 public final class SetCurrentUserLogin extends MethodOperation {
 

Modified: ofbiz/trunk/framework/minilang/src/org/ofbiz/minilang/method/entityops/SetNonpkFields.java
URL: http://svn.apache.org/viewvc/ofbiz/trunk/framework/minilang/src/org/ofbiz/minilang/method/entityops/SetNonpkFields.java?rev=1356836&r1=1356835&r2=1356836&view=diff
==============================================================================
--- ofbiz/trunk/framework/minilang/src/org/ofbiz/minilang/method/entityops/SetNonpkFields.java (original)
+++ ofbiz/trunk/framework/minilang/src/org/ofbiz/minilang/method/entityops/SetNonpkFields.java Tue Jul  3 17:08:16 2012
@@ -33,6 +33,8 @@ import org.w3c.dom.Element;
 
 /**
  * Implements the &lt;set-nonpk-fields&gt; element.
+ * 
+ * @see <a href="https://cwiki.apache.org/OFBADMIN/mini-language-reference.html#Mini-languageReference-{{%3Csetnonpkfields%3E}}">Mini-language Reference</a>
  */
 public final class SetNonpkFields extends MethodOperation {
 

Modified: ofbiz/trunk/framework/minilang/src/org/ofbiz/minilang/method/entityops/SetPkFields.java
URL: http://svn.apache.org/viewvc/ofbiz/trunk/framework/minilang/src/org/ofbiz/minilang/method/entityops/SetPkFields.java?rev=1356836&r1=1356835&r2=1356836&view=diff
==============================================================================
--- ofbiz/trunk/framework/minilang/src/org/ofbiz/minilang/method/entityops/SetPkFields.java (original)
+++ ofbiz/trunk/framework/minilang/src/org/ofbiz/minilang/method/entityops/SetPkFields.java Tue Jul  3 17:08:16 2012
@@ -33,6 +33,8 @@ import org.w3c.dom.Element;
 
 /**
  * Implements the &lt;set-pk-fields&gt; element.
+ * 
+ * @see <a href="https://cwiki.apache.org/OFBADMIN/mini-language-reference.html#Mini-languageReference-{{%3Csetpkfields%3E}}">Mini-language Reference</a>
  */
 public final class SetPkFields extends MethodOperation {
 

Modified: ofbiz/trunk/framework/minilang/src/org/ofbiz/minilang/method/entityops/StoreList.java
URL: http://svn.apache.org/viewvc/ofbiz/trunk/framework/minilang/src/org/ofbiz/minilang/method/entityops/StoreList.java?rev=1356836&r1=1356835&r2=1356836&view=diff
==============================================================================
--- ofbiz/trunk/framework/minilang/src/org/ofbiz/minilang/method/entityops/StoreList.java (original)
+++ ofbiz/trunk/framework/minilang/src/org/ofbiz/minilang/method/entityops/StoreList.java Tue Jul  3 17:08:16 2012
@@ -35,6 +35,8 @@ import org.w3c.dom.Element;
 
 /**
  * Implements the &lt;store-list&gt; element.
+ * 
+ * @see <a href="https://cwiki.apache.org/OFBADMIN/mini-language-reference.html#Mini-languageReference-{{%3Cstorelist%3E}}">Mini-language Reference</a>
  */
 public final class StoreList extends MethodOperation {
 

Modified: ofbiz/trunk/framework/minilang/src/org/ofbiz/minilang/method/entityops/StoreValue.java
URL: http://svn.apache.org/viewvc/ofbiz/trunk/framework/minilang/src/org/ofbiz/minilang/method/entityops/StoreValue.java?rev=1356836&r1=1356835&r2=1356836&view=diff
==============================================================================
--- ofbiz/trunk/framework/minilang/src/org/ofbiz/minilang/method/entityops/StoreValue.java (original)
+++ ofbiz/trunk/framework/minilang/src/org/ofbiz/minilang/method/entityops/StoreValue.java Tue Jul  3 17:08:16 2012
@@ -33,6 +33,8 @@ import org.w3c.dom.Element;
 
 /**
  * Implements the &lt;store-value&gt; element.
+ * 
+ * @see <a href="https://cwiki.apache.org/OFBADMIN/mini-language-reference.html#Mini-languageReference-{{%3Cstorevalue%3E}}">Mini-language Reference</a>
  */
 public final class StoreValue extends MethodOperation {
 

Modified: ofbiz/trunk/framework/minilang/src/org/ofbiz/minilang/method/entityops/TransactionBegin.java
URL: http://svn.apache.org/viewvc/ofbiz/trunk/framework/minilang/src/org/ofbiz/minilang/method/entityops/TransactionBegin.java?rev=1356836&r1=1356835&r2=1356836&view=diff
==============================================================================
--- ofbiz/trunk/framework/minilang/src/org/ofbiz/minilang/method/entityops/TransactionBegin.java (original)
+++ ofbiz/trunk/framework/minilang/src/org/ofbiz/minilang/method/entityops/TransactionBegin.java Tue Jul  3 17:08:16 2012
@@ -31,6 +31,8 @@ import org.w3c.dom.Element;
 
 /**
  * Implements the &lt;transaction-begin&gt; element.
+ * 
+ * @see <a href="https://cwiki.apache.org/OFBADMIN/mini-language-reference.html#Mini-languageReference-{{%3Ctransactionbegin%3E}}">Mini-language Reference</a>
  */
 public final class TransactionBegin extends MethodOperation {
 

Modified: ofbiz/trunk/framework/minilang/src/org/ofbiz/minilang/method/entityops/TransactionCommit.java
URL: http://svn.apache.org/viewvc/ofbiz/trunk/framework/minilang/src/org/ofbiz/minilang/method/entityops/TransactionCommit.java?rev=1356836&r1=1356835&r2=1356836&view=diff
==============================================================================
--- ofbiz/trunk/framework/minilang/src/org/ofbiz/minilang/method/entityops/TransactionCommit.java (original)
+++ ofbiz/trunk/framework/minilang/src/org/ofbiz/minilang/method/entityops/TransactionCommit.java Tue Jul  3 17:08:16 2012
@@ -31,6 +31,8 @@ import org.w3c.dom.Element;
 
 /**
  * Implements the &lt;transaction-commit&gt; element.
+ * 
+ * @see <a href="https://cwiki.apache.org/OFBADMIN/mini-language-reference.html#Mini-languageReference-{{%3Ctransactioncommit%3E}}">Mini-language Reference</a>
  */
 public final class TransactionCommit extends MethodOperation {
 

Modified: ofbiz/trunk/framework/minilang/src/org/ofbiz/minilang/method/entityops/TransactionRollback.java
URL: http://svn.apache.org/viewvc/ofbiz/trunk/framework/minilang/src/org/ofbiz/minilang/method/entityops/TransactionRollback.java?rev=1356836&r1=1356835&r2=1356836&view=diff
==============================================================================
--- ofbiz/trunk/framework/minilang/src/org/ofbiz/minilang/method/entityops/TransactionRollback.java (original)
+++ ofbiz/trunk/framework/minilang/src/org/ofbiz/minilang/method/entityops/TransactionRollback.java Tue Jul  3 17:08:16 2012
@@ -31,6 +31,8 @@ import org.w3c.dom.Element;
 
 /**
  * Implements the &lt;transaction-rollback&gt; element.
+ * 
+ * @see <a href="https://cwiki.apache.org/OFBADMIN/mini-language-reference.html#Mini-languageReference-{{%3Ctransactionrollback%3E}}">Mini-language Reference</a>
  */
 public final class TransactionRollback extends MethodOperation {
 

Modified: ofbiz/trunk/framework/minilang/src/org/ofbiz/minilang/method/envops/AddError.java
URL: http://svn.apache.org/viewvc/ofbiz/trunk/framework/minilang/src/org/ofbiz/minilang/method/envops/AddError.java?rev=1356836&r1=1356835&r2=1356836&view=diff
==============================================================================
--- ofbiz/trunk/framework/minilang/src/org/ofbiz/minilang/method/envops/AddError.java (original)
+++ ofbiz/trunk/framework/minilang/src/org/ofbiz/minilang/method/envops/AddError.java Tue Jul  3 17:08:16 2012
@@ -32,8 +32,10 @@ import org.ofbiz.minilang.method.MethodO
 import org.w3c.dom.Element;
 
 /**
- * Adds an error message to an error message list.
- */
+ * Implements the &lt;add-error&gt; element.
+ * 
+ * @see <a href="https://cwiki.apache.org/OFBADMIN/mini-language-reference.html#Mini-languageReference-{{%3Cadderror%3E}}">Mini-language Reference</a>
+*/
 public final class AddError extends MethodOperation {
 
     private final FlexibleMapAccessor<List<String>> errorListFma;
@@ -81,11 +83,16 @@ public final class AddError extends Meth
         return sb.toString();
     }
 
+    /**
+     * A factory for the &lt;add-error&gt; element.
+    */
     public static final class AddErrorFactory implements Factory<AddError> {
+        @Override
         public AddError createMethodOperation(Element element, SimpleMethod simpleMethod) throws MiniLangException {
             return new AddError(element, simpleMethod);
         }
 
+        @Override
         public String getName() {
             return "add-error";
         }

Modified: ofbiz/trunk/framework/minilang/src/org/ofbiz/minilang/method/envops/Break.java
URL: http://svn.apache.org/viewvc/ofbiz/trunk/framework/minilang/src/org/ofbiz/minilang/method/envops/Break.java?rev=1356836&r1=1356835&r2=1356836&view=diff
==============================================================================
--- ofbiz/trunk/framework/minilang/src/org/ofbiz/minilang/method/envops/Break.java (original)
+++ ofbiz/trunk/framework/minilang/src/org/ofbiz/minilang/method/envops/Break.java Tue Jul  3 17:08:16 2012
@@ -25,7 +25,9 @@ import org.ofbiz.minilang.method.MethodO
 import org.w3c.dom.Element;
 
 /**
- * Causes script execution to exit the nearest loop element.
+ * Implements the &lt;break&gt; element.
+ * 
+ * @see <a href="https://cwiki.apache.org/OFBADMIN/mini-language-reference.html#Mini-languageReference-{{%3Cbreak%3E}}">Mini-language Reference</a>
  */
 public class Break extends MethodOperation {
 
@@ -60,11 +62,16 @@ public class Break extends MethodOperati
         }
     }
 
+    /**
+     * A factory for the &lt;break&gt; element.
+     */
     public static final class BreakFactory implements Factory<Break> {
+        @Override
         public Break createMethodOperation(Element element, SimpleMethod simpleMethod) throws MiniLangException {
             return new Break(element, simpleMethod);
         }
 
+        @Override
         public String getName() {
             return "break";
         }

Modified: ofbiz/trunk/framework/minilang/src/org/ofbiz/minilang/method/envops/CheckId.java
URL: http://svn.apache.org/viewvc/ofbiz/trunk/framework/minilang/src/org/ofbiz/minilang/method/envops/CheckId.java?rev=1356836&r1=1356835&r2=1356836&view=diff
==============================================================================
--- ofbiz/trunk/framework/minilang/src/org/ofbiz/minilang/method/envops/CheckId.java (original)
+++ ofbiz/trunk/framework/minilang/src/org/ofbiz/minilang/method/envops/CheckId.java Tue Jul  3 17:08:16 2012
@@ -35,7 +35,9 @@ import org.ofbiz.minilang.method.MethodO
 import org.w3c.dom.Element;
 
 /**
- * If the given ID field is not valid the fail-message or fail-property sub-elements are used to add a message to the error-list.
+ * Implements the &lt;check-id&gt; element.
+ * 
+ * @see <a href="https://cwiki.apache.org/OFBADMIN/mini-language-reference.html#Mini-languageReference-{{%3Ccheckid%3E}}">Mini-language Reference</a>
  */
 public final class CheckId extends MethodOperation {
 
@@ -51,6 +53,7 @@ public final class CheckId extends Metho
         super(element, simpleMethod);
         if (MiniLangValidate.validationOn()) {
             MiniLangValidate.attributeNames(simpleMethod, element, "field", "error-list-name");
+            MiniLangValidate.requiredAttributes(simpleMethod, element, "field");
             MiniLangValidate.constantAttributes(simpleMethod, element, "error-list-name");
             MiniLangValidate.expressionAttributes(simpleMethod, element, "field");
             MiniLangValidate.childElements(simpleMethod, element, "fail-message", "fail-property");
@@ -134,11 +137,16 @@ public final class CheckId extends Metho
         return sb.toString();
     }
 
+    /**
+     * A factory for the &lt;check-id&gt; element.
+     */
     public static final class CheckIdFactory implements Factory<CheckId> {
+        @Override
         public CheckId createMethodOperation(Element element, SimpleMethod simpleMethod) throws MiniLangException {
             return new CheckId(element, simpleMethod);
         }
 
+        @Override
         public String getName() {
             return "check-id";
         }

Modified: ofbiz/trunk/framework/minilang/src/org/ofbiz/minilang/method/envops/ClearField.java
URL: http://svn.apache.org/viewvc/ofbiz/trunk/framework/minilang/src/org/ofbiz/minilang/method/envops/ClearField.java?rev=1356836&r1=1356835&r2=1356836&view=diff
==============================================================================
--- ofbiz/trunk/framework/minilang/src/org/ofbiz/minilang/method/envops/ClearField.java (original)
+++ ofbiz/trunk/framework/minilang/src/org/ofbiz/minilang/method/envops/ClearField.java Tue Jul  3 17:08:16 2012
@@ -28,6 +28,8 @@ import org.w3c.dom.Element;
 
 /**
  * Implements the &lt;clear-field&gt; element.
+ * 
+ * @see <a href="https://cwiki.apache.org/OFBADMIN/mini-language-reference.html#Mini-languageReference-{{<clearfield>}}">Mini-language Reference</a>
  */
 public final class ClearField extends MethodOperation {
 

Modified: ofbiz/trunk/framework/minilang/src/org/ofbiz/minilang/method/envops/Continue.java
URL: http://svn.apache.org/viewvc/ofbiz/trunk/framework/minilang/src/org/ofbiz/minilang/method/envops/Continue.java?rev=1356836&r1=1356835&r2=1356836&view=diff
==============================================================================
--- ofbiz/trunk/framework/minilang/src/org/ofbiz/minilang/method/envops/Continue.java (original)
+++ ofbiz/trunk/framework/minilang/src/org/ofbiz/minilang/method/envops/Continue.java Tue Jul  3 17:08:16 2012
@@ -25,7 +25,9 @@ import org.ofbiz.minilang.method.MethodO
 import org.w3c.dom.Element;
 
 /**
- * Causes script execution to return to the beginning of the nearest enclosing loop element.
+ * Implements the &lt;continue&gt; element.
+ * 
+ * @see <a href="https://cwiki.apache.org/OFBADMIN/mini-language-reference.html#Mini-languageReference-{{%3Ccontinue%3E}}">Mini-language Reference</a>
  */
 public class Continue extends MethodOperation {
 
@@ -60,11 +62,16 @@ public class Continue extends MethodOper
         }
     }
 
+    /**
+     * A factory for the &lt;continue&gt; element.
+     */
     public static final class ContinueFactory implements Factory<Continue> {
+        @Override
         public Continue createMethodOperation(Element element, SimpleMethod simpleMethod) throws MiniLangException {
             return new Continue(element, simpleMethod);
         }
 
+        @Override
         public String getName() {
             return "continue";
         }