Hi Biljana 

> -----Original Message-----
> From: [EMAIL PROTECTED] 
> [mailto:[EMAIL PROTECTED] On Behalf Of Biba
> Sent: donderdag 18 september 2008 13:50
> To: 'Hippo CMS development public mailinglist'
> Subject: [HippoCMS-dev] Check code
> 
> Hello,
> 
> I have found something strange in the hca-core code (at least 
> it's strange to me :-)).
> Class
> nl.hippo.hca.notifier.mailer.services.impl.NotifierMailerServi
> ceImpl.java
> Method:
> private boolean checkNotifierTemplates(Notifier notifier) {
>         try {
>             User user = new UserImpl();
> 
>             NotificationSnippet snippet = new 
> NotificationSnippetImpl(0, "", 
> notifier.getSnippetTextTemplate(), notifier.getSnippetTextTemplate());
> 
> Shouldn't it be a html argument instead of second text 
> argument - notifier.getSnippetHTMLTemplate ()? 

You are absolutely right! Thanks for reporting this bug. I've committed
the change in the trunk.

Regards,


Jasha Joachimsthal 
 
[EMAIL PROTECTED] - [EMAIL PROTECTED]
 
www.onehippo.com
Amsterdam - Hippo B.V. Oosteinde 11 1017 WT Amsterdam +31(0)20-5224466 
San Francisco - Hippo USA Inc. 101 H Street, suite Q Petaluma CA
94952-3329 +1 (707) 773-4646

********************************************
Hippocms-dev: Hippo CMS development public mailinglist

Searchable archives can be found at:
MarkMail: http://hippocms-dev.markmail.org
Nabble: http://www.nabble.com/Hippo-CMS-f26633.html

Reply via email to