Display Org Unit ID in Org Unit Configuration

[Terran McCanna]

I frequently need to know the numeric ID of a particular org unit and it's frustrating that it's not available in the Org Unit Configuration interface.

Branch coming shortly.

[Terran McCanna (tmccanna)]

Initially I tried just displaying the id as a read-only field in the edit table which uses the aou class of the IDL - however, that class uses the shortname as the ID (at least, that's what it looks like to me), so it just displayed the shortname instead of the numeric id.

So, I added it after the shortname above the edit table instead. Not sure if there is a better way to do this - suggestions welcome.

https://git.evergreen-ils.org/?p=working/Evergreen.git;a=shortlog;h=refs/heads/user/mccanna/lp2051879_display_id_in_org_config

[Galen Charlton (gmc)]

I looked into why just displaying the id field in the form didn't work, and it turns out it's because the IDL is asserting that the type of actor.org_unit.id is itself "org_unit", i.e., a link to an org_unit. While it's been that way since 2009, I'm not sure that's correct. On the other hand, changing the datatype might have unforeseen consequences. Consequently, I've opened bug 2051944 for that issue.

Regardless, sticking the ID in the heading above the form works for me, so I've tested and signed off. Branch is user/gmcharlt/lp2051879_signoff

[Galen Charlton (gmc)]

Also, while this is a wishlist (the Dojo OU administration page didn't display the ID either), I think this one is small enough to backport to maintenance branches.

[Galen Charlton (gmc)]

Noting that if there's a strong preference to have the ID be displayed in the form (and that would be more consistent with other interfaces), using the customTemplate field option for the ID field would work.

[Galen Charlton (gmc)]

Terran, now that I've tested and signed off on Mike's patch for bug 2051944, would you be willing to try going back to displaying the ID in the form itself?

[Terran McCanna (tmccanna)]

I don't understand how the patch for bug 2051944 has any impact on this bug. It looks like it's just adding some read-only safety?

[Terran McCanna (tmccanna)]

Never mind my last comment, I think I got it...

[Terran McCanna (tmccanna)]

Okay, new branch based atop the patch for bug 2051944 - https://git.evergreen-ils.org/?p=working/Evergreen.git;a=shortlog;h=refs/heads/user/mccanna/lp2051879_display_org_id

[Galen Charlton (gmc)]

Tested the latest patch and force-pushed a signoff to working/user/gmcharlt/lp2051879_signoff

[Jane Sandberg (sandbergja)]

Pushed to rel_3_11 and above. Thanks, Terran and Galen!