Issue 53586: Domain field with long name can result in "PropertyURI cannot exceed 300 characters, but was 302 characters long."

[cnathe]

Rationale

https://www.labkey.org/home/Developer/issues/issues-details.view?issueId=53586

Several places in the code generated PropertyURIs for a domain field by appending the encoded field name to the domain type URI prefix. This can create long PropertyURI values that exceed our 300 char limit. This PR updates the DomainUtil.createUniquePropertyURI() helper to replace the "encode field name" with a Lsid DB seq as we've moved towards for other URI related cases recently.

Related Pull Requests

• Issue 52339: Try to use shorter URIs for domains when possible #6805
• Issue 53586: Domain field with long name can result in "PropertyURI cannot exceed 300 characters, but was 302 characters long." #6939
• Issue 53586: Domain field with long name can result in "PropertyURI cannot exceed 300 characters, but was 302 characters long." testAutomation#2646
• https://github.com/LabKey/clientModules/pull/56
• Issue 53586: Domain field with long name can result in "PropertyURI cannot exceed 300 characters, but was 302 characters long." commonAssays#920

Changes

• Update DomainUtil.createUniquePropertyURI() to use LsidManager.getLsidPrefixDbSeq()
• Use DomainUtil.createUniquePropertyURI() in several places instead of appending encoded field name

[labkey-klum]

I pulled this down and ran with it locally. I'd be fine relying on the automated tests over a manual test pass.

[labkey-jeckels]

This behavior seems strange now. It's not really returning the property name for (most) new-style PropertyURIs

[cnathe]

@labkey-jeckels I agree but I don't think this code can actually be hit for any newly created properties. It is executed for Sample Types that have the parentCol, idCol1, idCol2, idCol3 behavior which is deprecated. I couldn't find a way in the UI to create one of these. I took the XAR file from that issue 30718 and imported it locally and this code did the right thing for those PropertyURIs in the XAR. Thoughts on other ways to test this? Since you can't create these types of Sample Types anymore, I think the code should continue working for any existing property descriptors in those deprecated sample types.

[labkey-jeckels]

Got it. I hadn't looked at the usages. Do you think we should mark this or related methods as deprecated?

[cnathe]

Yeah, I'll take a look