serials: allow loading of serials on non-journal documents

This commit allows:
* loading of holdings of type `serial` on any type of `document`
* loading of holdings of type `standard` of document of type `journal`
* loading of items of type `standard` on holdings of type `serial`

* Closes rero#1612

Co-Authored-by: Aly Badr <aly.badr@rero.ch>

rero_ils/modules/holdings/api.py

@@ -131,8 +131,6 @@ def extended_validation(self, **kwargs):
         frequency the next_expected_date should be given.
 
         :return: False if
-            - document type is not journal and holding type is serial.
-            - document type is journal and holding type is not serial.
             - document type is ebook and holding type is not electronic.
             - document type is not ebook and holding type is electronic.
             - holding type is serial and the next_expected_date
@@ -157,10 +155,9 @@ def extended_validation(self, **kwargs):
         is_electronic = self.holdings_type == 'electronic'
         is_issuance = \
             document.get('issuance', {}).get('main_type') == 'rdami:1003'
-        if (is_serial and not is_issuance) \
-                or (document.harvested ^ is_electronic):
-            msg = _('Holding is not attached to the correct document type.'
-                    ' document: {pid}')
+        if (document.harvested ^ is_electronic):
+            msg = _('Electronic Holding is not attached to the correct \
+                    document type. document: {pid}')
             return _(msg.format(pid=document_pid))
         # the enumeration and chronology optional fields are only allowed for
         # serial holdings

rero_ils/modules/items/api/record.py

@@ -62,8 +62,6 @@ def extended_validation(self, **kwargs):
         if not holding:
             return _('Holding does not exist: {pid}.'.format(pid=holding_pid))
         is_serial = holding.holdings_type == 'serial'
-        if is_serial and self.get('type') == 'standard':
-            return _('Standard item can not attached to a journal.')
         issue = self.get('issue', {})
         if issue and self.get('type') == 'standard':
             return _('Standard item can not have a issue field.')