diff --git a/rero_ils/modules/items/serializers/collector.py b/rero_ils/modules/items/serializers/collector.py index 09174fbb66..457afb56a7 100644 --- a/rero_ils/modules/items/serializers/collector.py +++ b/rero_ils/modules/items/serializers/collector.py @@ -144,21 +144,23 @@ def _build_doc(data): ) # process provision activity provision_activity = next( - filter(lambda x: x.get('type') - == 'bf:Publication', data.get('provisionActivity')) - ) - start_date = provision_activity.get('startDate', '') - end_date = provision_activity.get('endDate') - document_data['document_publication_year'] = \ - f'{start_date} - {end_date}' \ - if end_date else start_date - - document_data['document_publisher'] = cls.separator.join( - data['value'] - for stmt in provision_activity.get('statement', []) - for data in stmt.get('label', []) - if stmt['type'] == 'bf:Agent' + filter(lambda x: x.get('type') == 'bf:Publication', + data.get('provisionActivity')), + None ) + if provision_activity: + start_date = provision_activity.get('startDate', '') + end_date = provision_activity.get('endDate') + document_data['document_publication_year'] = \ + f'{start_date} - {end_date}' \ + if end_date else start_date + + document_data['document_publisher'] = cls.separator.join( + data['value'] + for stmt in provision_activity.get('statement', []) + for data in stmt.get('label', []) + if stmt['type'] == 'bf:Agent' + ) return document_data doc_search = DocumentsSearch() \