Uploaded image for project: 'KFS Archive'
  1. KFS Archive
  2. KFSOLD-22637

REQS is broken - after any action is taken the document is coming back read only.

          Details

          • Type: Code Cleanup
          • Status: Closed
          • Priority: Blocker
          • Resolution: Fixed
          • Affects Version/s: None
          • Fix Version/s: 5.0.x
          • Labels:
            None
          • Sub-Committee:
            PUR / VND
          • Impacted Modules:
            Purchasing / Accounts Payable

            Description

            Create a Requisition
            Entere a Description
            Click Save
            Document returned read only.

            Create a requisition
            Add a line
            Document returned read only

            Create a requisition
            Look up a value
            Document returned read only.

              Attachments

                Issue Links

                is related to

                Task - A task that needs to be done. KULRICE-9148 Disable SessionDocumentService in the KNS

                • Critical - Crashes, loss of data, severe memory leak.
                • Closed

                  Activity

                  Ascending order - Click to sort in descending order
                  Hide
                  Permalink
                  bhutchinson Bryan Hutchinson added a comment -

                  It looks like this is related to KULRICE-9148 which disabled the SessionDocumentService. Not sure why yet though or if it's a bug on the Rice side or KFS side.

                  Show
                  bhutchinson Bryan Hutchinson added a comment - It looks like this is related to KULRICE-9148 which disabled the SessionDocumentService. Not sure why yet though or if it's a bug on the Rice side or KFS side.
                  Hide
                  Permalink
                  bhutchinson Bryan Hutchinson added a comment -

                  The check to see if the REQS doc was editable was based on app doc status. When the document was initiated, the app doc status wasn't saved (it didn't used to be necessary to save the route header when the doc was initiated), so the app doc status wasn't available to check and the code determined the REQS doc shouldn't be editable. When the SessionDocumentService was enabled, this problem was masked because the doc, including the app doc status, was saved in the session.

                  To fix this, I modified the code (in FinancialSystemDocumentHeader.updateAndSaveAppDocStatus()) to save the routing data even when the doc is in initiated status. That way the app doc status is persisted, and is subsequently available to the canEdit check.

                  Show
                  bhutchinson Bryan Hutchinson added a comment - The check to see if the REQS doc was editable was based on app doc status. When the document was initiated, the app doc status wasn't saved (it didn't used to be necessary to save the route header when the doc was initiated), so the app doc status wasn't available to check and the code determined the REQS doc shouldn't be editable. When the SessionDocumentService was enabled, this problem was masked because the doc, including the app doc status, was saved in the session. To fix this, I modified the code (in FinancialSystemDocumentHeader.updateAndSaveAppDocStatus()) to save the routing data even when the doc is in initiated status. That way the app doc status is persisted, and is subsequently available to the canEdit check.
                  Hide
                  Permalink
                  bhutchinson Bryan Hutchinson added a comment -

                  I've committed a fix for this. It will be available to test in INT after tomorrow morning's rebuild.

                  Show
                  bhutchinson Bryan Hutchinson added a comment - I've committed a fix for this. It will be available to test in INT after tomorrow morning's rebuild.

                    People

                    • Assignee:
                      bhutchinson Bryan Hutchinson
                      Reporter:
                      abyrne Ailish Byrne
                    • Votes:
                      0 Vote for this issue
                      Watchers:
                      4 Start watching this issue

                      Dates

                      • Created:
                        Updated:
                        Resolved: