Scarab commit: svn commit: r10691 - trunk/src/java/org/tigris/scarab/actions/Search.java

Author: dabbous
Date: 2009-02-14 06:04:20-0800
New Revision: 10691

Modified:
   trunk/src/java/org/tigris/scarab/actions/Search.java

Log:
API Cleanup. The method doRunstoredqeury() is definitely not called from outside the class (even not from velocity).
So i changed the signature to private and removed the setup() call, which has already been performed right before
this method is called from doSelectquery(). There is no Scarab-issue open for this cleanup.

Modified: trunk/src/java/org/tigris/scarab/actions/Search.java
Url: http://scarab.tigris.org/source/browse/scarab/trunk/src/java/org/tigris/scarab/actions/Search.java?view=diff&pathrev=10691&r1=10690&r2=10691
==============================================================================
--- trunk/src/java/org/tigris/scarab/actions/Search.java	(original)
+++ trunk/src/java/org/tigris/scarab/actions/Search.java	2009-02-14 06:04:20-0800
@@ -532,11 +532,12 @@
 
     /**
         Runs the stored story.
+        [HD: As this method is only called internally, i made it private and removed the setup() call]
     */
-    public void doRunstoredquery(RunData data, TemplateContext context)
+    private void doRunstoredquery(RunData data, TemplateContext context)
          throws Exception
     {
-        setup(data, context);
+        //setup(data, context);
         // Set current query to the stored query
         Query query = scarabR.getQuery();
         MITList mitList = query.getMITList();

------------------------------------------------------
http://scarab.tigris.org/ds/viewMessage.do?dsForumId=3577&dsMessageId=1158038