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

          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