butlermh 02/02/22 03:30:02
Modified: src/java/org/apache/cocoon/transformation
TraxTransformer.java
Log:
Resolved problem detecting DELI component / removed unnecessary configuration option.
Revision Changes Path
1.16 +13 -30
xml-cocoon2/src/java/org/apache/cocoon/transformation/TraxTransformer.java
Index: TraxTransformer.java
===================================================================
RCS file:
/home/cvs/xml-cocoon2/src/java/org/apache/cocoon/transformation/TraxTransformer.java,v
retrieving revision 1.15
retrieving revision 1.16
diff -u -r1.15 -r1.16
--- TraxTransformer.java 22 Feb 2002 07:03:56 -0000 1.15
+++ TraxTransformer.java 22 Feb 2002 11:30:02 -0000 1.16
@@ -157,7 +157,7 @@
* @author <a href="mailto:[EMAIL PROTECTED]">Giacomo Pati</a>
* @author <a href="mailto:[EMAIL PROTECTED]">Ovidiu Predescu</a>
* @author <a href="mailto:[EMAIL PROTECTED]">Mark H. Butler</a>
- * @version CVS $Id: TraxTransformer.java,v 1.15 2002/02/22 07:03:56 cziegeler Exp $
+ * @version CVS $Id: TraxTransformer.java,v 1.16 2002/02/22 11:30:02 butlermh Exp $
*/
public class TraxTransformer extends AbstractTransformer
implements Transformer, Composable, Configurable, Cacheable, Disposable {
@@ -178,10 +178,6 @@
private boolean useBrowserCap = false;
private boolean _useBrowserCap = false;
- /* Should we make the DELI CC/PP profile available to the stylesheet (default
is off) */
- private boolean useDeli = false;
- private boolean _useDeli = false;
-
/** Should we make the cookies availalbe in the stylesheet? (default is off) */
private boolean useCookies = false;
private boolean _useCookies = false;
@@ -231,16 +227,11 @@
child = conf.getChild("xslt-processor-role");
String xsltRole = child.getValue(XSLTProcessor.ROLE);
- child = conf.getChild("use-deli");
- this.useDeli = child.getValueAsBoolean(false);
- this._useDeli = this.useDeli;
-
if (this.getLogger().isDebugEnabled()) {
this.getLogger().debug("Use parameters is " + this.useParameters +
" for " + this);
this.getLogger().debug("Use cookies is " + this.useCookies + " for
" + this);
this.getLogger().debug("Use browser capabilities is " +
this.useBrowserCap + " for " + this);
this.getLogger().debug("Use session info is " + this.useSessionInfo
+ " for " + this);
- this.getLogger().debug("Use DELI is " + this.useDeli + " for " +
this);
this.getLogger().debug("Use XSLTProcessor of role " + xsltRole);
}
@@ -264,20 +255,19 @@
}
this.browser = (Browser) manager.lookup(Browser.ROLE);
- // FIXME (VG): Replace try with hasComponent() check when it works as
expected
- // System.out.println(Deli.ROLE + ": " +
this.manager.hasComponent(Deli.ROLE));
- // if (this.manager.hasComponent(Deli.ROLE)) {
+
+ if (this.manager.hasComponent(Deli.ROLE)) {
try {
if (this.getLogger().isDebugEnabled()) {
getLogger().debug("Looking up " + Deli.ROLE);
}
this.deli = (Deli) this.manager.lookup(Deli.ROLE);
- // } else {
} catch (ComponentException e) {
getLogger().debug("Deli is not available");
} catch (NoClassDefFoundError e) {
getLogger().debug("Deli is not available");
}
+ }
}
/**
@@ -298,7 +288,6 @@
this.xsltProcessor.setSourceResolver(resolver);
_useParameters = par.getParameterAsBoolean("use-request-parameters",
this.useParameters);
_useBrowserCap = par.getParameterAsBoolean("use-browser-capabilities-db",
this.useBrowserCap);
- _useDeli = par.getParameterAsBoolean("use-deli", this.useDeli);
_useCookies = par.getParameterAsBoolean("use-cookies", this.useCookies);
_useSessionInfo = par.getParameterAsBoolean("use-session-info",
this.useSessionInfo);
if (this.getLogger().isDebugEnabled()) {
@@ -493,27 +482,22 @@
getLogger().error("Error setting Browser info", e);
}
- if (this._useDeli) {
+ if (this.deli != null) {
try {
Request request = ObjectModelHelper.getRequest(objectModel);
if (map == null) {
map = new HashMap();
}
- if (this.deli != null) {
- org.w3c.dom.Document deliCapabilities =
this.deli.getUACapabilities(request);
- map.put("deli-capabilities", deliCapabilities);
-
- String accept = request.getParameter("accept");
- if (accept == null)
- accept = request.getHeader("accept");
-
- /* add the accept param */
- map.put("accept", accept);
- } else {
- getLogger().warn("TraxTransformer configured to use DELI in
sitemap.xmap but DELI not enabled in cocoon.xconf.");
- }
+ org.w3c.dom.Document deliCapabilities =
this.deli.getUACapabilities(request);
+ map.put("deli-capabilities", deliCapabilities);
+
+ String accept = request.getParameter("accept");
+ if (accept == null)
+ accept = request.getHeader("accept");
+ /* add the accept param */
+ map.put("accept", accept);
} catch (Exception e) {
getLogger().error("Error setting DELI info", e);
}
@@ -568,7 +552,6 @@
this._useParameters = this.useParameters;
this._useCookies = this.useCookies;
this._useBrowserCap = this.useBrowserCap;
- this._useDeli = this.useDeli;
this._useSessionInfo = this.useSessionInfo;
super.recycle();
}
----------------------------------------------------------------------
In case of troubles, e-mail: [EMAIL PROTECTED]
To unsubscribe, e-mail: [EMAIL PROTECTED]
For additional commands, e-mail: [EMAIL PROTECTED]