All QC Sample ID are same cross worksheet for version 2.4.0+ about senaite.core HOT 13 CLOSED

Hi @erikking,
Can you share your ID Server configuration?
Thanks

from senaite.core.

Thanks for you response @ramonski ,

I used same ID server config for different version.

And for version 2.4.0 +, the ID server values referencesample-ECS always = 1 after add reference sample in worksheet.

from senaite.core.

Your split length must be 2, then it should work.

from senaite.core.

Ah, apologies, I was looking at the generated ID ECS-001-001 with the two dashes.
We will try to reproduce this issue according to your description to see what is going on there.
Thanks.

from senaite.core.

Thanks a lot for your quick response. My ultimate goal is to import lab result file for worksheet with reference sample. For version 2.3.0, I can use reference sample ID ECS-001-007 with lab value for worksheet WS-020 in imported csv file for the example above, then importing works well and the reference sample lab value will show in worksheet page, but when I upgrade to 2.4.0+, I cannot find a way to import reference sample lab value.

from senaite.core.

Hi @ramonski
I did further investigation in source code about the the issue, I found ReferenceAnalysesGroupID do not change when add new reference control in worksheet. And a potential cause may be related ReferenceAnalysesGroupID generation function nextRefAnalysesGroupID in worksheet.
Could you help to check with that?

from senaite.core.

Hi @ramonski
I thought I figured it out, there may two cause

• in line 435 ids = [int(i.split(prefix)[1]) for i in ids if i and rr.match(i)] the ids always return emtpy
• in line 415 ref_analysis.reindexObject(idxs=["getWorksheetUID", "getAnalyst"]) need add getReferenceAnalysesGroupID to build the index because nextRefAnalysesGroupID function call the index to get distinct count of reference analyses group

Hope it helps.

from senaite.core.

And also the ReferenceAnalysesGroupID is same for duplication as well in worksheet

from senaite.core.

Thanks @erikking for pointing out the code!
We will check this as soon as possible to find which fix is needed to make this work properly.
Best regards, Ramon

from senaite.core.

Fixed in #2388

from senaite.core.

Very nice to hear it got fixed. 👍

from senaite.core.

Please @erikking close this issue after you double check. Thanks

from senaite.core.

Sure, sorry for the late close.

from senaite.core.