[Yanel-dev] Re: [Yanel-commits] rev 43262 - public/yanel/trunk/src/realms/from-scratch-realm-template/res-types/layoutselector/src/java/org/wyona/yanel/impl/resources/layoutselector

[Guillaume Déflache]

michi at wyona.com schrieb:
> Author: michi
> Date: 2009-06-19 11:16:29 +0200 (Fri, 19 Jun 2009)
> New Revision: 43262
> 
> Modified:
>    public/yanel/trunk/src/realms/from-scratch-realm-template/res-types/layoutselector/src/java/org/wyona/yanel/impl/resources/layoutselector/LayoutSelector.java
> Log:
> copyright year fixed, formatting fixed and error logging improved
> 
> Modified: public/yanel/trunk/src/realms/from-scratch-realm-template/res-types/layoutselector/src/java/org/wyona/yanel/impl/resources/layoutselector/LayoutSelector.java
> ===================================================================
> --- public/yanel/trunk/src/realms/from-scratch-realm-template/res-types/layoutselector/src/java/org/wyona/yanel/impl/resources/layoutselector/LayoutSelector.java	2009-06-19 09:00:29 UTC (rev 43261)
> +++ public/yanel/trunk/src/realms/from-scratch-realm-template/res-types/layoutselector/src/java/org/wyona/yanel/impl/resources/layoutselector/LayoutSelector.java	2009-06-19 09:16:29 UTC (rev 43262)
> @@ -1,5 +1,5 @@
>  /*
> - * Copyright 2007 Wyona
> + * Copyright 2009 Wyona
>   */
>  
>  package org.wyona.yanel.impl.resources.layoutselector;
> @@ -47,19 +47,18 @@
>      private static Logger log = Logger.getLogger(LayoutSelector.class);
>      private String uploadMimeType; 
>      private String nodeVersionChecker ="";
> -    
>  
>      /**
>       * Constructor
> -     * 
>       */
>      public LayoutSelector() {
>      	try {
> -    		nodeVersionChecker =  getRealm().getName() + ":" + getResourceConfigProperty("layout_name") + ":xslt-version";
> -    	} catch(Exception ex) {
> -    		log.error("error initializing template version checker session string");
> +            nodeVersionChecker =  getRealm().getName() + ":" + getResourceConfigProperty("layout_name") + ":xslt-version";
> +    	} catch(Exception e) {
> +            log.error("error initializing template version checker session string");
> +            log.error(e, e);
>      	}
> -    };
> +    }
>  
>      /**
>       *
> 

  Just curious: why not: log.error("error initializing template version 
checker session string", e ); instead?