Details

    • Type: Bug Fix Bug Fix
    • Status: Closed Closed
    • Priority: Major Major
    • Resolution: Fixed
    • Affects Version/s: 2.2
    • Fix Version/s: 2.2.0-m1, 2.2
    • Component/s: Development
    • Security Level: Public (Public: Anyone can view)
    • Labels:
      None
    • Similar issues:
      KULRICE-7223Multivalue lookup not returning results
      KULRICE-124012.4.0 CDT: Multivalue Lookup returned incident report on return selected
      KULRICE-3219Move 'return selected' button location on multivalue lookups
      KULRICE-11214MultiValue Lookup - select only on this page and then deselect only on this page the return select button stays clickable.
      KULRICE-10302Analysis - Results not on the current page are not returned by multivalue lookup's "return selected" action
      KULRICE-10350Determine whether multivalue lookups can be used for EBOs
      KULRICE-9938Field values for a multivalue lookup aren't returned correctly if lookup results fields are null
      KULRICE-7283Returning nothing from multi-value lookup still refreshes the page
      KULRICE-3616Return Selected button on multivalue lookup has black border
      KULRICE-6934Lookup does close lightbox when returning results
    • Rice Module:
      KRAD
    • KAI Review Status:
      Not Required
    • KTI Review Status:
      Not Required

      Description

      Multi value lookup does not refresh page when return button is selected

        Issue Links

          Activity

          Hide
          Samuel Holtzkampf (Inactive) added a comment -

          Seeing different behaviour in different browsers :
          Chrome : returnSelected method on LookupController does not even get called
          Firefox : returnSelected gets called but returnLocation is null on form and causes exception
          IE 9 : returnSelected gets called but returnLocation is null on form and causes exception (NOTE : collection page unworkable lagging on IE )

          Show
          Samuel Holtzkampf (Inactive) added a comment - Seeing different behaviour in different browsers : Chrome : returnSelected method on LookupController does not even get called Firefox : returnSelected gets called but returnLocation is null on form and causes exception IE 9 : returnSelected gets called but returnLocation is null on form and causes exception (NOTE : collection page unworkable lagging on IE )
          Hide
          Samuel Holtzkampf (Inactive) added a comment -

          Looks like the returnLocation is empty because lightbox containing lookup form is closed before returnSelected is called. Removing closeLightBox() changes chrome behaviour to actually reach controller method.

          Show
          Samuel Holtzkampf (Inactive) added a comment - Looks like the returnLocation is empty because lightbox containing lookup form is closed before returnSelected is called. Removing closeLightBox() changes chrome behaviour to actually reach controller method.
          Hide
          Samuel Holtzkampf (Inactive) added a comment -

          Jerry mentioned you will look at this problem.

          Show
          Samuel Holtzkampf (Inactive) added a comment - Jerry mentioned you will look at this problem.
          Hide
          Martin Taylor (Inactive) added a comment -

          As mentioned the javascript for closing the box is coming too early. Commenting out the js in the krad.widget.js:closeLightbox allows it to save.

          When reviewing the rendering of the ActionButton (returnSelected) noticed its lightBoxLookup element was set to null. In the action:performFinalize it has a check if this is null or not, if it is it says not to use the onclick calls. Wondering if these field should be set/inherit a value for lightBoxLookup.

          Comments from Action.java code:

          • When there is a light box - don't add the on click script as it will be prevented from executing. Create a script map object which will be written to the form on click event
          Show
          Martin Taylor (Inactive) added a comment - As mentioned the javascript for closing the box is coming too early. Commenting out the js in the krad.widget.js:closeLightbox allows it to save. When reviewing the rendering of the ActionButton (returnSelected) noticed its lightBoxLookup element was set to null. In the action:performFinalize it has a check if this is null or not, if it is it says not to use the onclick calls. Wondering if these field should be set/inherit a value for lightBoxLookup. Comments from Action.java code: When there is a light box - don't add the on click script as it will be prevented from executing. Create a script map object which will be written to the form on click event

            People

            • Assignee:
              Martin Taylor (Inactive)
              Reporter:
              Samuel Holtzkampf (Inactive)
            • Votes:
              0 Vote for this issue
              Watchers:
              0 Start watching this issue

              Dates

              • Created:
                Updated:
                Resolved:

                Structure Helper Panel