Refactors commands and injection
[mkjsf-moved-to-github.git] / src / main / java / com / indexdata / mkjsf / pazpar2 / Pz2Bean.java
index d7a9235..05744cd 100644 (file)
@@ -1,6 +1,8 @@
 package com.indexdata.mkjsf.pazpar2;\r
 \r
 import java.io.Serializable;\r
+import java.lang.annotation.Retention;\r
+import java.lang.annotation.Target;\r
 import java.util.ArrayList;\r
 import java.util.Arrays;\r
 import java.util.HashMap;\r
@@ -10,8 +12,11 @@ import java.util.StringTokenizer;
 \r
 import javax.annotation.PostConstruct;\r
 import javax.enterprise.context.SessionScoped;\r
+import javax.enterprise.inject.Produces;\r
+import javax.faces.context.FacesContext;\r
 import javax.inject.Inject;\r
 import javax.inject.Named;\r
+import javax.inject.Qualifier;\r
 \r
 import org.apache.log4j.Logger;\r
 \r
@@ -30,7 +35,6 @@ import com.indexdata.mkjsf.pazpar2.data.RecordResponse;
 import com.indexdata.mkjsf.pazpar2.data.ResponseDataObject;\r
 import com.indexdata.mkjsf.pazpar2.data.ResponseParser;\r
 import com.indexdata.mkjsf.pazpar2.data.Responses;\r
-import com.indexdata.mkjsf.pazpar2.sp.auth.ServiceProxyUser;\r
 import com.indexdata.mkjsf.pazpar2.state.StateListener;\r
 import com.indexdata.mkjsf.pazpar2.state.StateManager;\r
 import com.indexdata.mkjsf.utils.Utils;\r
@@ -57,11 +61,11 @@ public class Pz2Bean implements Pz2Interface, StateListener, Configurable, Seria
   protected SearchClient searchClient = null;  \r
     \r
   @Inject ConfigurationReader configurator;\r
-  @Inject StateManager stateMgr;\r
-  @Inject Pazpar2Commands pzreq;\r
-  @Inject Responses pzresp;\r
-  @Inject ErrorCentral errors;\r
-  @Inject ServiceProxyUser user;\r
+  \r
+  private StateManager stateMgr = null;\r
+  private Pazpar2Commands pzreq = null;\r
+  private Responses pzresp = null;\r
+  private ErrorCentral errors = null;  \r
   \r
   protected ResultsPager pager = null; \r
   \r
@@ -71,9 +75,21 @@ public class Pz2Bean implements Pz2Interface, StateListener, Configurable, Seria
     logger.info("Instantiating pz2 bean [" + Utils.objectId(this) + "]");    \r
   }\r
   \r
+  public static Pz2Bean get() {\r
+    FacesContext context = FacesContext.getCurrentInstance();\r
+    return (Pz2Bean) context.getApplication().evaluateExpressionGet(context, "#{pz2}", Object.class); \r
+  }\r
+  \r
   @PostConstruct\r
-  public void postConstruct() {    \r
-    logger.debug("Pz2Bean post-construct: Configurator is " + configurator);\r
+  public void postConstruct() {\r
+    logger.info("Pz2Bean PostConstruct of " + this);\r
+    stateMgr = new StateManager();\r
+    pzreq = new Pazpar2Commands();\r
+    pzresp = new Responses();    \r
+    errors = new ErrorCentral(); \r
+    pzresp.setErrorHelper(errors.getHelper());\r
+    \r
+    logger.debug("Pz2Bean PostConstruct: Configurator is " + configurator);\r
     logger.debug(Utils.objectId(this) + " will instantiate a Pz2Client next.");\r
     pz2Client = new Pz2Client();\r
     configureClient(pz2Client,configurator);\r
@@ -86,7 +102,36 @@ public class Pz2Bean implements Pz2Interface, StateListener, Configurable, Seria
       e.printStackTrace();\r
     }    \r
     stateMgr.addStateListener(this);    \r
-  }  \r
+  }\r
+  \r
+  @Qualifier\r
+  @Target({java.lang.annotation.ElementType.TYPE,\r
+           java.lang.annotation.ElementType.METHOD,\r
+           java.lang.annotation.ElementType.PARAMETER,\r
+           java.lang.annotation.ElementType.FIELD})\r
+  @Retention(java.lang.annotation.RetentionPolicy.RUNTIME)\r
+  public  @interface Preferred{}\r
+  \r
+  @Produces @Preferred @SessionScoped @Named("pzresp") public Responses getPzresp () {\r
+    logger.info("Producing pzresp");\r
+    return pzresp;\r
+  }\r
+  \r
+  @Produces @Preferred @SessionScoped @Named("pzreq") public Pazpar2Commands getPzreq () {\r
+    logger.info("Producing pzreq");\r
+    return pzreq;\r
+  }\r
+  \r
+  @Produces @Preferred @SessionScoped @Named("errors") public ErrorCentral getErrors() {\r
+    logger.info("Producing errors");\r
+    return errors;\r
+  }\r
+  \r
+  @Produces @Preferred @SessionScoped @Named("stateMgr") public StateManager getStateMgr() {\r
+    logger.info("Produces stateMgr");\r
+    return stateMgr;\r
+  }\r
+\r
   \r
   public void configureClient(SearchClient client, ConfigurationReader configReader) {\r
     logger.debug(Utils.objectId(this) + " will configure search client for the session");\r
@@ -97,42 +142,55 @@ public class Pz2Bean implements Pz2Interface, StateListener, Configurable, Seria
       errors.addConfigurationError(new ConfigurationError("Search Client","Configuration",e.getMessage()));                \r
     } \r
     logger.info(configReader.document());\r
-    pzresp.resetAllSessionData();    \r
+    pzresp.getSp().resetAuthAndBeyond(true);    \r
   }\r
   \r
   public void resetSearchAndRecordCommands () {\r
     pzreq.getRecord().removeParametersInState();\r
     pzreq.getSearch().removeParametersInState();   \r
   }\r
-\r
     \r
   public void doSearch(String query) {\r
     pzreq.getSearch().setParameter(new CommandParameter("query","=",query));     \r
     doSearch();\r
   }\r
 \r
-  public void doSearch() { \r
-    stateMgr.hasPendingStateChange("search",false);\r
-    pzresp.resetSearchResponses();\r
-    // resets some record and show command parameters without \r
-    // changing state or creating state change feedback\r
-    pzreq.getRecord().removeParametersInState();        \r
-    pzreq.getShow().setParameterInState(new CommandParameter("start","=",0));    \r
-    logger.debug(Utils.objectId(this) + " is searching using "+pzreq.getCommand("search").getUrlEncodedParameterValue("query"));\r
-    doCommand("search");      \r
+  public void doSearch() {\r
+    try {\r
+    if (errors.hasConfigurationErrors()) {\r
+      logger.error("Ignoring search request due to configuration errors.");\r
+    } else if (searchClient == null){\r
+      logger.error("No search client defined. A client must either be pre-configured or selected before searching.");\r
+      errors.addConfigurationError(new ConfigurationError("No client defined","Client is null","No search client defined. A client must be pre-configured or selected runtime, prior to searching."));\r
+    } else {\r
+      stateMgr.hasPendingStateChange("search",false);\r
+      pzresp.resetSearchAndBeyond();\r
+      pzreq.getRecord().removeParametersInState();        \r
+      pzreq.getShow().setParameterInState(new CommandParameter("start","=",0));    \r
+      logger.debug(Utils.objectId(this) + " is searching using "+pzreq.getCommand("search").getUrlEncodedParameterValue("query"));\r
+      searchClient.setSearchCommand(pzreq.getCommand("search"));\r
+      doCommand("search");\r
+    }\r
+    } catch (NullPointerException npe) {\r
+      npe.printStackTrace();      \r
+    }\r
   }\r
   \r
   public String doRecord() {\r
-    ResponseDataObject responseObject = doCommand("record");\r
-    if (pzreq.getRecord().hasParameterValue("offset") ||\r
-          pzreq.getRecord().hasParameterValue("checksum")) {\r
-        RecordResponse recordResponse = new RecordResponse();\r
-        recordResponse.setType("record");\r
-        recordResponse.setXml(responseObject.getXml());\r
-        recordResponse.setAttribute("activeclients", "0");\r
-        pzresp.put("record", recordResponse);\r
-     }\r
-     return pzresp.getRecord().getActiveClients();    \r
+    if (errors.hasConfigurationErrors()) {\r
+      logger.error("Ignoring record request due to configuration errors.");\r
+      return "";\r
+    } else if (!pzreq.getCommand("record").hasParameterValue("id")) {\r
+      logger.debug("Ignoring record request due to no id parameter.");\r
+      return "";\r
+    } else if (pzresp.getSearch().hasApplicationError()) {\r
+      logger.debug("Ignoring record request due search error.");\r
+      return "";\r
+    } else {\r
+      logger.debug("Executing record command");\r
+      doCommand("record");\r
+      return pzresp.getRecord().getActiveClients();\r
+    }\r
   }\r
       \r
   /**\r
@@ -142,57 +200,50 @@ public class Pz2Bean implements Pz2Interface, StateListener, Configurable, Seria
    */\r
   public String update () {\r
     logger.debug("Updating show,stat,termlist,bytarget from pazpar2");\r
-    return update("show,stat,termlist,bytarget");\r
-  }\r
-  \r
-  public boolean validateUpdateRequest(String commands) {\r
     if (errors.hasConfigurationErrors()) {\r
-      logger.error("The command(s) " + commands + " are cancelled due to configuration errors.");\r
-      return false;\r
-    } else if (!commands.equals("search") && pzresp.getSearch().hasApplicationError()) {\r
-      logger.error("The command(s) " + commands + " are cancelled because the latest search command had an error.");\r
-      return false;\r
-    } else if (!commandsAreValid(commands)) {\r
-      logger.debug("The command(s) " + commands + " are cancelled because the were not found to be ready/valid.");\r
-      return false;\r
-    } else if (!hasQuery() &&  !(commands.equals("record") && pzreq.getCommand("record").hasParameterValue("recordquery"))) {\r
-      logger.debug("The command(s) " + commands + " are held off because there's not yet a query.");\r
-      return false;\r
+      logger.error("Ignoring show,stat,termlist,bytarget commands due to configuration errors.");\r
+      return "";\r
+    } else if (pzresp.getSearch().hasApplicationError()) {\r
+      logger.error("Ignoring show,stat,termlist,bytarget commands due to problem with most recent search.");\r
+      return "";\r
+    } else if (!hasQuery()) {\r
+      logger.debug("Ignoring show,stat,termlist,bytarget commands because there is not yet a query.");\r
+      return "";\r
     } else {\r
-      return true;\r
+      return update("show,stat,termlist,bytarget");\r
     }\r
-    \r
-    \r
   }\r
-   \r
+     \r
   /**\r
    * Refreshes the data objects listed in 'commands' from pazpar2\r
    * \r
    * @param commands\r
-   * @return Number of activeclients at the time of the 'show' command\r
+   * @return Number of activeclients at the time of the 'show' command,\r
+   *         or 'new' if search was just initiated.\r
    */\r
   public String update (String commands) {\r
-    logger.info("Request to update: " + commands);\r
+    logger.debug("Request to update: " + commands);\r
     try {\r
-      if (!validateUpdateRequest(commands)) {\r
-        return "0";\r
-      } else if (commands.equals("search")) {\r
+      if (commands.equals("search")) {\r
         doSearch();\r
-        return "";\r
+        return "new";\r
       } else if (commands.equals("record")) {\r
         return doRecord();\r
       } else if (pzresp.getSearch().isNew()) {\r
+        // For returning notification of 'search started' quickly to UI\r
         logger.info("New search. Marking it old, then returning 'new' to trigger another round-trip.");\r
         pzresp.getSearch().setIsNew(false);\r
         return "new";\r
       } else {\r
         handleQueryStateChanges(commands);\r
         if (pzresp.getSearch().hasApplicationError()) {\r
-          logger.error("The command(s) " + commands + " are cancelled because the latest search command had an error.");\r
+          logger.error("The command(s) " + commands + " cancelled because the latest search command had an error.");\r
+          return "0";\r
+        } else if (errors.hasConfigurationErrors()) {\r
+          logger.error("The command(s) " + commands + " cancelled due to configuration errors.");\r
           return "0";\r
         } else {\r
           logger.debug("Processing request for " + commands); \r
-          \r
           List<CommandThread> threadList = new ArrayList<CommandThread>();\r
           StringTokenizer tokens = new StringTokenizer(commands,",");\r
           while (tokens.hasMoreElements()) {          \r
@@ -234,17 +285,7 @@ public class Pz2Bean implements Pz2Interface, StateListener, Configurable, Seria
     }\r
     \r
   }\r
-  \r
-  public boolean commandsAreValid(String commands) {\r
-    if (commands.equals("record")) {\r
-      if (!pzreq.getCommand("record").hasParameterValue("id")) {\r
-        logger.debug("Skips sending record command due to lacking id parameter");\r
-        return false;\r
-      }\r
-    }\r
-    return true;\r
-  }\r
-                                \r
+                                  \r
   public String toggleRecord (String recId) {\r
     if (hasRecord(recId)) {\r
       pzreq.getRecord().removeParameters();  \r
@@ -288,10 +329,22 @@ public class Pz2Bean implements Pz2Interface, StateListener, Configurable, Seria
     pager =  new ResultsPager(pzresp,pageRange,pzreq);\r
     return pager;\r
   }\r
-    \r
+   \r
+  /**\r
+   * This methods main purpose is to support browser history.\r
+   *  \r
+   * When the browsers back or forward buttons are pressed, a  \r
+   * re-search /might/ be required - namely if the query changes.\r
+   * So, as the UI requests updates of the page (show,facets,\r
+   * etc) this method checks if a search must be executed\r
+   * before those updates are performed.\r
+   *  \r
+   * @see {@link com.indexdata.mkjsf.pazpar2.state.StateManager#setCurrentStateKey} \r
+   * @param commands\r
+   */\r
   protected void handleQueryStateChanges (String commands) {\r
     if (stateMgr.hasPendingStateChange("search") && hasQuery()) { \r
-      logger.info("Triggered search: Found pending search change, doing search before updating " + commands);      \r
+      logger.info("Triggered search: Found pending search change [" + pzreq.getCommand("search").toString() + "], doing search before updating " + commands);      \r
       doSearch();\r
     } \r
     if (stateMgr.hasPendingStateChange("record") && ! commands.equals("record")) {        \r
@@ -306,28 +359,25 @@ public class Pz2Bean implements Pz2Interface, StateListener, Configurable, Seria
   }\r
   \r
   /**\r
-   * Validates the request then executes the command and parses the response.\r
+   * Executes the command and parses the response to create data objects.\r
    * If the parsed response is of a known type it will be cached in 'pzresp'\r
    * \r
    * @param commandName The command to be executed\r
    * @return An XML response parsed to form a response data object\r
    */\r
   protected ResponseDataObject doCommand(String commandName) {\r
-    ResponseDataObject responseObject = null; \r
-    if (validateUpdateRequest(commandName)) {\r
-      logger.debug(pzreq.getCommand(commandName).getEncodedQueryString() + ": Results for "+ pzreq.getCommand("search").getEncodedQueryString());\r
-      Pazpar2Command command = pzreq.getCommand(commandName);\r
+    Pazpar2Command command = pzreq.getCommand(commandName);\r
+    if (command.spOnly() && isPazpar2Service()) {\r
+      logger.warn("Skipping " + commandName + " - SP-only command, not supported by Pazpar2");\r
+      return new ResponseDataObject();\r
+    } else {\r
+      logger.info("Request "+commandName + ". Search command is: "+ pzreq.getCommand("search").toString());      \r
       long start = System.currentTimeMillis();\r
-      HttpResponseWrapper commandResponse = searchClient.executeCommand(command);\r
+      ResponseDataObject responseObject = command.run();      \r
       long end = System.currentTimeMillis();\r
       logger.debug("Executed " + command.getCommandName() + " in " + (end-start) + " ms." );\r
-      responseLogger.debug("Response was: " + commandResponse.getResponseString());\r
-      responseObject = ResponseParser.getParser().getDataObject((ClientCommandResponse)commandResponse);\r
-      if (ResponseParser.docTypes.contains(responseObject.getType())) {\r
-        pzresp.put(commandName, responseObject);\r
-      }      \r
+      return responseObject;\r
     }\r
-    return responseObject;\r
   }\r
     \r
   @Override\r
@@ -373,9 +423,9 @@ public class Pz2Bean implements Pz2Interface, StateListener, Configurable, Seria
     if (url!=null && searchClient != null && !url.equals(searchClient.getServiceUrl())) {\r
       pzreq.getRecord().removeParametersInState();\r
       pzreq.getSearch().removeParametersInState();\r
-      pzresp.resetAllSessionData();\r
-      user.clear();\r
+      pzresp.getSp().resetAuthAndBeyond(true);      \r
       searchClient.setServiceUrl(url);\r
+      // pzreq.setService(this);\r
     }    \r
   }\r
   \r
@@ -383,6 +433,17 @@ public class Pz2Bean implements Pz2Interface, StateListener, Configurable, Seria
     return (searchClient!=null ? searchClient.getServiceUrl() : "");\r
   }\r
   \r
+  public void setServiceId () {\r
+    pzreq.getRecord().removeParametersInState();\r
+    pzreq.getSearch().removeParametersInState();\r
+    pzresp.resetSearchAndBeyond();\r
+    pz2Client.setServiceId(pzreq.getInit().getService());\r
+  }\r
+  \r
+  public String getServiceId () {\r
+    return pzreq.getInit().getService();\r
+  }\r
+  \r
   public boolean getServiceUrlIsDefined() {\r
     return (searchClient != null && searchClient.hasServiceUrl());\r
   }\r
@@ -460,6 +521,7 @@ public class Pz2Bean implements Pz2Interface, StateListener, Configurable, Seria
       serviceProxyUrls = config.getMultiProperty(SERVICE_PROXY_URL_LIST,",");\r
       pazpar2Urls = config.getMultiProperty(PAZPAR2_URL_LIST, ",");\r
     }\r
+    logger.info(reader.document());\r
     logger.info("Service Type is configured to " + serviceType);\r
     \r
   }\r
@@ -498,7 +560,7 @@ public class Pz2Bean implements Pz2Interface, StateListener, Configurable, Seria
     if (!serviceType.equals(type)  &&\r
         !serviceType.equals(SERVICE_TYPE_TBD)) {\r
       resetSearchAndRecordCommands();\r
-      pzresp.resetAllSessionData();\r
+      pzresp.getSp().resetAuthAndBeyond(true);\r
     }\r
     serviceType = type;\r
     if (serviceType.equals(SERVICE_TYPE_PZ2)) {\r
@@ -513,4 +575,8 @@ public class Pz2Bean implements Pz2Interface, StateListener, Configurable, Seria
     }\r
   }\r
   \r
+  public SearchClient getSearchClient() {\r
+    return searchClient;\r
+  }\r
+  \r
 }\r