xlawrence 2005/11/02 18:54:33 CET
Modified files:
core/src/java/org/jahia/engines/importexport
ClipboardEngine.java
Log:
added some useful log messages + some code optimizations
Revision Changes Path
1.5 +39 -31
jahia/core/src/java/org/jahia/engines/importexport/ClipboardEngine.java
http://jahia.mine.nu:8080/cgi-bin/cvsweb.cgi/jahia/core/src/java/org/jahia/engines/importexport/ClipboardEngine.java.diff?r1=1.4&r2=1.5&f=h
Index: ClipboardEngine.java
===================================================================
RCS file:
/home/cvs/repository/jahia/core/src/java/org/jahia/engines/importexport/ClipboardEngine.java,v
retrieving revision 1.4
retrieving revision 1.5
diff -u -r1.4 -r1.5
--- ClipboardEngine.java 2 Nov 2005 15:35:28 -0000 1.4
+++ ClipboardEngine.java 2 Nov 2005 17:54:32 -0000 1.5
@@ -1,44 +1,49 @@
package org.jahia.engines.importexport;
+import org.jahia.content.ContentObject;
+import org.jahia.content.ContentObjectKey;
+import org.jahia.data.JahiaData;
import org.jahia.engines.JahiaEngine;
-import org.jahia.engines.EngineToolBox;
-import org.jahia.engines.workflow.WorkflowEngine;
import org.jahia.engines.validation.EngineValidationHelper;
-import org.jahia.params.ProcessingContext;
import org.jahia.exceptions.JahiaException;
-import org.jahia.data.JahiaData;
-import org.jahia.content.ContentObject;
-import org.jahia.content.ContentObjectKey;
+import org.jahia.params.ProcessingContext;
+import org.jahia.params.SessionState;
import org.jahia.registries.ServicesRegistry;
import org.jahia.services.version.EntryLoadRequest;
+import org.jahia.services.importexport.ImportExportService;
/**
* Created by IntelliJ IDEA.
* User: toto
- * Date: 25 août 2005
+ * Date: 25 aout 2005
* Time: 15:04:17
* To change this template use File | Settings | File Templates.
*/
public class ClipboardEngine implements JahiaEngine {
private static final org.apache.log4j.Logger logger =
- org.apache.log4j.Logger.getLogger (JahiaEngine.class);
+ org.apache.log4j.Logger.getLogger(JahiaEngine.class);
+
+ private static final ImportExportService importExportService =
+ ServicesRegistry.getInstance().getImportExportService();
- /** the unique instance of this class */
+ /**
+ * the unique instance of this class
+ */
private static ClipboardEngine instance = null;
/**
- * Default constructor, creates a new <code>WorkflowEngine</code>
instance.
+ * Default constructor, creates a new <code>ClipboardEngine</code>
instance.
*/
- protected ClipboardEngine () {
- logger.debug ("***** Starting " + ClipboardEngine.class.getName () +
" engine *****");
+ protected ClipboardEngine() {
+ logger.debug("***** Starting " + ClipboardEngine.class.getName() + "
engine *****");
}
/**
* @return The instance of this engine
*/
- public static synchronized ClipboardEngine getInstance () {
+ public static synchronized ClipboardEngine getInstance() {
if (instance == null) {
- instance = new ClipboardEngine ();
+ instance = new ClipboardEngine();
}
return instance;
}
@@ -47,36 +52,39 @@
return true;
}
- public String renderLink(ProcessingContext processingContext, Object
theObj) throws JahiaException {
- return processingContext.composeEngineUrl ("clipboard", "?key="+
((ContentObject)theObj).getObjectKey() );
+ public String renderLink(final ProcessingContext processingContext,
final Object theObj) throws JahiaException {
+ return processingContext.composeEngineUrl("clipboard", "?key=" +
((ContentObject) theObj).getObjectKey());
}
public boolean needsJahiaData(ProcessingContext processingContext) {
return false;
}
- public EngineValidationHelper handleActions(ProcessingContext
processingContext, JahiaData jData) throws JahiaException {
- String op = processingContext.getParameter("cop");
- String key = processingContext.getParameter("key");
- if ("cut".equals(op) || "copy".equals(op) ) {
-
processingContext.getSessionState().setAttribute("clipboard_op",op);
-
processingContext.getSessionState().setAttribute("clipboard_key",key);
+ public EngineValidationHelper handleActions(final ProcessingContext
processingContext, final JahiaData jData) throws JahiaException {
+ final String op = processingContext.getParameter("cop");
+ final String key = processingContext.getParameter("key");
+
+ logger.debug("ClipboardEngine - op: " + op + ", key: " + key);
+
+ final SessionState sessionState =
processingContext.getSessionState();
+ if ("cut".equals(op) || "copy".equals(op)) {
+ sessionState.setAttribute("clipboard_op", op);
+ sessionState.setAttribute("clipboard_key", key);
}
if ("paste".equals(op)) {
try {
- String sOp = (String)
processingContext.getSessionState().getAttribute("clipboard_op");
-
processingContext.getSessionState().removeAttribute("clipboard_op");
+ sessionState.removeAttribute("clipboard_op");
- String skey = (String)
processingContext.getSessionState().getAttribute("clipboard_key");
-
processingContext.getSessionState().removeAttribute("clipboard_key");
- ContentObject source =
ContentObject.getContentObjectInstance(ContentObjectKey.getInstance(skey));
+ final String skey = (String)
processingContext.getSessionState().getAttribute("clipboard_key");
+ sessionState.removeAttribute("clipboard_key");
+ final ContentObject source =
ContentObject.getContentObjectInstance(ContentObjectKey.getInstance(skey));
- String dkey = processingContext.getParameter("key");
- ContentObject dest =
ContentObject.getContentObjectInstance(ContentObjectKey.getInstance(dkey));
+ final String dkey = processingContext.getParameter("key");
+ final ContentObject dest =
ContentObject.getContentObjectInstance(ContentObjectKey.getInstance(dkey));
-
ServicesRegistry.getInstance().getImportExportService().copy(source,dest ,
processingContext, EntryLoadRequest.STAGED, false);
+ importExportService.copy(source, dest, processingContext,
EntryLoadRequest.STAGED, false);
} catch (ClassNotFoundException e) {
- e.printStackTrace(); //To change body of catch statement
use File | Settings | File Templates.
+ logger.error(e.getMessage(), e);
}
}
return null;