[KFSOLD-22637] REQS is broken - after any action is taken the document is coming back read only. Created: 15/Apr/13 Updated: 08/Feb/17 Resolved: 17/Apr/13 |
|
Status: | Closed |
Project: | KFS Archive |
Affects Version/s: | None |
Fix Version/s: | 5.0.x |
Type: | Code Cleanup | Priority: | Blocker |
Reporter: | Ailish Byrne | Assignee: | Bryan Hutchinson |
Resolution: | Fixed | Votes: | 0 |
Labels: | None | ||
Remaining Estimate: | Not Specified | ||
Time Spent: | Not Specified | ||
Original Estimate: | Not Specified |
Issue Links: |
|
||||||||
Sub-Committee: |
PUR / VND
|
||||||||
Impacted Modules: |
Purchasing / Accounts Payable
|
Description |
Create a Requisition Create a requisition Create a requisition |
Comments |
Comment by Bryan Hutchinson [ 15/Apr/13 ] |
It looks like this is related to |
Comment by Bryan Hutchinson [ 17/Apr/13 ] |
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. |
Comment by Bryan Hutchinson [ 17/Apr/13 ] |
I've committed a fix for this. It will be available to test in INT after tomorrow morning's rebuild. |