Wicket
  1. Wicket
  2. WICKET-1634

ClassName needs conversion from Path to dotted notation in AutoLinkResolver

    Details

    • Type: Bug Bug
    • Status: Resolved
    • Priority: Major Major
    • Resolution: Fixed
    • Affects Version/s: 1.4-M1
    • Fix Version/s: 1.3.5, 1.4-RC1
    • Component/s: None
    • Labels:
      None

      Description

      Current svn snapshot contains the following code in AutoLinkResolver.newAutoComponent:

      if (clazz != null)
      {
      // Href is relative. Resolve the url given relative to
      // the current page
      className = Packages.absolutePath(clazz, pathInfo.path);

      try

      { clazz = defaultClassResolver.resolveClass(className); return new AutolinkBookmarkablePageLink(autoId, clazz, pathInfo.getPageParameters(), pathInfo.anchor); }

      catch (ClassNotFoundException ex)

      { log.warn("Did not find corresponding java class: " + className); // fall through }

      }

      This obviously fails since Packages.absolutePath() return a path whereas the class resolver expects a classname. Simply add the conversion as in the beginning of this method (or even better, extrract this conversion into an utility method)

        Issue Links

          Activity

          Transition Time In Source Status Execution Times Last Executer Last Execution Date
          Open Open Resolved Resolved
          60d 22h 46m 1 Igor Vaynberg 14/Jul/08 14:55
          Igor Vaynberg made changes -
          Assignee Igor Vaynberg [ ivaynberg ]
          Fix Version/s 1.3.5 [ 12313175 ]
          Status Open [ 1 ] Resolved [ 5 ]
          Resolution Fixed [ 1 ]
          Fix Version/s 1.4-M4 [ 12313295 ]
          Hide
          Igor Vaynberg added a comment -

          fixed via WICKET-1731

          Show
          Igor Vaynberg added a comment - fixed via WICKET-1731
          Roland Huss made changes -
          Field Original Value New Value
          Link This issue is blocked by WICKET-1731 [ WICKET-1731 ]
          Roland Huss created issue -

            People

            • Assignee:
              Igor Vaynberg
              Reporter:
              Roland Huss
            • Votes:
              0 Vote for this issue
              Watchers:
              1 Start watching this issue

              Dates

              • Created:
                Updated:
                Resolved:

                Development