Uploaded image for project: 'WINDUP - Red Hat Application Migration Toolkit'
  1. WINDUP - Red Hat Application Migration Toolkit
  2. WINDUP-1155

Windup returning wrong column/length for a rule

    XMLWordPrintable

Details

    • Bug
    • Resolution: Unresolved
    • Major
    • None
    • None
    • None

    Description

      from josteele

      Here's my problem. I'm running Windup on an XHMTL file, that contains a Seam tag that needs to be migrated.

      <!DOCTYPE composition PUBLIC "-//W3C//DTD XHTML 1.0 Transitional//EN" "http://www.w3.org/TR/xhtml1/DTD/xhtml1-transitional.dtd">
      <ui:composition xmlns="http://www.w3.org/1999/xhtml"
      	    		xmlns:ui="http://java.sun.com/jsf/facelets"
      	  			xmlns:h="http://java.sun.com/jsf/html"
      	  			xmlns:f="http://java.sun.com/jsf/core"
      	  			xmlns:s="http://jboss.com/products/seam/taglib"
                      xmlns:a="http://richfaces.org/a4j"
      				template="template.xhtml">
      <s:link id="MoreResultsLink" value="More results" action="#{hotelSearch.nextPage}" rendered="#{hotelSearch.nextPageAvailable}"/>
      
      </ui:composition>
      

      Once Windup completes, the tooling API says the error is on line 9, column 129, and length is 12. However, the only thing
      that seems correct is the line number.

      I'm not sure if I'm misinterpreting the results, but the way I see it is the length is 6, the line number is 9, and
      column is 2 (maybe we're computing the column to mean the number of characters leading up to the String starting from
      the beginning of the file?).

      Here's the rule:

      <quickfix name="slink-qf" type="REPLACE">
      <replacement>h:link</replacement>
      <search>s:link</search>
      </quickfix>

      Of course in order for me to apply the quick fix using String replacement, the column and length need to be exact.

      Attachments

        Activity

          People

            Unassigned Unassigned
            dansmit Dan Smith (Inactive)
            Votes:
            0 Vote for this issue
            Watchers:
            1 Start watching this issue

            Dates

              Created:
              Updated: