[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

[Prabodh Upreti]

Dear Guillaume
=A0
Thanks for pointing that out. My oversight. Appreciate your feedback.
=A0
Regards
=A0
Prabodh

--- On Fri, 6/19/09, Guillaume D=E9flache <guillaume.deflache at wyona.com> wr=
ote:


From: Guillaume D=E9flache <guillaume.deflache at wyona.com>
Subject: [Yanel-dev] Re: [Yanel-commits] rev 43262 - public/yanel/trunk/src=
/realms/from-scratch-realm-template/res-types/layoutselector/src/java/org/w=
yona/yanel/impl/resources/layoutselector
To: yanel-development at wyona.com
Date: Friday, June 19, 2009, 7:38 AM


michi at wyona.com schrieb:
> Author: michi
> Date: 2009-06-19 11:16:29 +0200 (Fri, 19 Jun 2009)
> New Revision: 43262
> =

> Modified:
>=A0 =A0 public/yanel/trunk/src/realms/from-scratch-realm-template/res-type=
s/layoutselector/src/java/org/wyona/yanel/impl/resources/layoutselector/Lay=
outSelector.java
> Log:
> copyright year fixed, formatting fixed and error logging improved
> =

> Modified: public/yanel/trunk/src/realms/from-scratch-realm-template/res-t=
ypes/layoutselector/src/java/org/wyona/yanel/impl/resources/layoutselector/=
LayoutSelector.java
> =3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=
=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=
=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D
> --- public/yanel/trunk/src/realms/from-scratch-realm-template/res-types/l=
ayoutselector/src/java/org/wyona/yanel/impl/resources/layoutselector/Layout=
Selector.java=A0=A0=A0 2009-06-19 09:00:29 UTC (rev 43261)
> +++ public/yanel/trunk/src/realms/from-scratch-realm-template/res-types/l=
ayoutselector/src/java/org/wyona/yanel/impl/resources/layoutselector/Layout=
Selector.java=A0=A0=A0 2009-06-19 09:16:29 UTC (rev 43262)
> @@ -1,5 +1,5 @@
>=A0 /*
> - * Copyright 2007 Wyona
> + * Copyright 2009 Wyona
>=A0=A0=A0*/
>=A0=A0=A0package org.wyona.yanel.impl.resources.layoutselector;
> @@ -47,19 +47,18 @@
>=A0 =A0 =A0 private static Logger log =3D Logger.getLogger(LayoutSelector.=
class);
>=A0 =A0 =A0 private String uploadMimeType;=A0 =A0 =A0 private String nodeV=
ersionChecker =3D"";
> -=A0 =A0 =A0 =A0 =A0 /**
>=A0 =A0 =A0=A0=A0* Constructor
> -=A0 =A0=A0=A0*=A0 =A0 =A0=A0=A0*/
>=A0 =A0 =A0 public LayoutSelector() {
>=A0 =A0 =A0 =A0=A0=A0 try {
> -=A0 =A0 =A0=A0=A0 =A0=A0=A0 nodeVersionChecker =3D=A0 getRealm().getName=
() + ":" + getResourceConfigProperty("layout_name") + ":xslt-version";
> -=A0 =A0 =A0=A0=A0 } catch(Exception ex) {
> -=A0 =A0 =A0=A0=A0 =A0=A0=A0 log.error("error initializing template versi=
on checker session string");
> +=A0 =A0 =A0 =A0 =A0 =A0 nodeVersionChecker =3D=A0 getRealm().getName() +=
 ":" + getResourceConfigProperty("layout_name") + ":xslt-version";
> +=A0 =A0 =A0=A0=A0 } catch(Exception e) {
> +=A0 =A0 =A0 =A0 =A0 =A0 log.error("error initializing template version c=
hecker session string");
> +=A0 =A0 =A0 =A0 =A0 =A0 log.error(e, e);
>=A0 =A0 =A0 =A0=A0=A0 }
> -=A0 =A0 };
> +=A0 =A0 }
>=A0 =A0 =A0=A0=A0/**
>=A0 =A0 =A0=A0=A0*
> =


Just curious: why not: log.error("error initializing template version check=
er session string", e ); instead?
-- Yanel-development mailing list Yanel-development at wyona.com
http://lists.wyona.org/cgi-bin/mailman/listinfo/yanel-development



      =

-------------- next part --------------
An HTML attachment was scrubbed...
URL: http://wyona.com/pipermail/yanel-development/attachments/20090619/b5a3=
4b0c/attachment.htm