-
-
+/*
+ * Jalview - A Sequence Alignment Editor and Viewer ($$Version-Rel$$)
+ * Copyright (C) $$Year-Rel$$ The Jalview Authors
+ *
+ * This file is part of Jalview.
+ *
+ * Jalview is free software: you can redistribute it and/or
+ * modify it under the terms of the GNU General Public License
+ * as published by the Free Software Foundation, either version 3
+ * of the License, or (at your option) any later version.
+ *
+ * Jalview is distributed in the hope that it will be useful, but
+ * WITHOUT ANY WARRANTY; without even the implied warranty
+ * of MERCHANTABILITY or FITNESS FOR A PARTICULAR
+ * PURPOSE. See the GNU General Public License for more details.
+ *
+ * You should have received a copy of the GNU General Public License
+ * along with Jalview. If not, see <http://www.gnu.org/licenses/>.
+ * The Jalview Authors are detailed in the 'AUTHORS' file.
+ */
import java.io.BufferedReader;
import java.io.File;
import java.io.FileReader;
* @author gmcarstairs
*
*/
-public class HelpLinksChecker
+public class HelpLinksChecker implements BufferedLineReader.LineCleaner
{
private static final String HELP_HS = "help.hs";
private int anchorRefCount = 0;
+ private int invalidAnchorRefCount = 0;
+
private int externalHrefCount = 0;
private int invalidMapUrlCount = 0;
if (args.length == 0 || args.length > 2
|| (args.length == 2 && !args[1].equals("-nointernet")))
{
- System.out.println("Usage: <pathToHelpFolder> [-nointernet]");
+ log("Usage: <pathToHelpFolder> [-nointernet]");
return;
}
*/
void checkLinks(String helpDirectoryPath) throws IOException
{
- System.out.println("Checking help file links");
- File helpFolder = new File(helpDirectoryPath);
+ log("Checking help file links");
+ File helpFolder = new File(helpDirectoryPath).getCanonicalFile();
if (!helpFolder.exists())
{
- System.out.println("Can't find " + helpDirectoryPath);
+ log("Can't find " + helpDirectoryPath);
return;
}
unusedTargets.remove(image);
if (!tocTargets.containsKey(image))
{
- System.out.println(String.format(
- "Invalid image '%s' at line %d of %s", image, lineNo,
- HELP_HS));
+ log(String.format("Invalid image '%s' at line %d of %s", image,
+ lineNo, HELP_HS));
invalidImageCount++;
}
}
*/
private void reportResults(Map<String, String> unusedTargets)
{
- System.out.println("\nResults:");
- System.out.println(targetCount + " distinct help targets");
- System.out.println(mapCount + " help mappings");
- System.out.println(invalidTargetCount + " invalid targets");
- System.out.println(unusedTargets.size() + " unused targets");
+ log("\nResults:");
+ log(targetCount + " distinct help targets");
+ log(mapCount + " help mappings");
+ log(invalidTargetCount + " invalid targets");
+ log(unusedTargets.size() + " unused targets");
for (String target : unusedTargets.keySet())
{
- System.out.println(String.format(" %s: %s", target,
- unusedTargets.get(target)));
+ log(String.format(" %s: %s", target, unusedTargets.get(target)));
}
- System.out.println(invalidMapUrlCount + " invalid map urls");
- System.out.println(invalidImageCount + " invalid image attributes");
- System.out.println(String.format(
- "%d internal href links (%d with anchors - not checked)",
+ log(invalidMapUrlCount + " invalid map urls");
+ log(invalidImageCount + " invalid image attributes");
+ log(String.format("%d internal href links (%d with anchors)",
internalHrefCount, anchorRefCount));
- System.out.println(invalidInternalHrefCount
- + " invalid internal href links");
- System.out.println(externalHrefCount + " external href links");
+ log(invalidInternalHrefCount + " invalid internal href links");
+ log(invalidAnchorRefCount + " invalid internal anchor links");
+ log(externalHrefCount + " external href links");
if (internetAvailable)
{
- System.out.println(invalidExternalHrefCount
- + " invalid external href links");
+ log(invalidExternalHrefCount + " invalid external href links");
}
else
{
System.out
.println("External links not verified as internet not available");
}
+ if (invalidInternalHrefCount > 0 || invalidExternalHrefCount > 0
+ || invalidImageCount > 0 || invalidAnchorRefCount > 0)
+ {
+ log("*** Failed ***");
+ System.exit(1);
+ }
+ log("*** Success ***");
+ }
+ /**
+ * @param s
+ */
+ static void log(String s)
+ {
+ System.out.println(s);
}
/**
else
{
internalHrefCount++;
+ String relFile = System.getProperty("os.name").indexOf("Win") > -1 ? href.replace("/", File.separator) : href;
File hrefFile = href.equals("") ? htmlFile : new File(htmlFolder,
href);
- if (!hrefFile.exists())
+ if (hrefFile != htmlFile && !fileExists(hrefFile, relFile))
{
badLink = true;
invalidInternalHrefCount++;
{
if (!checkAnchorExists(hrefFile, anchor))
{
- System.out.println(String.format(
- "Invalid anchor: %s at line %d of %s", anchor,
- lineNo, getPath(htmlFile)));
+ log(String.format("Invalid anchor: %s at line %d of %s",
+ anchor, lineNo, getPath(htmlFile)));
+ invalidAnchorRefCount++;
}
}
}
}
if (badLink)
{
- System.out.println(String.format(
- "Invalid href %s at line %d of %s", href, lineNo,
- getPath(htmlFile)));
+ log(String.format("Invalid href %s at line %d of %s", href,
+ lineNo, getPath(htmlFile)));
}
}
data = br.readLine();
}
/**
+ * Performs a case-sensitive check that the href'd file exists
+ *
+ * @param hrefFile
+ * @return
+ * @throws IOException
+ */
+ boolean fileExists(File hrefFile, String href) throws IOException
+ {
+ if (!hrefFile.exists())
+ {
+ return false;
+ }
+
+ /*
+ * On Mac or Windows, file.exists() is not case sensitive, so do an
+ * additional check with case sensitivity
+ */
+ int slashPos = href.lastIndexOf(File.separator);
+ String expectedFileName = slashPos == -1 ? href : href
+ .substring(slashPos + 1);
+ String cp = hrefFile.getCanonicalPath();
+ slashPos = cp.lastIndexOf(File.separator);
+ String actualFileName = slashPos == -1 ? cp : cp
+ .substring(slashPos + 1);
+
+ return expectedFileName.equals(actualFileName);
+ }
+
+ /**
* Reads the file and checks for the presence of the given html anchor
*
* @param hrefFile
try
{
BufferedReader br = new BufferedReader(new FileReader(hrefFile));
- String data = br.readLine();
+ BufferedLineReader blr = new BufferedLineReader(br, 3, this);
+ String data = blr.read();
while (data != null)
{
if (data.contains(nameAnchor) || data.contains(idAnchor))
found = true;
break;
}
- data = br.readLine();
+ data = blr.read();
}
br.close();
} catch (IOException e)
mapCount++;
if (targets.containsKey(target))
{
- System.out.println(String.format(
+ log(String.format(
"Duplicate target mapping to %s at line %d of %s",
target, lineNo, HELP_JHM));
}
}
if (!new File(helpFolder, url).exists())
{
- System.out.println(String.format(
- "Invalid url path '%s' at line %d of %s", url, lineNo,
- HELP_JHM));
+ log(String.format("Invalid url path '%s' at line %d of %s", url,
+ lineNo, HELP_JHM));
invalidMapUrlCount++;
}
}
unusedTargets.remove(target);
if (!tocTargets.containsKey(target))
{
- System.out.println(String.format(
- "Invalid target '%s' at line %d of %s", target, lineNo,
- HELP_TOC_XML));
+ log(String.format("Invalid target '%s' at line %d of %s", target,
+ lineNo, HELP_TOC_XML));
invalidTargetCount++;
}
}
}
return value;
}
+
+ /**
+ * Trim whitespace from concatenated lines but preserve one space for valid
+ * parsing
+ */
+ @Override
+ public String cleanLine(String l)
+ {
+ return l.trim() + " ";
+ }
}