Author: deepak
Date: Sat May 12 06:43:47 2018
New Revision: 1831450
URL: http://svn.apache.org/viewvc?rev=1831450&view=rev
Log:
Improved: Remove createContent Java service implementation
(OFBIZ-10393)
The createContent service has been replaced by a minilang implementation since
2007
Thanks Mathieu Lirzin for your contribution
Modified:
ofbiz/ofbiz-framework/trunk/applications/content/src/main/java/org/apache/ofbiz/content/content/ContentServices.java
Modified:
ofbiz/ofbiz-framework/trunk/applications/content/src/main/java/org/apache/ofbiz/content/content/ContentServices.java
URL:
http://svn.apache.org/viewvc/ofbiz/ofbiz-framework/trunk/applications/content/src/main/java/org/apache/ofbiz/content/content/ContentServices.java?rev=1831450&r1=1831449&r2=1831450&view=diff
==============================================================================
---
ofbiz/ofbiz-framework/trunk/applications/content/src/main/java/org/apache/ofbiz/content/content/ContentServices.java
(original)
+++
ofbiz/ofbiz-framework/trunk/applications/content/src/main/java/org/apache/ofbiz/content/content/ContentServices.java
Sat May 12 06:43:47 2018
@@ -225,89 +225,6 @@ public class ContentServices {
}
/**
- * Create a Content service. The work is done in a separate method so that
complex services that need this functionality do not need to incur the
- * reflection performance penalty.
- */
- public static Map<String, Object> createContent(DispatchContext dctx,
Map<String, ? extends Object> context) {
- Map<String, Object> result = createContentMethod(dctx, context);
- return result;
- }
-
- /**
- * Create a Content method. The work is done in this separate method so
that complex services that need this functionality do not need to incur the
- * reflection performance penalty.
- */
- public static Map<String, Object> createContentMethod(DispatchContext
dctx, Map<String, ? extends Object> rcontext) {
- Map<String, Object> context = UtilMisc.makeMapWritable(rcontext);
- context.put("entityOperation", "_CREATE");
- List<String> targetOperationList =
ContentWorker.prepTargetOperationList(context, "_CREATE");
- if (Debug.infoOn()) Debug.logInfo("in createContentMethod,
targetOperationList: " + targetOperationList, null);
-
- List<String> contentPurposeList =
ContentWorker.prepContentPurposeList(context);
- context.put("targetOperationList", targetOperationList);
- context.put("contentPurposeList", contentPurposeList);
-
- Timestamp nowTimestamp = UtilDateTime.nowTimestamp();
- Map<String, Object> result = new HashMap<String, Object>();
- Delegator delegator = dctx.getDelegator();
- LocalDispatcher dispatcher = dctx.getDispatcher();
- String contentId = (String) context.get("contentId");
-
- if (UtilValidate.isEmpty(contentId)) {
- contentId = delegator.getNextSeqId("Content");
- }
-
- GenericValue content = delegator.makeValue("Content",
UtilMisc.toMap("contentId", contentId));
- content.setNonPKFields(context);
-
- GenericValue userLogin = (GenericValue) context.get("userLogin");
- String userLoginId = (String) userLogin.get("userLoginId");
-
- // get first statusId for content out of the statusItem table if not
provided
- if (UtilValidate.isEmpty(context.get("statusId"))) {
- try {
- GenericValue statusItem =
EntityQuery.use(delegator).from("StatusItem")
- .where("statusTypeId", "CONTENT_STATUS")
- .orderBy("sequenceId").queryFirst();
- if (statusItem != null) {
- content.put("statusId", statusItem.get("statusId"));
- }
- } catch (GenericEntityException e) {
- return ServiceUtil.returnError(e.getMessage());
- }
- }
-
- content.put("createdByUserLogin", userLoginId);
- content.put("lastModifiedByUserLogin", userLoginId);
- content.put("createdDate", nowTimestamp);
- content.put("lastModifiedDate", nowTimestamp);
-
- context.put("currentContent", content);
- if (Debug.infoOn()) Debug.logInfo("in createContentMethod, context: "
+ context, null);
-
- Map<String, Object> permResults =
ContentWorker.callContentPermissionCheckResult(delegator, dispatcher, context);
- String permissionStatus = (String) permResults.get("permissionStatus");
- if (permissionStatus != null &&
"granted".equalsIgnoreCase(permissionStatus)) {
- try {
- content.create();
- } catch (GenericEntityException e) {
- return ServiceUtil.returnError(e.getMessage());
- } catch (Exception e2) {
- return ServiceUtil.returnError(e2.getMessage());
- }
-
- result.put("contentId", contentId);
- } else {
- String errorMsg = (String)
permResults.get(ModelService.ERROR_MESSAGE);
- result.put(ModelService.ERROR_MESSAGE, errorMsg);
- return ServiceUtil.returnFailure(errorMsg);
- }
-
- context.remove("currentContent");
- return result;
- }
-
- /**
* Create a ContentAssoc service. The work is done in a separate method so
that complex services that need this functionality do not need to incur the
* reflection performance penalty.
*/
