invenio-app-rdm 13.0.0b1.dev26__tar.gz → 13.0.0b1.dev28__tar.gz

{invenio_app_rdm-13.0.0b1.dev26 → invenio_app_rdm-13.0.0b1.dev28}/CHANGES.rst

@@ -10,6 +10,13 @@
 Changes
 =======
 
+Version v13.0.0b1.dev27 (released 2025-01-16)
+
+- Revert "config: add record and membership comment notifications"
+    * This is actually a breaking change since it introduces a new
+      set of notification templates that will potentialy not be
+      styled if overridden in an instance's overlay.
+
 Version v13.0.0b1.dev26 (released 2025-01-16)
 
 - config: add record and membership comment notifications

{invenio_app_rdm-13.0.0b1.dev26 → invenio_app_rdm-13.0.0b1.dev28}/PKG-INFO

@@ -1,6 +1,6 @@
 Metadata-Version: 2.2
 Name: invenio-app-rdm
-Version: 13.0.0b1.dev26
+Version: 13.0.0b1.dev28
 Summary: Invenio Research Data Management.
 Home-page: https://github.com/inveniosoftware/invenio-app-rdm
 Author: CERN
@@ -97,6 +97,13 @@ https://inveniordm.docs.cern.ch
 Changes
 =======
 
+Version v13.0.0b1.dev27 (released 2025-01-16)
+
+- Revert "config: add record and membership comment notifications"
+    * This is actually a breaking change since it introduces a new
+      set of notification templates that will potentialy not be
+      styled if overridden in an instance's overlay.
+
 Version v13.0.0b1.dev26 (released 2025-01-16)
 
 - config: add record and membership comment notifications

{invenio_app_rdm-13.0.0b1.dev26 → invenio_app_rdm-13.0.0b1.dev28}/invenio_app_rdm/__init__.py

@@ -17,6 +17,6 @@
 #
 # See PEP 0440 for details - https://www.python.org/dev/peps/pep-0440
 
-__version__ = "13.0.0b1.dev26"
+__version__ = "13.0.0b1.dev28"
 
 __all__ = ("__version__",)

{invenio_app_rdm-13.0.0b1.dev26 → invenio_app_rdm-13.0.0b1.dev28}/invenio_app_rdm/config.py

@@ -50,7 +50,6 @@ from invenio_notifications.backends import EmailNotificationBackend
 from invenio_rdm_records.notifications.builders import (
     CommunityInclusionAcceptNotificationBuilder,
     CommunityInclusionCancelNotificationBuilder,
-    CommunityInclusionCommentNotificationBuilder,
     CommunityInclusionDeclineNotificationBuilder,
     CommunityInclusionExpireNotificationBuilder,
     CommunityInclusionSubmittedNotificationBuilder,
@@ -1369,9 +1368,6 @@ NOTIFICATIONS_BUILDERS = {
     GrantUserAccessNotificationBuilder.type: GrantUserAccessNotificationBuilder,
     # Comment request event
     CommentRequestEventCreateNotificationBuilder.type: CommentRequestEventCreateNotificationBuilder,
-    CommunityInclusionCommentNotificationBuilder.type: CommunityInclusionCommentNotificationBuilder,
-    community_notifications.CommunityInvitationCommentNotificationBuilder.type: community_notifications.CommunityInvitationCommentNotificationBuilder,
-    community_notifications.CommunityRequestCommentNotificationBuilder.type: community_notifications.CommunityRequestCommentNotificationBuilder,
     community_notifications.SubComReqCommentNotificationBuilder.type: community_notifications.SubComReqCommentNotificationBuilder,
     community_notifications.SubComInvCommentNotificationBuilder.type: community_notifications.SubComInvCommentNotificationBuilder,
     # Community inclusion

{invenio_app_rdm-13.0.0b1.dev26 → invenio_app_rdm-13.0.0b1.dev28}/invenio_app_rdm/records_ui/views/decorators.py

@@ -403,3 +403,24 @@ def secret_link_or_login_required():
         return view
 
     return decorator
+
+
+def no_cache_response(f):
+    """Add appropriate response headers to force no caching.
+
+    This decorator is used to prevent caching of the response in the browser. This is needed
+    in the deposit form as we initialize the form with the record metadata included in the html page
+    and we don't want the browser to cache this page so that the user always gets the latest version of the record.
+    """
+
+    @wraps(f)
+    def view(*args, **kwargs):
+        response = make_response(f(*args, **kwargs))
+
+        response.cache_control.no_cache = True
+        response.cache_control.no_store = True
+        response.cache_control.must_revalidate = True
+
+        return response
+
+    return view

{invenio_app_rdm-13.0.0b1.dev26 → invenio_app_rdm-13.0.0b1.dev28}/invenio_app_rdm/records_ui/views/deposits.py

@@ -23,6 +23,7 @@ from invenio_i18n.ext import current_i18n
 from invenio_rdm_records.proxies import current_rdm_records
 from invenio_rdm_records.records.api import get_files_quota
 from invenio_rdm_records.resources.serializers import UIJSONSerializer
+from invenio_rdm_records.services.components.pids import _get_optional_doi_transitions
 from invenio_rdm_records.services.schemas import RDMRecordSchema
 from invenio_rdm_records.services.schemas.utils import dump_empty
 from invenio_records_resources.services.errors import PermissionDeniedError
@@ -34,6 +35,7 @@ from sqlalchemy.orm import load_only
 
 from ..utils import set_default_value
 from .decorators import (
+    no_cache_response,
     pass_draft,
     pass_draft_community,
     pass_draft_files,
@@ -45,7 +47,7 @@ from .filters import get_scheme_label
 #
 # Helpers
 #
-def get_form_pids_config():
+def get_form_pids_config(record=None):
     """Prepare configuration for the pids field.
 
     Currently supporting only doi.
@@ -55,17 +57,45 @@ def get_form_pids_config():
     # FIXME: User provider.is_managed() requires tiny fix in config
     can_be_managed = True
     can_be_unmanaged = True
+    # We initialize the optional doi to empty to indicate that there is no restriction on the transitions
+    # This is valid for new uploads and when the DOI is required in an instance
+    optional_doi_transitions = []
     for scheme in service.config.pids_providers.keys():
         if not scheme == "doi":
             continue
+
         record_pid_config = current_app.config["RDM_PERSISTENT_IDENTIFIERS"]
         scheme_label = record_pid_config.get(scheme, {}).get("label", scheme)
         is_doi_required = record_pid_config.get(scheme, {}).get("required")
         default_selected = (
             record_pid_config.get(scheme, {}).get("ui", {}).get("default_selected")
         )
+        if record is not None and not is_doi_required:
+            sitename = current_app.config.get("THEME_SITENAME", "this repository")
+            previous_published_record = (
+                service.record_cls.get_latest_published_by_parent(record.parent)
+            )
+            optional_doi_transitions = _get_optional_doi_transitions(
+                previous_published_record
+            )
+            if optional_doi_transitions:
+                optional_doi_transitions["message"] = optional_doi_transitions.get(
+                    "message"
+                ).format(sitename=sitename)
+                if set(optional_doi_transitions.get("allowed_providers", [])) - set(
+                    ["external", "not_needed"]
+                ):
+                    # In case we have locally managed provider as an allowed one, we need to
+                    # select it by default. That is relevant for the case when the
+                    # user creates a new version of the record and the previous version
+                    # had a datacite DOI.
+                    default_selected = "no"
+
+        # if the DOI is required but the default selected is not_needed then we set it to yes
+        # to force the user to mint a DOI
         if is_doi_required and default_selected == "not_needed":
             default_selected = "yes"
+
         pids_provider = {
             "scheme": scheme,
             "field_label": "Digital Object Identifier",
@@ -89,6 +119,7 @@ def get_form_pids_config():
                 "unambiguously cited. Example: 10.1234/foo.bar"
             ).format(scheme_label=scheme_label),
             "default_selected": default_selected,
+            "optional_doi_transitions": optional_doi_transitions,
         }
         pids_providers.append(pids_provider)
 
@@ -332,6 +363,8 @@ def get_form_config(**kwargs):
     if record_quota:
         quota["maxStorage"] = record_quota["quota_size"]
 
+    record = kwargs.pop("record", None)
+
     return dict(
         vocabularies=VocabulariesOptions().dump(),
         autocomplete_names=conf.get(
@@ -339,7 +372,7 @@ def get_form_config(**kwargs):
         ),
         current_locale=str(current_i18n.locale),
         default_locale=conf.get("BABEL_DEFAULT_LOCALE", "en"),
-        pids=get_form_pids_config(),
+        pids=get_form_pids_config(record=record),
         quota=quota,
         decimal_size_display=conf.get("APP_RDM_DISPLAY_DECIMAL_FILE_SIZES", True),
         links=dict(
@@ -390,6 +423,7 @@ def new_record():
 # Views
 #
 @login_required
+@no_cache_response
 @pass_draft_community
 def deposit_create(community=None):
     """Create a new deposit."""
@@ -441,6 +475,7 @@ def deposit_create(community=None):
 @secret_link_or_login_required()
 @pass_draft(expand=True)
 @pass_draft_files
+@no_cache_response
 def deposit_edit(pid_value, draft=None, draft_files=None, files_locked=True):
     """Edit an existing deposit."""
     # don't show draft's deposit form if the user can't edit it
@@ -490,6 +525,7 @@ def deposit_edit(pid_value, draft=None, draft_files=None, files_locked=True):
         # hide react community component
         hide_community_selection=community_use_jinja_header,
         is_doi_required=is_doi_required,
+        record=draft._record,
     )
 
     if is_doi_required and not record.get("pids", {}).get("doi"):

{invenio_app_rdm-13.0.0b1.dev26 → invenio_app_rdm-13.0.0b1.dev28}/invenio_app_rdm/theme/assets/semantic-ui/js/invenio_app_rdm/deposit/RDMDepositForm.js

@@ -211,6 +211,7 @@ export class RDMDepositForm extends Component {
                             btnLabelGetPID={pid.btn_label_get_pid}
                             canBeManaged={pid.can_be_managed}
                             canBeUnmanaged={pid.can_be_unmanaged}
+                            optionalDOItransitions={pid.optional_doi_transitions}
                             fieldPath={`pids.${pid.scheme}`}
                             fieldLabel={pid.field_label}
                             isEditingPublishedRecord={

{invenio_app_rdm-13.0.0b1.dev26 → invenio_app_rdm-13.0.0b1.dev28}/invenio_app_rdm.egg-info/PKG-INFO

@@ -1,6 +1,6 @@
 Metadata-Version: 2.2
 Name: invenio-app-rdm
-Version: 13.0.0b1.dev26
+Version: 13.0.0b1.dev28
 Summary: Invenio Research Data Management.
 Home-page: https://github.com/inveniosoftware/invenio-app-rdm
 Author: CERN
@@ -97,6 +97,13 @@ https://inveniordm.docs.cern.ch
 Changes
 =======
 
+Version v13.0.0b1.dev27 (released 2025-01-16)
+
+- Revert "config: add record and membership comment notifications"
+    * This is actually a breaking change since it introduces a new
+      set of notification templates that will potentialy not be
+      styled if overridden in an instance's overlay.
+
 Version v13.0.0b1.dev26 (released 2025-01-16)
 
 - config: add record and membership comment notifications