That's it! Thank you, I'll try your patch tomorrow, ok?

Regards Ard


> 
> Probably not want you wanted. Maybe this is better?
> 
> On Nov 15, 2007 2:55 PM, nathalie looijmans 
> <[EMAIL PROTECTED]> wrote:
> > I dont know, i tried.
> >
> >
> > On Nov 15, 2007 2:48 PM, Ard Schrijvers 
> <[EMAIL PROTECTED]> wrote:
> > > Good work Nathalie,
> > >
> > > Do you know how to make a patch (svn diff > myname.patch)? It is 
> > > easier for us to check out your solution.
> > >
> > > Thx,
> > >
> > > Ard
> > >
> > >
> > > >
> > > > Hi,
> > > >
> > > > There's a bug in the bug in the broken link checker. When the 
> > > > broken link checker checks on an empty link the broken 
> checker dies.
> > > >
> > > > When the method checkExternalLink(String link) of the class 
> > > > BrokenLinkCheckTask gets an extra check this problem 
> will be solved:
> > > >
> > > > just after the first try:
> > > >
> > > >                       if
> > > > (normalizeExternalUri(link).equals("http://";)) {
> > > >                               Element el = new Element("text", 
> > > > "i18n", I18N_NS_URI);
> > > >                               el.addContent(HTTP_ERROR_PREFIX
> > > > + new Integer(400).toString());
> > > >                               return el;
> > > >                       }
> > > >
> > > >
> > > > Maybe there should also be an check on https or another 
> protocol?
> > > >
> > > >
> > > > Regards, Nathalie
> > > > ********************************************
> > > > Hippocms-dev: Hippo CMS development public mailinglist
> > > >
> > > ********************************************
> > > Hippocms-dev: Hippo CMS development public mailinglist
> > >
> >
> 
********************************************
Hippocms-dev: Hippo CMS development public mailinglist

Reply via email to