Not able to edit invoice item type Code

Bug #1860474 reported by Tiffany Little
8
This bug affects 1 person
Affects Status Importance Assigned to Milestone
Evergreen
Confirmed
Undecided
Unassigned

Bug Description

Tested on 3.4.1 and 3.3.

I am not able to edit existing invoice item types. The modal opens to edit, and I get the success toast when I hit Save, but no changes are made.

I'm able to create new item types fine.

Tags: acq acq-admin
Revision history for this message
Bill Erickson (berick) wrote :

Tiffany, can you confirm the issue relates specifically to modifying the invoice item type "Code" value? I was unable modify Code's in both the old and new invoice item type interfaces. I was able to edit other values, though.

Revision history for this message
Tiffany Little (tslittle) wrote :

Bill, you are correct! I'm able to change other values (prorate yes/no, and the label), but cannot edit the code. I don't think I ever changed an invoice item type code when it was in Dojo, so had no idea that wasn't possible.

Is there a reason we shouldn't be able to edit the code, or would that be the bug?

Revision history for this message
Tiffany Little (tslittle) wrote :

Oh, I see--the code is the primary key. Follow-up question: can we not just add an id field to that table so that the code is no longer the PK? That would be more consistent with other tables, and I can see wanting to edit the code.

Revision history for this message
Bill Erickson (berick) wrote :

Thanks for confirming, Tiffany.

Options that come to mind:

1. Make it non-changeable in the UI, requiring changes be made directly in the database. This is a trivial IDL change.

2. Add an API or stored procedure to support changing the value, which would also update any rows linked to the pkey from other tables (e.g. acq.invoice_item).

3. As suggested, add an ID surrogate key to the invoice_item_type table. This would resolve the update-ability but adds other complications at the SQL level. (We've been talking about dropping the ID from permission.perm_list table for a long time, because it complicates SQL upgrades).

Changed in evergreen:
status: New → Confirmed
tags: added: acq-admin
summary: - Not able to edit invoice item types
+ Not able to edit invoice item type Code
To post a comment you must log in.
This report contains Public information  
Everyone can see this information.

Other bug subscribers

Remote bug watches

Bug watches keep track of this bug in other bug trackers.