How to configure the legal parts of the NewMessage API

Created on 6 February 2023, almost 2 years ago
Updated 21 February 2023, almost 2 years ago

Problem/Motivation

As part of sending a NewMessage to VG Wort we need to send the correct set of booleans concerning the legal rights.

Here are the rights we have to decide how to set:

Proposed resolution

The flags can be set on the site level. In order to successful sending a newMessage to VG Wort, Distribution right, Right of public access, Reproduction Rights and Declaration of Granting of Rights have to be set to TRUE anyway,

Added hook_vgwort_enable_for_entity(\Drupal\Core\Entity\EntityInterface $entity): bool that allows custom modules to opt specific entities out of VG Wort. The entity will not get a counter ID and can not be queue or pushed to VG Wort via the newMessage API.

Remaining tasks

User interface changes

API changes

Data model changes

📌 Task
Status

Fixed

Version

2.0

Component

Code

Created by

🇬🇧United Kingdom alexpott 🇪🇺🌍

Live updates comments and jobs are added and updated live.
Sign in to follow issues

Comments & Activities

  • Issue created by @alexpott
  • @alexpott opened merge request.
  • Status changed to Needs review almost 2 years ago
  • 🇬🇧United Kingdom alexpott 🇪🇺🌍
  • 🇬🇧United Kingdom alexpott 🇪🇺🌍
  • 🇬🇧United Kingdom alexpott 🇪🇺🌍

    Discussed with @daniel.bosen. We agreed to merge now and hopefully @daniel.bosen can review this later and we'll make any changes in follow-ups.

  • Status changed to Fixed almost 2 years ago
    • alexpott committed 5e841757 on 2.0.x
      Issue #3339609 by alexpott: How to configure the legal parts of the...
  • 🇩🇪Germany IT-Cru Munich

    I think this part is not adapted in the right way in post update hook:

    The flags can be set on the site level. In order to successful sending a newMessage to VG Wort, Distribution right, Right of public access, Reproduction Rights and Declaration of Granting of Rights have to be set to TRUE anyway.

    In current code of post update hook all legal options are set to FALSE instead of already defined ones which should be set to TRUE anyway.

  • 🇬🇧United Kingdom alexpott 🇪🇺🌍

    @IT-Cru well at the moment we're unsure of what these flags should be set to. Given that these flags have legal ramifications I don't think we can set them to true by default. This might change as we get more understanding.

  • 🇩🇪Germany IT-Cru Munich

    Sounds good and fine for me.

Production build 0.71.5 2024