senaite / senaite.core

Enterprise Open Source Laboratory System (LIMS)
https://senaite.com
GNU General Public License v2.0
254 stars 145 forks source link

Fix dynamic analysisspecs are not linked to keywords containing only numbers #2495

Closed ramonski closed 8 months ago

ramonski commented 8 months ago

Description of the issue/feature this PR addresses

This PR fixes an issue where a linked dynamic analysis specification does not hook to analysis service keywords if they consist only of numbers, e.g. 413.

Current behavior before PR

Cannot add Analysis Specifications based on Dynamic Analysis Specification sheet for certain analyses · Issue #96 · ridingbytes… 2024-02-19 11 AM-11-16 Cannot add Analysis Specifications based on Dynamic Analysis Specification sheet for certain analyses · Issue #96 · ridingbytes… 2024-02-19 11 AM-11-42

Desired behavior after PR is merged

Dynamic analysisspecs also hook services with number keywords

Dynamic spes

-- I confirm I have tested this PR thoroughly and coded it according to PEP8 and Plone's Python styleguide standards.