You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@turbine.apache.org by oron ogdan <or...@netada.co.uk> on 2004/01/22 00:08:24 UTC
Bug in VelocityHtmlEmail class ???
Hi
Not sure how to officially report bugs to turbine as I didn't find an
'apache bug database' entry for it.
Anyway, here is my little contribution to this top quality project
seems like someone forgot to populate the hashmap for the cid entries
BR
Oron
http://netada.co.uk
patch follows :
---
D:\dev\jakarta-turbine-2.3.1-dev-20031124\src\java\org\apache\turbine\util\v
elocity\VelocityHtmlEmail.java 2004-01-21 22:57:50.000000000 -0000
+++
D:\dev\jakarta-turbine-2.3.1-dev-20031124\src\java\org\apache\turbine\util\v
elocity\VelocityHtmlEmail.java.bak 2004-01-21 22:58:03.000000000 -0000
@@ -280,13 +280,12 @@
{
String cid = "";
try
{
URL url = new URL(surl);
cid = embed(url, name);
- embmap.put(name, cid);
}
catch (Exception e)
{
log.error("cannot embed " + surl + ": ", e);
}
return cid;
---------------------------------------------------------------------
To unsubscribe, e-mail: turbine-dev-unsubscribe@jakarta.apache.org
For additional commands, e-mail: turbine-dev-help@jakarta.apache.org
RE: Bug in VelocityHtmlEmail class ???
Posted by Eric Pugh <ep...@upstate.com>.
Yes, that is always a problem in Open source stuff, what version do you use!
I am not big on backporting every bugfix into the old code, so the 2.3.1 has
most of the fixes, but not all. If a bug was fixed by looking at 2.3.1,
then 2.3.1 has the fix. If the bug was fixed in 2.4, then probably not. I
am more focused on getting 2.4 out the door.
As far as bug trackers go, Jakarta is switching to Jira, however we haven't
followed up on what to do to get turbine up on that yet...
Eric
> -----Original Message-----
> From: oron ogdan [mailto:oron.ogdan@netada.co.uk]
> Sent: Thursday, January 22, 2004 11:13 AM
> To: 'Turbine Developers List'; epugh@upstate.com
> Subject: RE: Bug in VelocityHtmlEmail class ???
>
>
> Thank you Eric
>
> This brings me to some more questions which you might be able
> to help me
> with
>
> -- what's the best place to report bugs (is it the scarab
> tool at nagoya ?)
>
> -- what's the best approach for me as a user of turbine in
> terms of interim
> release usage ?, I know release early and often is the key to
> the success of
> such projects. And I am interested in working with the latest
> and greatest
> as long as it does not break my app. I was running until last
> night on T2.3
> when I discovered this bug and decided to go through the
> maven build route
> recompile T2.3.1, which seems to work and didn't break any of
> my current
> code. However you say this specific bug has been fixed in
> 2.4, so from this
> I understand 2.3.1-dev does not have all the latest bug fixes
> in there.
>
> Oron
>
> -----Original Message-----
> From: Eric Pugh [mailto:epugh@upstate.com]
> Sent: Thursday, January 22, 2004 9:55 AM
> To: 'Turbine Developers List'; oron.ogdan@netada.co.uk
> Subject: RE: Bug in VelocityHtmlEmail class ???
>
>
> I think this bug was fixed in 2.4.. I checked it, and it looks good
> there..
>
> Eric
>
> > -----Original Message-----
> > From: oron ogdan [mailto:oron.ogdan@netada.co.uk]
> > Sent: Thursday, January 22, 2004 12:08 AM
> > To: 'Turbine Developers List'
> > Subject: Bug in VelocityHtmlEmail class ???
> >
> >
> > Hi
> >
> > Not sure how to officially report bugs to turbine as I
> didn't find an
> > 'apache bug database' entry for it.
> > Anyway, here is my little contribution to this top quality project
> >
> > seems like someone forgot to populate the hashmap for the
> cid entries
> >
> > BR
> >
> > Oron
> >
> > http://netada.co.uk
> >
> > patch follows :
> >
> > ---
> > D:\dev\jakarta-turbine-2.3.1-dev-20031124\src\java\org\apache\
> > turbine\util\v
> > elocity\VelocityHtmlEmail.java 2004-01-21
> > 22:57:50.000000000 -0000
> > +++
> > D:\dev\jakarta-turbine-2.3.1-dev-20031124\src\java\org\apache\
> > turbine\util\v
> > elocity\VelocityHtmlEmail.java.bak 2004-01-21
> > 22:58:03.000000000 -0000
> > @@ -280,13 +280,12 @@
> > {
> > String cid = "";
> > try
> > {
> > URL url = new URL(surl);
> > cid = embed(url, name);
> > - embmap.put(name, cid);
> > }
> > catch (Exception e)
> > {
> > log.error("cannot embed " + surl + ": ", e);
> > }
> > return cid;
> >
> >
> >
> ---------------------------------------------------------------------
> > To unsubscribe, e-mail: turbine-dev-unsubscribe@jakarta.apache.org
> > For additional commands, e-mail: turbine-dev-help@jakarta.apache.org
>
>
> ---------------------------------------------------------------------
> To unsubscribe, e-mail: turbine-dev-unsubscribe@jakarta.apache.org
> For additional commands, e-mail: turbine-dev-help@jakarta.apache.org
>
>
> ---------------------------------------------------------------------
> To unsubscribe, e-mail: turbine-dev-unsubscribe@jakarta.apache.org
> For additional commands, e-mail: turbine-dev-help@jakarta.apache.org
---------------------------------------------------------------------
To unsubscribe, e-mail: turbine-dev-unsubscribe@jakarta.apache.org
For additional commands, e-mail: turbine-dev-help@jakarta.apache.org
RE: Bug in VelocityHtmlEmail class ???
Posted by oron ogdan <or...@netada.co.uk>.
Thank you Eric
This brings me to some more questions which you might be able to help me
with
-- what's the best place to report bugs (is it the scarab tool at nagoya ?)
-- what's the best approach for me as a user of turbine in terms of interim
release usage ?, I know release early and often is the key to the success of
such projects. And I am interested in working with the latest and greatest
as long as it does not break my app. I was running until last night on T2.3
when I discovered this bug and decided to go through the maven build route
recompile T2.3.1, which seems to work and didn't break any of my current
code. However you say this specific bug has been fixed in 2.4, so from this
I understand 2.3.1-dev does not have all the latest bug fixes in there.
Oron
-----Original Message-----
From: Eric Pugh [mailto:epugh@upstate.com]
Sent: Thursday, January 22, 2004 9:55 AM
To: 'Turbine Developers List'; oron.ogdan@netada.co.uk
Subject: RE: Bug in VelocityHtmlEmail class ???
I think this bug was fixed in 2.4.. I checked it, and it looks good
there..
Eric
> -----Original Message-----
> From: oron ogdan [mailto:oron.ogdan@netada.co.uk]
> Sent: Thursday, January 22, 2004 12:08 AM
> To: 'Turbine Developers List'
> Subject: Bug in VelocityHtmlEmail class ???
>
>
> Hi
>
> Not sure how to officially report bugs to turbine as I didn't find an
> 'apache bug database' entry for it.
> Anyway, here is my little contribution to this top quality project
>
> seems like someone forgot to populate the hashmap for the cid entries
>
> BR
>
> Oron
>
> http://netada.co.uk
>
> patch follows :
>
> ---
> D:\dev\jakarta-turbine-2.3.1-dev-20031124\src\java\org\apache\
> turbine\util\v
> elocity\VelocityHtmlEmail.java 2004-01-21
> 22:57:50.000000000 -0000
> +++
> D:\dev\jakarta-turbine-2.3.1-dev-20031124\src\java\org\apache\
> turbine\util\v
> elocity\VelocityHtmlEmail.java.bak 2004-01-21
> 22:58:03.000000000 -0000
> @@ -280,13 +280,12 @@
> {
> String cid = "";
> try
> {
> URL url = new URL(surl);
> cid = embed(url, name);
> - embmap.put(name, cid);
> }
> catch (Exception e)
> {
> log.error("cannot embed " + surl + ": ", e);
> }
> return cid;
>
>
> ---------------------------------------------------------------------
> To unsubscribe, e-mail: turbine-dev-unsubscribe@jakarta.apache.org
> For additional commands, e-mail: turbine-dev-help@jakarta.apache.org
---------------------------------------------------------------------
To unsubscribe, e-mail: turbine-dev-unsubscribe@jakarta.apache.org
For additional commands, e-mail: turbine-dev-help@jakarta.apache.org
---------------------------------------------------------------------
To unsubscribe, e-mail: turbine-dev-unsubscribe@jakarta.apache.org
For additional commands, e-mail: turbine-dev-help@jakarta.apache.org
RE: Bug in VelocityHtmlEmail class ???
Posted by Eric Pugh <ep...@upstate.com>.
I think this bug was fixed in 2.4.. I checked it, and it looks good
there..
Eric
> -----Original Message-----
> From: oron ogdan [mailto:oron.ogdan@netada.co.uk]
> Sent: Thursday, January 22, 2004 12:08 AM
> To: 'Turbine Developers List'
> Subject: Bug in VelocityHtmlEmail class ???
>
>
> Hi
>
> Not sure how to officially report bugs to turbine as I didn't find an
> 'apache bug database' entry for it.
> Anyway, here is my little contribution to this top quality project
>
> seems like someone forgot to populate the hashmap for the cid entries
>
> BR
>
> Oron
>
> http://netada.co.uk
>
> patch follows :
>
> ---
> D:\dev\jakarta-turbine-2.3.1-dev-20031124\src\java\org\apache\
> turbine\util\v
> elocity\VelocityHtmlEmail.java 2004-01-21
> 22:57:50.000000000 -0000
> +++
> D:\dev\jakarta-turbine-2.3.1-dev-20031124\src\java\org\apache\
> turbine\util\v
> elocity\VelocityHtmlEmail.java.bak 2004-01-21
> 22:58:03.000000000 -0000
> @@ -280,13 +280,12 @@
> {
> String cid = "";
> try
> {
> URL url = new URL(surl);
> cid = embed(url, name);
> - embmap.put(name, cid);
> }
> catch (Exception e)
> {
> log.error("cannot embed " + surl + ": ", e);
> }
> return cid;
>
>
> ---------------------------------------------------------------------
> To unsubscribe, e-mail: turbine-dev-unsubscribe@jakarta.apache.org
> For additional commands, e-mail: turbine-dev-help@jakarta.apache.org
---------------------------------------------------------------------
To unsubscribe, e-mail: turbine-dev-unsubscribe@jakarta.apache.org
For additional commands, e-mail: turbine-dev-help@jakarta.apache.org