Details

    • Type: Bug Fix
    • Status: Closed
    • Priority: 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
    • 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

        Attachments

          Issue Links

            Activity

            Hide
            samuel 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 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 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 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 Samuel Holtzkampf (Inactive) added a comment -

            Jerry mentioned you will look at this problem.

            Show
            samuel Samuel Holtzkampf (Inactive) added a comment - Jerry mentioned you will look at this problem.
            Hide
            mztaylor 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
            mztaylor 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:
                mztaylor Martin Taylor (Inactive)
                Reporter:
                samuel Samuel Holtzkampf (Inactive)
              • Votes:
                0 Vote for this issue
                Watchers:
                0 Start watching this issue

                Dates

                • Created:
                  Updated:
                  Resolved: