0011824: Dependencies only show name and not version

diff -r 9bc2be16a52a src/org/openbravo/erpCommon/ad_forms/ModuleManagement.java
--- a/src/org/openbravo/erpCommon/ad_forms/ModuleManagement.java	Fri Dec 18 19:10:04 2009 +0530
+++ b/src/org/openbravo/erpCommon/ad_forms/ModuleManagement.java	Sun Dec 20 18:48:08 2009 +0100
@@ -470,10 +470,10 @@
     xmlDocument.setParameter("license", module.getLicenseType());
 
     if (dependencies != null && dependencies.length > 0)
-      xmlDocument.setData("dependencies", FieldProviderFactory.getFieldProviderArray(dependencies));
+      xmlDocument.setData("dependencies", formatDeps4Display(dependencies));
 
     if (includes != null && includes.length > 0)
-      xmlDocument.setData("includes", FieldProviderFactory.getFieldProviderArray(includes));
+      xmlDocument.setData("includes", formatDeps4Display(includes));
 
     response.setContentType("text/html; charset=UTF-8");
     final PrintWriter out = response.getWriter();
@@ -481,6 +481,21 @@
     out.close();
   }
 
+  private static FieldProvider[] formatDeps4Display(ModuleDependency[] deps) {
+    HashMap<String, String>[] res = new HashMap[deps.length];
+
+    for (int i = 0; i < deps.length; i++) {
+      res[i] = new HashMap<String, String>();
+      res[i].put("moduleName", getDisplayString(deps[i]));
+    }
+    return FieldProviderFactory.getFieldProviderArray(res);
+  }
+
+  private static String getDisplayString(ModuleDependency dep) {
+    return dep.getModuleName() + " (" + ">=" + dep.getVersionStart() + " & <="
+        + dep.getVersionEnd() + ")";
+  }
+
   /**
    * A decision needs to be made when executing this method/pop-up:
    *