Changeset 1232 for trunk/autoquest-plugin-html/src/main/java/de/ugoe/cs/autoquest/plugin/html/HTMLLogParser.java

Timestamp:

06/27/13 09:16:35 (11 years ago)

Author:

pharms

Message:

• added some documentation
• corrected handling of unparseable file during compression

File:

• trunk/autoquest-plugin-html/src/main/java/de/ugoe/cs/autoquest/plugin/html/HTMLLogParser.java (modified) (6 diffs)

trunk/autoquest-plugin-html/src/main/java/de/ugoe/cs/autoquest/plugin/html/HTMLLogParser.java

@@ -47 +47 @@
     
     /**
-     *
+     * <p>
+     * the pattern used for parsing HTML GUI element paths
+     * </p>
      */
     private Pattern htmlElementPattern =
@@ -53 +55 @@
 
     /* (non-Javadoc)
-     * @see de.ugoe.cs.autoquest.plugin.html.AbstractDefaultLogParser#handleGUIElement(java.lang.String, java.util.Map)
+     * @see de.ugoe.cs.autoquest.plugin.html.AbstractDefaultLogParser#handleGUIElement(String, Map)
      */
     @Override
@@ -149 +151 @@
 
     /* (non-Javadoc)
-     * @see de.ugoe.cs.autoquest.plugin.html.AbstractDefaultLogParser#handleEvent(java.lang.String, java.util.Map)
+     * @see de.ugoe.cs.autoquest.plugin.html.AbstractDefaultLogParser#handleEvent(String, Map)
      */
     @Override
@@ -201 +203 @@
     /**
      * <p>
-     * TODO: comment
+     * used to determine the id of a target denoted by an event. This is only required for older
+     * document formats. The new formats use concrete ids.
      * </p>
-     *
-     * @param targetDocument
-     * @param targetDOMPath
-     * @return
      */
     private String determineTargetId(String targetDocument, String targetDOMPath)
@@ -280 +279 @@
      * </p>
      *
-     * @param tagName
-     * @return
+     * @param tagName the tag name to check
+     * 
+     * @return true, if the tag must be considered, false else
      */
     private boolean tagNameMustBeConsidered(String tagName) {
@@ -289 +289 @@
             !"iframe".equals(tagName) && !"input_hidden".equals(tagName) &&
             !"option".equals(tagName) && !"tt".equals(tagName) && !"br".equals(tagName) &&
-            !"colgroup".equals(tagName) && !"col".equals(tagName);
+            !"colgroup".equals(tagName) && !"col".equals(tagName) && !"hr".equals(tagName) &&
+            !"param".equals(tagName);
 
     }