Details

    • Type: Sub Task
    • Status: Closed
    • Priority: Major
    • Resolution: Fixed
    • Affects Version/s: None
    • Fix Version/s: 2.3.0-m2, 2.3
    • Component/s: Roadmap, User Interface
    • Security Level: Public (Public: Anyone can view)
    • Labels:
      None
    • Rice Module:
      KRAD
    • Application Requirement:
      Rice
    • KAI Review Status:
      Not Required
    • KTI Review Status:
      Not Required
    • Include in Release Notes?:
      Yes

      Description

      From Jerry:
      " There is an issue with history when we go directly do a KRAD page. When going to directly a page, the CAS login comes up, then after login redirects page to the requested page. However, this does a POST instead GET. So when this method is called on History:

      public void setCurrent(UifFormBase form, HttpServletRequest request) {
      if (!request.getMethod().equals("POST")) {
      boolean showHomeValue = false;
      boolean pageIdValue = false;
      boolean formKeyValue = false;

      String queryString = "";
      String url = request.getRequestURL().toString();

      Since the method is POST it gets skipped over and current is not set. Then when the page is rendering and the call is made to this method in History:

      public HistoryEntry getGeneratedCurrentBreadcrumb() {
      HistoryEntry breadcrumb = copyEntry(current);
      String historyParam = "";

      current is null and a NPE is thrown."

        Attachments

          Issue Links

            Activity

            Hide
            bsmith Brian Smith (Inactive) added a comment -

            I can't reproduce this issue

            Show
            bsmith Brian Smith (Inactive) added a comment - I can't reproduce this issue
            Hide
            jkneal Jerry Neal (Inactive) added a comment -

            Hey Brian,

            I am going to bring in the sports app this weekend and you should be able to reproduce the issue from there.

            Jerry

            Show
            jkneal Jerry Neal (Inactive) added a comment - Hey Brian, I am going to bring in the sports app this weekend and you should be able to reproduce the issue from there. Jerry
            Hide
            bsmith Brian Smith (Inactive) added a comment -

            should be fixed by History removal/breadcrumb rewrite

            Show
            bsmith Brian Smith (Inactive) added a comment - should be fixed by History removal/breadcrumb rewrite

              People

              • Assignee:
                Unassigned
                Reporter:
                bsmith Brian Smith (Inactive)
              • Votes:
                0 Vote for this issue
                Watchers:
                2 Start watching this issue

                Dates

                • Created:
                  Updated:
                  Resolved: