67 2002-11-12 23:28:11 +0000 Unescaped ampersands not caught 2002-11-15 03:27:58 +0000 1 1 1 Unclassified Validator check 0.6.0b2 Other other RESOLVED REMIND P2 major --- 1 ville.skytta link oldest_to_newest 142 0 ville.skytta 2002-11-12 23:28:11 +0000 Reported by "elfin". Here's a couple of samples: <http://koti.welho.com/vskytta/foo.html> <http://koti.welho.com/vskytta/foo2.html> ...and a possible fix follows. Maybe the relationship between $File->{Is Valid} and $File->{Errors} needs checking, it makes the code pretty hard to read in various places. --- check 2002/11/11 23:31:31 1.291 +++ check 2002/11/12 22:07:13 @@ -568,7 +568,7 @@ if ($File->{ESIS}->[-1] =~ /^C$/) { undef $File->{ESIS}->[-1]; - $File->{'Is Valid'} = TRUE; + $File->{'Is Valid'} = scalar(@{$File->{Errors}}) ? FALSE : TRUE; } else { $File->{'Is Valid'} = FALSE; } 143 1 link 2002-11-14 22:27:58 +0000 Demoting a document to invalid based on these messages is not necessarily correct. They're tagged as "Warnings" from OpenSP; probably because an entity reference cannot be null and so these bare amperstands aren't actually entity references, they're just a character that normally introduces one. I've added these warnings to the output for Valid documents until we figure out what the canon has to say.