Number of batch samples

[Lunga001]

Description of the issue/feature this PR addresses

On th batch listing(batches) add a column that shows the number of samples

Current behavior before PR

On the Batch Listing(Batches) there isn't a column that shows the number of samples that are on the batch

Desired behavior after PR is merged

To have a column that shows the number of samples that are on a batch.

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

[ramonski]

Hi @Lunga001, this PR is still marked as "Draft".
Was this by accident or what is the current status of this PR?

[Lunga001]

Hi @ramonski I missed this one, I wasn't sure if getNumberOfSamples need to be added as an index instead waking up the object even though it seemed we are already got the object from the previous lines.

[ramonski]

Hi @Lunga001,
since you are just counting the samples, you don't need to retrieve the full objects from getAnalysisRequests. Instead, you could just use getAnalysisRequestsBrains:
https://github.com/senaite/senaite.core/blob/2.x/src/bika/lims/content/batch.py#L237-L242

Regarding the sorting, it is a bit tricky because any index would need to get updated as soon as samples are assigned/unassigned from a batch. Therefore, I would rely on the manual sorting.

Maybe a good idea would be to move the count method directly to the Batch content class.