Index and Metadata optimizations for AR Listing View #70

nihadness · 2018-04-19T14:18:44Z

Current behavior before PR

Indexers of 'BikaCatalogAnalysisRequestListing' are defined inside 'AnalysisRequest' content class.
While listing AR's, in order to get columns regarding Doctor and Patient such as 'DoctorTitle', 'PatientTitle', 'PatientID' and etc., doctor and patient brains are called for each AR which slows down the total rendering time.

Move indexer methods under catalog/indexers
Instead of getting values of related fields from the brains, add metadata to the BikaCatalogAnalysisRequestListing and use them directly.
Also addes more fields from Health Add-on to the new SearchableText field which was introduced in Slow Searches in Listing Views senaite.core#771

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

nihadness added 7 commits April 16, 2018 13:45

Override NG3 Indexer of AR Listing Catalog.

af7981e

Move all the AR indexers to catalog/indexers.

d113ab9

Do not save null values.

e50b411

Do not call Patient and Doctor brains in AR listing View

c784552

New metadata columns in Upgrade Step.

527c69b

Define indexer methods.

e6aedb2

Do not render empty Doctor URL's in AR Listing View.

6cd6f5a

nihadness added addition P0: Critical labels Apr 19, 2018

nihadness self-assigned this Apr 19, 2018

nihadness requested a review from xispa April 19, 2018 14:18

nihadness added 4 commits April 20, 2018 10:54

Polish Indexers.

a54f5fb

Rename plone util import

72e8e47

Default values in indexers.

0e49284

Changes.rst

91c30ed

xispa approved these changes Apr 20, 2018

View reviewed changes

xispa merged commit e11526a into senaite:master Apr 20, 2018