Author: doogie
Date: Sat Aug 2 16:31:04 2008 New Revision: 682078 URL: http://svn.apache.org/viewvc?rev=682078&view=rev Log: Generics galore. Modified: ofbiz/trunk/framework/webtools/src/org/ofbiz/webtools/GenericWebEvent.java ofbiz/trunk/framework/webtools/src/org/ofbiz/webtools/WebToolsServices.java ofbiz/trunk/framework/webtools/src/org/ofbiz/webtools/artifactinfo/ArtifactInfoFactory.java ofbiz/trunk/framework/webtools/src/org/ofbiz/webtools/artifactinfo/ControllerRequestArtifactInfo.java ofbiz/trunk/framework/webtools/src/org/ofbiz/webtools/print/FoPrintServerEvents.java Modified: ofbiz/trunk/framework/webtools/src/org/ofbiz/webtools/GenericWebEvent.java URL: http://svn.apache.org/viewvc/ofbiz/trunk/framework/webtools/src/org/ofbiz/webtools/GenericWebEvent.java?rev=682078&r1=682077&r2=682078&view=diff ============================================================================== --- ofbiz/trunk/framework/webtools/src/org/ofbiz/webtools/GenericWebEvent.java (original) +++ ofbiz/trunk/framework/webtools/src/org/ofbiz/webtools/GenericWebEvent.java Sat Aug 2 16:31:04 2008 @@ -104,7 +104,7 @@ // check permissions before moving on... if (!security.hasEntityPermission("ENTITY_DATA", "_" + updateMode, request.getSession()) && !security.hasEntityPermission(entity.getPlainTableName(), "_" + updateMode, request.getSession())) { - Map messageMap = UtilMisc.toMap("updateMode", updateMode, "entityName", entity.getEntityName(), "entityPlainTableName", entity.getPlainTableName()); + Map<String, String> messageMap = UtilMisc.toMap("updateMode", updateMode, "entityName", entity.getEntityName(), "entityPlainTableName", entity.getPlainTableName()); String errMsg = UtilProperties.getMessage(GenericWebEvent.err_resource, "genericWebEvent.not_sufficient_permissions_01", messageMap, locale); errMsg += UtilProperties.getMessage(GenericWebEvent.err_resource, "genericWebEvent.not_sufficient_permissions_02", messageMap, locale) + "."; @@ -117,17 +117,17 @@ // get the primary key parameters... String errMsgPk = ""; - Iterator pksIter = entity.getPksIterator(); + Iterator<ModelField> pksIter = entity.getPksIterator(); while (pksIter.hasNext()) { String errMsg = ""; - ModelField field = (ModelField) pksIter.next(); + ModelField field = pksIter.next(); ModelFieldType type = null; try { type = delegator.getEntityFieldType(entity, field.getType()); } catch (GenericEntityException e) { Debug.logWarning(e, module); - Map messageMap = UtilMisc.toMap("fieldType", field.getType()); + Map<String, String> messageMap = UtilMisc.toMap("fieldType", field.getType()); errMsg += UtilProperties.getMessage(GenericWebEvent.err_resource, "genericWebEvent.fatal_error_param", messageMap, locale) + "."; } @@ -136,7 +136,7 @@ try { findByEntity.setString(field.getName(), fval); } catch (Exception e) { - Map messageMap = UtilMisc.toMap("fval", fval); + Map<String, String> messageMap = UtilMisc.toMap("fval", fval); errMsg = errMsg + "<li>" + field.getColName() + UtilProperties.getMessage(GenericWebEvent.err_resource, "genericWebEvent.conversion_failed", messageMap, locale) + type.getJavaType() + "."; Debug.logWarning("[updateGeneric] " + field.getColName() + " conversion failed: \"" + fval + "\" is not a valid " + type.getJavaType() + "; entityName: " + entityName, module); } @@ -166,16 +166,16 @@ // get the non-primary key parameters String errMsgNonPk = ""; - Iterator nopksIter = entity.getNopksIterator(); + Iterator<ModelField> nopksIter = entity.getNopksIterator(); while (nopksIter.hasNext()) { - ModelField field = (ModelField) nopksIter.next(); + ModelField field = nopksIter.next(); ModelFieldType type = null; try { type = delegator.getEntityFieldType(entity, field.getType()); } catch (GenericEntityException e) { Debug.logWarning(e, module); - Map messageMap = UtilMisc.toMap("fieldType", field.getType()); + Map<String, String> messageMap = UtilMisc.toMap("fieldType", field.getType()); errMsgNonPk += UtilProperties.getMessage(GenericWebEvent.err_resource, "genericWebEvent.fatal_error_param", messageMap, locale) + "."; } @@ -184,7 +184,7 @@ try { findByEntity.setString(field.getName(), fval); } catch (Exception e) { - Map messageMap = UtilMisc.toMap("fval", fval); + Map<String, String> messageMap = UtilMisc.toMap("fval", fval); errMsgNonPk += field.getColName() + UtilProperties.getMessage(GenericWebEvent.err_resource, "genericWebEvent.conversion_failed", messageMap, locale) + type.getJavaType() + "."; Debug.logWarning("[updateGeneric] " + field.getColName() + " conversion failed: \"" + fval + "\" is not a valid " + type.getJavaType() + "; entityName: " + entityName, module); @@ -214,7 +214,7 @@ return "error"; } if (tempEntity != null) { - Map messageMap = UtilMisc.toMap("primaryKey", findByEntity.getPrimaryKey().toString()); + Map<String, String> messageMap = UtilMisc.toMap("primaryKey", findByEntity.getPrimaryKey().toString()); String errMsg = entity.getEntityName() + UtilProperties.getMessage(GenericWebEvent.err_resource, "genericWebEvent.already_exists_pk", messageMap, locale)+ "."; Debug.logWarning("[updateGeneric] " + entity.getEntityName() + " already exists with primary key: " + findByEntity.getPrimaryKey().toString() + "; please change.", module); } @@ -222,9 +222,9 @@ // Validate parameters... String errMsgParam = ""; - Iterator fieldIter = entity.getFieldsIterator(); + Iterator<ModelField> fieldIter = entity.getFieldsIterator(); while (fieldIter.hasNext()) { - ModelField field = (ModelField) fieldIter.next(); + ModelField field = fieldIter.next(); for (int j = 0; j < field.getValidatorsSize(); j++) { String curValidate = field.getValidator(j); @@ -238,7 +238,7 @@ className = curValidate.substring(0, curValidate.lastIndexOf('.')); methodName = curValidate.substring(curValidate.lastIndexOf('.') + 1); } - Class valClass; + Class<?> valClass; try { ClassLoader loader = Thread.currentThread().getContextClassLoader(); @@ -296,7 +296,7 @@ try { value = delegator.create(findByEntity.getEntityName(), findByEntity.getAllFields()); } catch (GenericEntityException e) { - Map messageMap = UtilMisc.toMap("entityName", entity.getEntityName()); + Map<String, String> messageMap = UtilMisc.toMap("entityName", entity.getEntityName()); String errMsg = UtilProperties.getMessage(GenericWebEvent.err_resource, "genericWebEvent.creation_param_failed", messageMap, locale)+ ": " + findByEntity.toString() + ": " + e.toString(); Debug.logWarning(e, errMsg, module); request.setAttribute("_ERROR_MESSAGE_", errMsg); @@ -308,14 +308,14 @@ try { value.store(); } catch (GenericEntityException e) { - Map messageMap = UtilMisc.toMap("entityName", entity.getEntityName()); + Map<String, String> messageMap = UtilMisc.toMap("entityName", entity.getEntityName()); String errMsg = UtilProperties.getMessage(GenericWebEvent.err_resource, "genericWebEvent.update_of_param_failed", messageMap, locale)+ ": " + value.toString() + ": " + e.toString(); Debug.logWarning(e, errMsg, module); request.setAttribute("_ERROR_MESSAGE_", errMsg); return "error"; } } else { - Map messageMap = UtilMisc.toMap("updateMode", updateMode); + Map<String, String> messageMap = UtilMisc.toMap("updateMode", updateMode); String errMsg = UtilProperties.getMessage(GenericWebEvent.err_resource, "genericWebEvent.update_of_param_failed", messageMap, locale)+ "."; request.setAttribute("_ERROR_MESSAGE_", errMsg); Modified: ofbiz/trunk/framework/webtools/src/org/ofbiz/webtools/WebToolsServices.java URL: http://svn.apache.org/viewvc/ofbiz/trunk/framework/webtools/src/org/ofbiz/webtools/WebToolsServices.java?rev=682078&r1=682077&r2=682078&view=diff ============================================================================== --- ofbiz/trunk/framework/webtools/src/org/ofbiz/webtools/WebToolsServices.java (original) +++ ofbiz/trunk/framework/webtools/src/org/ofbiz/webtools/WebToolsServices.java Sat Aug 2 16:31:04 2008 @@ -107,7 +107,7 @@ public static final String module = WebToolsServices.class.getName(); - public static Map entityImport(DispatchContext dctx, Map context) { + public static Map<String, Object> entityImport(DispatchContext dctx, Map<String, ? extends Object> context) { GenericValue userLogin = (GenericValue) context.get("userLogin"); LocalDispatcher dispatcher = dctx.getDispatcher(); @@ -173,7 +173,7 @@ try { Configuration conf = org.ofbiz.base.util.template.FreeMarkerWorker.getDefaultOfbizConfig(); template = new Template("FMImportFilter", templateReader, conf); - Map fmcontext = new HashMap(); + Map<String, Object> fmcontext = FastMap.newInstance(); NodeModel nodeModel = NodeModel.parse(ins); fmcontext.put("doc", nodeModel); @@ -193,7 +193,7 @@ // ############################# if (s != null || fulltext != null || url != null) { try{ - Map inputMap = UtilMisc.toMap("mostlyInserts", mostlyInserts, + Map<String, Object> inputMap = UtilMisc.toMap("mostlyInserts", mostlyInserts, "createDummyFks", createDummyFks, "checkDataOnly", checkDataOnly, "maintainTimeStamps", maintainTimeStamps, @@ -208,7 +208,7 @@ inputMap.put("url", url); } } - Map outputMap = dispatcher.runSync("parseEntityXmlFile", inputMap); + Map<String, Object> outputMap = dispatcher.runSync("parseEntityXmlFile", inputMap); if (ServiceUtil.isError(outputMap)) { return ServiceUtil.returnError("ERROR: " + ServiceUtil.getErrorMessage(outputMap)); } else { @@ -223,15 +223,15 @@ } // send the notification - Map resp = UtilMisc.toMap("messages", messages); + Map<String, Object> resp = UtilMisc.toMap("messages", (Object) messages); return resp; } - public static Map entityImportDir(DispatchContext dctx, Map context) { + public static Map<String, Object> entityImportDir(DispatchContext dctx, Map<String, ? extends Object> context) { GenericValue userLogin = (GenericValue) context.get("userLogin"); LocalDispatcher dispatcher = dctx.getDispatcher(); - List messages = FastList.newInstance(); + List<String> messages = FastList.newInstance(); String path = (String) context.get("path"); String mostlyInserts = (String) context.get("mostlyInserts"); @@ -256,35 +256,33 @@ if (baseDir.isDirectory() && baseDir.canRead()) { File[] fileArray = baseDir.listFiles(); - FastList files = FastList.newInstance(); - for (int a=0; a<fileArray.length; a++){ - if (fileArray[a].getName().toUpperCase().endsWith("XML")) { - files.add(fileArray[a]); + FastList<File> files = FastList.newInstance(); + for (File file: fileArray) { + if (file.getName().toUpperCase().endsWith("XML")) { + files.add(file); } } int passes=0; int initialListSize = files.size(); int lastUnprocessedFilesCount = 0; - FastList unprocessedFiles = FastList.newInstance(); + FastList<File> unprocessedFiles = FastList.newInstance(); while (files.size()>0 && files.size() != lastUnprocessedFilesCount) { lastUnprocessedFilesCount = files.size(); unprocessedFiles = FastList.newInstance(); - Iterator filesItr = files.iterator(); - while (filesItr.hasNext()) { - Map parseEntityXmlFileArgs = UtilMisc.toMap("mostlyInserts", mostlyInserts, + for (File f: files) { + Map<String, Object> parseEntityXmlFileArgs = UtilMisc.toMap("mostlyInserts", mostlyInserts, "createDummyFks", createDummyFks, "checkDataOnly", checkDataOnly, "maintainTimeStamps", maintainTimeStamps, "txTimeout", txTimeout, "userLogin", userLogin); - File f = (File) filesItr.next(); try { URL furl = f.toURI().toURL(); parseEntityXmlFileArgs.put("url", furl); - Map outputMap = dispatcher.runSync("parseEntityXmlFile", parseEntityXmlFileArgs); + Map<String, Object> outputMap = dispatcher.runSync("parseEntityXmlFile", parseEntityXmlFileArgs); Long numberRead = (Long) outputMap.get("rowProcessed"); messages.add("Got " + numberRead.longValue() + " entities from " + f); if (deleteFiles) { @@ -316,10 +314,8 @@ messages.add("Failed: " + lastUnprocessedFilesCount + " of " + initialListSize); messages.add("---------------------------------------"); messages.add("Failed Files:"); - Iterator unprocessedFilesItr = unprocessedFiles.iterator(); - while (unprocessedFilesItr.hasNext()) { - File file = (File) unprocessedFilesItr.next(); - messages.add("" + file); + for (File file: unprocessedFiles) { + messages.add(file.toString()); } } else { messages.add("path not found or can't be read"); @@ -328,11 +324,11 @@ messages.add("No path specified, doing nothing."); } // send the notification - Map resp = UtilMisc.toMap("messages", messages); + Map<String, Object> resp = UtilMisc.toMap("messages", (Object) messages); return resp; } - public static Map entityImportReaders(DispatchContext dctx, Map context) { + public static Map<String, Object> entityImportReaders(DispatchContext dctx, Map<String, Object> context) { String readers = (String) context.get("readers"); String overrideDelegator = (String) context.get("overrideDelegator"); String overrideGroup = (String) context.get("overrideGroup"); @@ -344,10 +340,10 @@ Integer txTimeoutInt = (Integer) context.get("txTimeout"); int txTimeout = txTimeoutInt != null ? txTimeoutInt.intValue() : -1; - List messages = FastList.newInstance(); + List<Object> messages = FastList.newInstance(); // parse the pass in list of readers to use - List readerNames = null; + List<String> readerNames = null; if (UtilValidate.isNotEmpty(readers) && !"none".equalsIgnoreCase(readers)) { if (readers.indexOf(",") == -1) { readerNames = FastList.newInstance(); @@ -366,7 +362,7 @@ } // get the reader name URLs first - List urlList = null; + List<URL> urlList = null; if (readerNames != null) { urlList = EntityDataLoader.getUrlList(helperName, readerNames); } else if (!"none".equalsIgnoreCase(readers)) { @@ -383,22 +379,18 @@ changedFormat.setMinimumIntegerDigits(5); changedFormat.setGroupingUsed(false); - List errorMessages = new LinkedList(); - List infoMessages = new LinkedList(); + List<Object> errorMessages = FastList.newInstance(); + List<String> infoMessages = FastList.newInstance(); int totalRowsChanged = 0; if (urlList != null && urlList.size() > 0) { messages.add("=-=-=-=-=-=-= Doing a data " + (checkDataOnly ? "check" : "load") + " with the following files:"); - Iterator urlIter = urlList.iterator(); - while (urlIter.hasNext()) { - URL dataUrl = (URL) urlIter.next(); + for (URL dataUrl: urlList) { messages.add(dataUrl.toExternalForm()); } messages.add("=-=-=-=-=-=-= Starting the data " + (checkDataOnly ? "check" : "load") + "..."); - urlIter = urlList.iterator(); - while (urlIter.hasNext()) { - URL dataUrl = (URL) urlIter.next(); + for (URL dataUrl: urlList) { try { int rowsChanged = 0; if (checkDataOnly) { @@ -437,12 +429,12 @@ messages.add("=-=-=-=-=-=-= Finished the data " + (checkDataOnly ? "check" : "load") + " with " + totalRowsChanged + " rows " + (checkDataOnly ? "checked" : "changed") + "."); - Map resultMap = ServiceUtil.returnSuccess(); + Map<String, Object> resultMap = ServiceUtil.returnSuccess(); resultMap.put("messages", messages); return resultMap; } - public static Map parseEntityXmlFile(DispatchContext dctx, Map context) { + public static Map<String, Object> parseEntityXmlFile(DispatchContext dctx, Map<String, ? extends Object> context) { GenericDelegator delegator = dctx.getDelegator(); URL url = (URL)context.get("url"); @@ -461,7 +453,7 @@ txTimeout = Integer.valueOf(7200); } - Long rowProcessed = Long.valueOf(0); + long rowProcessed = 0; try { EntitySaxReader reader = new EntitySaxReader(delegator); reader.setUseTryInsertMethod(mostlyInserts); @@ -471,16 +463,16 @@ reader.setCheckDataOnly(checkDataOnly); long numberRead = (url != null ? reader.parse(url) : reader.parse(xmltext)); - rowProcessed = Long.valueOf(numberRead); + rowProcessed = numberRead; } catch (Exception ex){ return ServiceUtil.returnError("Error parsing entity xml file: " + ex.toString()); } // send the notification - Map resp = UtilMisc.toMap("rowProcessed", rowProcessed); + Map<String, Object> resp = UtilMisc.<String, Object>toMap("rowProcessed", rowProcessed); return resp; } - public static Map entityExportAll(DispatchContext dctx, Map context) { + public static Map<String, Object> entityExportAll(DispatchContext dctx, Map<String, ? extends Object> context) { GenericDelegator delegator = dctx.getDelegator(); String outpath = (String)context.get("outpath"); // mandatory @@ -489,7 +481,7 @@ txTimeout = Integer.valueOf(7200); } - List results = new ArrayList(); + List<String> results = FastList.newInstance(); if (outpath != null && outpath.length() > 0) { File outdir = new File(outpath); @@ -497,20 +489,18 @@ outdir.mkdir(); } if (outdir.isDirectory() && outdir.canWrite()) { - Iterator passedEntityNames = null; + Set<String> passedEntityNames; try { ModelReader reader = delegator.getModelReader(); - Collection ec = reader.getEntityNames(); - TreeSet entityNames = new TreeSet(ec); - passedEntityNames = entityNames.iterator(); + Collection<String> ec = reader.getEntityNames(); + passedEntityNames = new TreeSet<String>(ec); } catch(Exception exc) { return ServiceUtil.returnError("Error retrieving entity names."); } int fileNumber = 1; - while (passedEntityNames.hasNext()) { + for (String curEntityName: passedEntityNames) { long numberWritten = 0; - String curEntityName = (String)passedEntityNames.next(); EntityListIterator values = null; try { @@ -566,7 +556,7 @@ results.add("No path specified, doing nothing."); } // send the notification - Map resp = UtilMisc.toMap("results", results); + Map<String, Object> resp = UtilMisc.<String, Object>toMap("results", results); return resp; } @@ -621,7 +611,7 @@ </ul> </li></ul> * */ - public static Map getEntityRefData(DispatchContext dctx, Map context) { + public static Map<String, Object> getEntityRefData(DispatchContext dctx, Map<String, ? extends Object> context) { GenericDelegator delegator = dctx.getDelegator(); Locale locale = (Locale) context.get("locale"); ClassLoader loader = Thread.currentThread().getContextClassLoader(); @@ -634,11 +624,9 @@ //put the entityNames TreeSets in a HashMap by packageName try { - Collection ec = reader.getEntityNames(); + Collection<String> ec = reader.getEntityNames(); resultMap.put("numberOfEntities", ec.size()); - Iterator ecIter = ec.iterator(); - while (ecIter.hasNext()) { - String eName = (String) ecIter.next(); + for (String eName: ec) { ModelEntity ent = reader.getModelEntity(eName); //make sure the table name is in the list of all table names, if not null if (UtilValidate.isNotEmpty(ent.getPlainTableName())) { @@ -665,10 +653,8 @@ String pName = (String) piter.next(); TreeSet<String> entities = entitiesByPackage.get(pName); List<Map<String, Object>> entitiesList = FastList.newInstance(); - Iterator e = entities.iterator(); - while (e.hasNext()) { + for (String entityName: entities) { Map<String, Object> entityMap = FastMap.newInstance(); - String entityName = (String) e.next(); String helperName = delegator.getEntityHelperName(entityName); String groupName = delegator.getEntityGroupName(entityName); if (search == null || entityName.toLowerCase().indexOf(search.toLowerCase()) != -1) { @@ -684,9 +670,9 @@ // fields list List<Map<String, Object>> javaNameList = FastList.newInstance(); - for (Iterator f = entity.getFieldsIterator(); f.hasNext();) { + for (Iterator<ModelField> f = entity.getFieldsIterator(); f.hasNext();) { Map<String, Object> javaNameMap = FastMap.newInstance(); - ModelField field = (ModelField) f.next(); + ModelField field = f.next(); ModelFieldType type = delegator.getEntityFieldType(entity, field.getType()); javaNameMap.put("isPk", field.getIsPk()); javaNameMap.put("name", field.getName()); @@ -755,8 +741,8 @@ List<String> fieldNameList = FastList.newInstance(); ModelIndex index = entity.getIndex(r); - for (Iterator fieldIterator = index.getIndexFieldsIterator(); fieldIterator.hasNext();) { - fieldNameList.add((String) fieldIterator.next()); + for (Iterator<String> fieldIterator = index.getIndexFieldsIterator(); fieldIterator.hasNext();) { + fieldNameList.add(fieldIterator.next()); } Map<String, Object> indexMap = FastMap.newInstance(); @@ -793,7 +779,7 @@ return resultMap; } - public static Map exportEntityEoModelBundle(DispatchContext dctx, Map context) { + public static Map<String, Object> exportEntityEoModelBundle(DispatchContext dctx, Map<String, ? extends Object> context) { String eomodeldFullPath = (String) context.get("eomodeldFullPath"); String entityPackageNameOrig = (String) context.get("entityPackageName"); String entityGroupId = (String) context.get("entityGroupId"); @@ -820,7 +806,7 @@ return ServiceUtil.returnError("eomodel Full Path is not write-able: " + eomodeldFullPath); } - Set<String> entityNames = new TreeSet(); + Set<String> entityNames = new TreeSet<String>(); if (UtilValidate.isNotEmpty(entityPackageNameOrig)) { Set<String> entityPackageNameSet = FastSet.newInstance(); entityPackageNameSet.addAll(StringUtil.split(entityPackageNameOrig, ",")); @@ -884,11 +870,11 @@ * @param context * @return */ - public static Map entityMaintPermCheck(DispatchContext dctx, Map context) { + public static Map<String, Object> entityMaintPermCheck(DispatchContext dctx, Map<String, ? extends Object> context) { GenericValue userLogin = (GenericValue) context.get("userLogin"); Locale locale = (Locale) context.get("locale"); Security security = dctx.getSecurity(); - Map resultMap = null; + Map<String, Object> resultMap = null; if (security.hasPermission("ENTITY_MAINT", userLogin)) { resultMap = ServiceUtil.returnSuccess(); resultMap.put("hasPermission", true); @@ -899,8 +885,8 @@ return resultMap; } - public static Map findJobs(DispatchContext dctx, Map context) { - Map result = ServiceUtil.returnSuccess(); + public static Map<String, Object> findJobs(DispatchContext dctx, Map<String, ? extends Object> context) { + Map<String, Object> result = ServiceUtil.returnSuccess(); GenericDelegator delegator = dctx.getDelegator(); String serviceName = (String) context.get("serviceName"); String jobId = (String) context.get("jobId"); @@ -913,7 +899,7 @@ } catch (Exception e) { viewIndex = 0; } - result.put("viewIndex", Integer.valueOf(viewIndex)); + result.put("viewIndex", viewIndex); int viewSize = 50; try { @@ -921,15 +907,15 @@ } catch (Exception e) { viewSize = 50; } - result.put("viewSize", Integer.valueOf(viewSize)); + result.put("viewSize", viewSize); // get the lookup flag String lookupFlag = (String) context.get("lookupFlag"); // list to hold the parameters - List paramList = FastList.newInstance(); - List conditions = FastList.newInstance(); - List jobList = null; + List<String> paramList = FastList.newInstance(); + List<EntityCondition> conditions = FastList.newInstance(); + List<GenericValue> jobList = null; int jobListSize = 0; int lowIndex = 0; int highIndex = 0; @@ -951,7 +937,7 @@ paramList.add("jobName=" + jobName); conditions.add(EntityCondition.makeCondition(EntityFunction.UPPER_FIELD("jobName"), EntityOperator.LIKE, EntityFunction.UPPER("%"+jobName+"%"))); } - List filterExprs = FastList.newInstance(); + List<EntityCondition> filterExprs = FastList.newInstance(); String filterJobPending = (String) context.get("filterJobsWithPendingStatus"); String filterJobRunning = (String) context.get("filterJobsWithRunningStatus"); String filterJobFinished = (String) context.get("filterJobsWithFinishedStatus"); @@ -985,7 +971,7 @@ } // set distinct on so we only get one row per job EntityFindOptions findOpts = new EntityFindOptions(true, EntityFindOptions.TYPE_SCROLL_INSENSITIVE, EntityFindOptions.CONCUR_READ_ONLY, true); - List orderBy = UtilMisc.toList("-runTime"); + List<String> orderBy = UtilMisc.toList("-runTime"); EntityCondition cond = null; if (conditions.size() > 0) { cond = EntityCondition.makeCondition(conditions); @@ -1021,14 +1007,14 @@ if (jobList == null) jobList = FastList.newInstance(); String paramString = StringUtil.join(paramList, "&"); result.put("paramList", (paramString != null ? paramString: "")); - result.put("lowIndex", Integer.valueOf(lowIndex)); - result.put("highIndex", Integer.valueOf(highIndex)); + result.put("lowIndex", lowIndex); + result.put("highIndex", highIndex); result.put("jobs", jobList); result.put("jobListSize", Integer.valueOf(jobListSize)); return result; } - public static Map exportServiceEoModelBundle(DispatchContext dctx, Map context) { + public static Map<String, Object> exportServiceEoModelBundle(DispatchContext dctx, Map<String, ? extends Object> context) { String eomodeldFullPath = (String) context.get("eomodeldFullPath"); String serviceName = (String) context.get("serviceName"); Modified: ofbiz/trunk/framework/webtools/src/org/ofbiz/webtools/artifactinfo/ArtifactInfoFactory.java URL: http://svn.apache.org/viewvc/ofbiz/trunk/framework/webtools/src/org/ofbiz/webtools/artifactinfo/ArtifactInfoFactory.java?rev=682078&r1=682077&r2=682078&view=diff ============================================================================== --- ofbiz/trunk/framework/webtools/src/org/ofbiz/webtools/artifactinfo/ArtifactInfoFactory.java (original) +++ ofbiz/trunk/framework/webtools/src/org/ofbiz/webtools/artifactinfo/ArtifactInfoFactory.java Sat Aug 2 16:31:04 2008 @@ -67,7 +67,7 @@ public static final String module = ArtifactInfoFactory.class.getName(); - protected static UtilCache<String, ArtifactInfoFactory> artifactInfoFactoryCache = new UtilCache("ArtifactInfoFactory"); + protected static UtilCache<String, ArtifactInfoFactory> artifactInfoFactoryCache = new UtilCache<String, ArtifactInfoFactory>("ArtifactInfoFactory"); public static final String EntityInfoTypeId = "entity"; public static final String ServiceInfoTypeId = "service"; Modified: ofbiz/trunk/framework/webtools/src/org/ofbiz/webtools/artifactinfo/ControllerRequestArtifactInfo.java URL: http://svn.apache.org/viewvc/ofbiz/trunk/framework/webtools/src/org/ofbiz/webtools/artifactinfo/ControllerRequestArtifactInfo.java?rev=682078&r1=682077&r2=682078&view=diff ============================================================================== --- ofbiz/trunk/framework/webtools/src/org/ofbiz/webtools/artifactinfo/ControllerRequestArtifactInfo.java (original) +++ ofbiz/trunk/framework/webtools/src/org/ofbiz/webtools/artifactinfo/ControllerRequestArtifactInfo.java Sat Aug 2 16:31:04 2008 @@ -26,6 +26,7 @@ import org.ofbiz.base.util.Debug; import org.ofbiz.base.util.GeneralException; +import org.ofbiz.base.util.UtilGenerics; import org.ofbiz.base.util.UtilMisc; import org.ofbiz.base.util.UtilObject; import org.ofbiz.base.util.UtilURL; @@ -79,7 +80,7 @@ } } - Map<String, String> responseMap = (Map<String, String>) this.requestInfoMap.get(ConfigXMLReader.RESPONSE_MAP); + Map<String, String> responseMap = UtilGenerics.checkMap(this.requestInfoMap.get(ConfigXMLReader.RESPONSE_MAP)); for (String responseValue: responseMap.values()) { if (responseValue.startsWith("view:")) { String viewUri = responseValue.substring(5); Modified: ofbiz/trunk/framework/webtools/src/org/ofbiz/webtools/print/FoPrintServerEvents.java URL: http://svn.apache.org/viewvc/ofbiz/trunk/framework/webtools/src/org/ofbiz/webtools/print/FoPrintServerEvents.java?rev=682078&r1=682077&r2=682078&view=diff ============================================================================== --- ofbiz/trunk/framework/webtools/src/org/ofbiz/webtools/print/FoPrintServerEvents.java (original) +++ ofbiz/trunk/framework/webtools/src/org/ofbiz/webtools/print/FoPrintServerEvents.java Sat Aug 2 16:31:04 2008 @@ -50,7 +50,7 @@ public static String getXslFo(HttpServletRequest req, HttpServletResponse resp) { LocalDispatcher dispatcher = (LocalDispatcher) req.getAttribute("dispatcher"); - Map reqParams = UtilHttp.getParameterMap(req); + Map<String, Object> reqParams = UtilHttp.getParameterMap(req); reqParams.put("locale", UtilHttp.getLocale(req)); String screenUri = (String) reqParams.remove("screenUri"); @@ -84,7 +84,7 @@ return null; } - public static byte[] getXslFo(DispatchContext dctx, String screen, Map parameters) throws GeneralException { + public static byte[] getXslFo(DispatchContext dctx, String screen, Map<String, Object> parameters) throws GeneralException { // run as the system user GenericValue system = null; try { |
Free forum by Nabble | Edit this page |