Author: ssmiweve
Date: 2007-08-13 15:47:55 +0200 (Mon, 13 Aug 2007)
New Revision: 5604
Removed:
trunk/core-api/src/main/java/no/schibstedsok/searchportal/mode/navigation/
Modified:
trunk/core-api/src/main/java/no/schibstedsok/searchportal/run/QueryFactoryImpl.java
trunk/core-api/src/main/java/no/schibstedsok/searchportal/run/RunningQueryImpl.java
trunk/core-api/src/main/java/no/schibstedsok/searchportal/run/RunningWebQuery.java
Log:
trunk merge of r5603
Modified:
trunk/core-api/src/main/java/no/schibstedsok/searchportal/run/QueryFactoryImpl.java
===================================================================
---
trunk/core-api/src/main/java/no/schibstedsok/searchportal/run/QueryFactoryImpl.java
2007-08-13 12:30:18 UTC (rev 5603)
+++
trunk/core-api/src/main/java/no/schibstedsok/searchportal/run/QueryFactoryImpl.java
2007-08-13 13:47:55 UTC (rev 5604)
@@ -86,7 +86,7 @@
for (Cookie cookie : cookies) {
if ("myNews".equals(cookie.getName().trim())) {
LOG.debug("Adding cookie: " + cookie.getName() +
"=" + cookie.getValue());
- query.addParameter("myNews", cookie.getValue());
+ datamodel.getJunkYard().getValues().put("myNews",
cookie.getValue());
}
}
}
@@ -95,7 +95,7 @@
if (cookies != null) {
for (Cookie cookie : cookies) {
if ("myChannels".equals(cookie.getName())) {
- query.addParameter("myChannels",
cookie.getValue());
+
datamodel.getJunkYard().getValues().put("myChannels", cookie.getValue());
}
}
}
Modified:
trunk/core-api/src/main/java/no/schibstedsok/searchportal/run/RunningQueryImpl.java
===================================================================
---
trunk/core-api/src/main/java/no/schibstedsok/searchportal/run/RunningQueryImpl.java
2007-08-13 12:30:18 UTC (rev 5603)
+++
trunk/core-api/src/main/java/no/schibstedsok/searchportal/run/RunningQueryImpl.java
2007-08-13 13:47:55 UTC (rev 5604)
@@ -477,12 +477,6 @@
}
}
-
- /** TODO comment me. **/
- protected void addParameter(final String key, final Object obj) {
- datamodel.getJunkYard().getValues().put(key, obj);
- }
-
/** [EMAIL PROTECTED] **/
public Locale getLocale() {
Modified:
trunk/core-api/src/main/java/no/schibstedsok/searchportal/run/RunningWebQuery.java
===================================================================
---
trunk/core-api/src/main/java/no/schibstedsok/searchportal/run/RunningWebQuery.java
2007-08-13 12:30:18 UTC (rev 5603)
+++
trunk/core-api/src/main/java/no/schibstedsok/searchportal/run/RunningWebQuery.java
2007-08-13 13:47:55 UTC (rev 5604)
@@ -64,23 +64,29 @@
super(cxt, query);
+ this.request = request;
+ this.response = response;
+
if (LOG.isTraceEnabled()) {
LOG.trace("RunningWebQuery(mode, " + query + ", request,
response)");
}
+ // XXX The rest is redundant code!! stop using junkyard!
+ final Map<String,Object> parameters =
datamodel.getJunkYard().getValues();
+
// Add all request parameters
/* SEE "Add all request attributes" below */
- for (String parameterName :
(Set<String>)request.getParameterMap().keySet()) {
+
+ for (String key : datamodel.getParameters().getValues().keySet()) {
- final String[] parameterValues =
request.getParameterValues(parameterName);
- addParameter(parameterName, parameterValues.length>1 ?
parameterValues : parameterValues[0]);
+ final String value =
datamodel.getParameters().getValue(key).getString();
+ parameters.put(key, value);
if (LOG.isTraceEnabled()) {
- LOG.trace("Added " + parameterName + ", values: " +
StringUtils.join(parameterValues, ", "));
+ LOG.trace("Added " + key + ", value: " + value + ", ");
}
}
- final Map<String,Object> parameters =
datamodel.getJunkYard().getValues();
// Hack to keep vg site search working. Dependent on old query
// parameters. Remove when vg has been reimplented a proper site
search.
if (parameters.containsKey("nav_newspaperNames")) {
@@ -107,7 +113,7 @@
* this has now been changed. request parameters are first put
into the parameters map and
* are overwritten with request attributes. this is a basic
attempt to prevent parameter injection.
*/
- addParameter(attrName, request.getAttribute(attrName));
+ parameters.put(attrName, request.getAttribute(attrName));
if (LOG.isTraceEnabled()) {
LOG.trace("Added " + attrName + ", value: " +
request.getAttribute(attrName));
}
@@ -115,7 +121,7 @@
for (final String attrName : ATTRS_TO_COPY) {
if (request.getAttribute(attrName) != null) {
- addParameter(attrName, request.getAttribute(attrName));
+ parameters.put(attrName, request.getAttribute(attrName));
if (LOG.isTraceEnabled()) {
LOG.trace("Added(Manually) " + attrName + ", value: " +
request.getAttribute(attrName));
@@ -125,7 +131,7 @@
for (final String header : HEADERS_TO_COPY) {
if (request.getHeader(header) != null) {
- addParameter(header, request.getHeader(header));
+ parameters.put(header, request.getHeader(header));
if (LOG.isTraceEnabled()) {
LOG.trace("Added HTTP header " + header + ", value: " +
request.getHeader(header));
@@ -133,10 +139,8 @@
}
}
- this.request = request;
- this.response = response;
- addParameter("request", request);
- addParameter("response", response);
+ parameters.put("request", request);
+ parameters.put("response", response);
}
_______________________________________________
Kernel-commits mailing list
[email protected]
http://sesat.no/mailman/listinfo/kernel-commits
