Kuali Rice Development
  1. Kuali Rice Development
  2. KULRICE-4774

Too many and duplicate buttons on eDoc after save

    Details

    • Type: Bug Fix Bug Fix
    • Status: Closed Closed
    • Priority: Major Major
    • Resolution: Fixed
    • Affects Version/s: 1.0.3
    • Fix Version/s: 1.0.3
    • Component/s: Development
    • Labels:
      None
    • Similar issues:
      KULRICE-6326Too many entries in the namespace dropdown causes it to be unwieldy
      KULRICE-11786AFT Failure No values match this search no longer displayed in many tests, DemoLookUpOperatorsAft too, plus commas added to input fields
      KULRICE-7654Rice KIM documents display buttons such as "save" and "recall" even after submission
      KULRICE-12201Collection Group With Refresh has way too many rows
      KULRICE-4212KIM screens have too many namespace options
      KULRICE-13255KNS Maintenance Documents display Save button after a document submitted.
      KULRICE-4037KFS user preference records cause batch slowness due to too many krew_usr_optn_t records in Rice 1.0.1.1
      KULRICE-1061The Save button is disappearing after the doc has been saved
      KULRICE-3800Security / label checks are performed too many times during lookups
      KULRICE-7487Lookup links on Role document not functioning and appearing in too many places
    • Rice Module:
      KEW
    • KAI Review Status:
      Not Required
    • KTI Review Status:
      Not Required

      Description

      If initiating and eDoc and the user saves they are presented with complete, approve, submit, and couple other buttons get duplicated - it shows around 7 buttons. UMD reworked the code to remove duplicates and only show submit (out of submit,complete,approve) when the document is in a saved state. One side effect is that the complete button will never show, it is unclear how do determine if this button should show.
      Attached is a patch for WorkflowDocumentState.

        Issue Links

          Activity

          Hide
          Eric Westfall added a comment -

          Assigning this issue to myself. This patch was code reviewed with one of my edl developers here at IU. I will check it as well and then integrate and test.

          Show
          Eric Westfall added a comment - Assigning this issue to myself. This patch was code reviewed with one of my edl developers here at IU. I will check it as well and then integrate and test.
          Hide
          Eric Westfall added a comment -

          I took a look through the code and the logic. It looks good. Included my one comment on the code review:

          http://test.fisheye.kuali.org/cru/rice-9

          Basically it wasn't a problem with the patch, just the general way edl was doing this originally (i.e. edl really shouldn't have the logic in it that tries to decide what workflow actions are valid, KEW should be able to provide that to edl)

          Show
          Eric Westfall added a comment - I took a look through the code and the logic. It looks good. Included my one comment on the code review: http://test.fisheye.kuali.org/cru/rice-9 Basically it wasn't a problem with the patch, just the general way edl was doing this originally (i.e. edl really shouldn't have the logic in it that tries to decide what workflow actions are valid, KEW should be able to provide that to edl)
          Hide
          Eric Westfall added a comment -

          Code has been integrated and tested. Seems to work in our edoclite example.

          Show
          Eric Westfall added a comment - Code has been integrated and tested. Seems to work in our edoclite example.

            People

            • Assignee:
              Eric Westfall
              Reporter:
              Scott Gibson (Inactive)
            • Votes:
              0 Vote for this issue
              Watchers:
              0 Start watching this issue

              Dates

              • Created:
                Updated:
                Resolved:

                Structure Helper Panel