Wink
  1. Wink
  2. WINK-316

AdminServlet doesn't process QueryParams properly

    Details

    • Type: Bug Bug
    • Status: Closed
    • Priority: Minor Minor
    • Resolution: Fixed
    • Affects Version/s: 1.1.1
    • Fix Version/s: 1.1.2
    • Component/s: Server
    • Labels:
      None
    • Patch Info:
      Patch Available

      Description

      The AdminServlet doesn't include parameters of type QueryParam in its output.
      The reason is that the code to do that uses MatrixParam instead.

      The method buildQueryParams handles query params and instead of checking for QueryParam it's checking for MatrixParam (the problem is the "==injectable.ParamType.MATRIX").

      Here is the fixed method:

      private void buildQueryParams(MethodMetadata methodMetadata, QueryParameters xmlQueryVariables) {
              List<Injectable> formalParameters = methodMetadata.getFormalParameters();
              for (Injectable var : formalParameters) {
                  if (var.getParamType() == Injectable.ParamType.QUERY) {
                      Parameter param = resourcesObjectFactory.createParameter();
                      param.setValue(((BoundInjectable)var).getName());
                      xmlQueryVariables.getParameter().add(param);
                  }
              }
          }
      
      1. AdminServlet.java
        23 kB
        Ruby Boyarski

        Activity

        Hide
        Hudson added a comment -

        Integrated in Wink-Trunk-JDK1.5 #390 (See https://hudson.apache.org/hudson/job/Wink-Trunk-JDK1.5/390/)
        Fix AdminServlet to process query parameters correctly

        See WINK-316

        Thanks to Ruby Boyarski for reporting the issue
        and the patch.

        Show
        Hudson added a comment - Integrated in Wink-Trunk-JDK1.5 #390 (See https://hudson.apache.org/hudson/job/Wink-Trunk-JDK1.5/390/ ) Fix AdminServlet to process query parameters correctly See WINK-316 Thanks to Ruby Boyarski for reporting the issue and the patch.
        Hide
        Bryant Luk added a comment -

        Patch looks good to me so I went ahead and applied it. Thanks for the patch.

        Show
        Bryant Luk added a comment - Patch looks good to me so I went ahead and applied it. Thanks for the patch.

          People

          • Assignee:
            Bryant Luk
            Reporter:
            Ruby Boyarski
          • Votes:
            0 Vote for this issue
            Watchers:
            1 Start watching this issue

            Dates

            • Created:
              Updated:
              Resolved:

              Time Tracking

              Estimated:
              Original Estimate - 2h
              2h
              Remaining:
              Remaining Estimate - 2h
              2h
              Logged:
              Time Spent - Not Specified
              Not Specified

                Development