[Port to dtq-dev] Issue 1343: add PUT and DELETE endpoint methods to ClarinLicenseLabel REST repository

[kosarko]

Port of ufal#1357 by @kuchtiak-ufal to dtq-dev.

Summary by CodeRabbit

• New Features
  • Enhanced REST handling for CLARIN license labels: label/title validation, duplicate detection on create/update, and safer deletes that block removing labels used by existing licenses.
• Bug Fixes
  • Improved error responses for invalid payloads and missing labels; consistent “not found” behavior for label resources.
• Database
  • Updated H2/PostgreSQL constraints to enforce label uniqueness and correct foreign key relationships.
• Tests
  • Expanded integration test coverage for create, update, and delete scenarios.

[coderabbitai]

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info

⚙️ Run configuration

Configuration used: Organization UI

Review profile: CHILL

Plan: Pro

Run ID: 699e3457-ad54-4946-a105-4868c2ef43b2

📥 Commits

Reviewing files that changed from the base of the PR and between 91f94ce and ff14f78.

📒 Files selected for processing (2)

• dspace-api/src/test/java/org/dspace/builder/util/AbstractBuilderCleanupUtil.java
• dspace-server-webapp/src/main/java/org/dspace/app/rest/repository/ClarinLicenseLabelRestRepository.java

💤 Files with no reviewable changes (1)

• dspace-server-webapp/src/main/java/org/dspace/app/rest/repository/ClarinLicenseLabelRestRepository.java

🚧 Files skipped from review as they are similar to previous changes (1)

• dspace-api/src/test/java/org/dspace/builder/util/AbstractBuilderCleanupUtil.java

---

📝 Walkthrough

Walkthrough

The PR adds label-based lookup capability for ClarinLicense and ClarinLicenseLabel entities across the data access, REST, and test layers. It introduces findByLabel methods in DAOs and services, enforces label uniqueness via database migrations, adds request validation and duplicate/in-use checks in REST endpoints, and provides comprehensive integration test coverage.

Changes

ClarinLicense findByLabel and REST Validation

Layer / File(s)	Summary
Data access layer: findByLabel queries and service delegation dspace-api/src/main/java/org/dspace/content/dao/clarin/ClarinLicense*DAO.java, dspace-api/src/main/java/org/dspace/content/dao/impl/clarin/ClarinLicense*DAOImpl.java, dspace-api/src/main/java/org/dspace/content/service/clarin/ClarinLicense*Service.java, dspace-api/src/main/java/org/dspace/content/clarin/ClarinLicense*ServiceImpl.java	DAO interfaces and JPA criteria implementations for finding licenses/labels by label string; corresponding service interfaces and implementations delegate to DAO methods.
Database schema constraints for label uniqueness dspace-api/src/main/resources/org/dspace/storage/rdbms/sqlmigration/h2/V7.6_2026.06.01__license_label_constraint.sql, dspace-api/src/main/resources/org/dspace/storage/rdbms/sqlmigration/postgres/V7.6_2026.06.01__license_label_constraint.sql	SQL migrations enforce foreign key constraints on license_label_extended_mapping.label_id and add unique constraints on license_label.label for both H2 and PostgreSQL dialects.
REST API: License label validation and deletion checks dspace-server-webapp/src/main/java/org/dspace/app/rest/exception/ClarinLicenseLabelNotFoundException.java, dspace-server-webapp/src/main/java/org/dspace/app/rest/repository/ClarinLicenseLabelRestRepository.java	New exception for missing labels; repository uses ObjectMapper for request parsing, centralizes label validation (length, null/blank, uniqueness), rejects duplicate updates, and prevents deletion of labels in use by existing licenses via findByLabel.
REST integration tests: License label endpoint coverage dspace-server-webapp/src/test/java/org/dspace/app/rest/ClarinLicenseLabelRestRepositoryIT.java	Comprehensive integration tests for create (validation for long/duplicate labels), update (success, not found, invalid body, missing title, unauthorized, duplicate handling), and delete (not authorized, success, not found, in-use rejection) scenarios.
Test fixtures: Helper method signature updates dspace-server-webapp/src/test/java/org/dspace/app/rest/ClarinWorkspaceItemRestRepositoryIT.java, dspace-server-webapp/src/test/java/org/dspace/app/rest/ProvenanceServiceIT.java, dspace-api/src/test/java/org/dspace/builder/util/AbstractBuilderCleanupUtil.java	Test helper methods createClarinLicense now accept an explicit label parameter to generate test fixtures with distinct labels; ProvenanceServiceIT deleteClarinLicense reordered to delete the license before its labels; builder cleanup utility registers ClarinLicenseBuilder and ClarinLicenseLabelBuilder.

Estimated code review effort

🎯 3 (Moderate) | ⏱️ ~25 minutes

Suggested labels

REVIEW-in-progress, track_to_dtq-dev

Suggested reviewers

• milanmajchrak
• vidiecan

🚥 Pre-merge checks | ✅ 3 | ❌ 2

❌ Failed checks (2 warnings)

Check name	Status	Explanation	Resolution
Description check	⚠️ Warning	The description is incomplete; it only states this is a port of another PR without providing required details like problem description, analysis, or testing information mandated by the repository template.	Add comprehensive description following the template: explain the problem being solved, provide analysis of the changes, document manual testing steps, and confirm Copilot review.
Docstring Coverage	⚠️ Warning	Docstring coverage is 25.45% which is insufficient. The required threshold is 80.00%.	Write docstrings for the functions missing them to satisfy the coverage threshold.

✅ Passed checks (3 passed)

Check name	Status	Explanation
Title check	✅ Passed	The title clearly and specifically summarizes the main change: adding PUT and DELETE endpoint methods to ClarinLicenseLabel REST repository to address Issue 1343.
Linked Issues check	✅ Passed	Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check	✅ Passed	Check skipped because no linked issues were found for this pull request.

_{✏️ Tip: You can configure your own custom pre-merge checks in the settings.}

---

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

• X
• Mastodon
• Reddit
• LinkedIn

_{Comment @coderabbitai help to get the list of available commands and usage tips.}

[coderabbitai]

Actionable comments posted: 5

🧹 Nitpick comments (6)

dspace-api/src/main/java/org/dspace/content/dao/impl/clarin/ClarinLicenseLabelDAOImpl.java (1)

34-35: ⚡ Quick win

Add method Javadoc on the new public override.

For overridden public methods, a brief /** {@inheritdoc} */ is sufficient to satisfy documentation requirements.

As per coding guidelines, "Provide Javadoc comments for all public classes and methods".

🤖 Prompt for AI Agents

Verify each finding against current code. Fix only still-valid issues, skip the
rest with a brief reason, keep changes minimal, and validate.

In
`@dspace-api/src/main/java/org/dspace/content/dao/impl/clarin/ClarinLicenseLabelDAOImpl.java`
around lines 34 - 35, Add a Javadoc comment to the public overridden method
findByLabel in class ClarinLicenseLabelDAOImpl: place a brief /** {`@inheritDoc`}
*/ directly above the method signature of public ClarinLicenseLabel
findByLabel(Context context, String label) throws SQLException to satisfy the
coding guideline requiring Javadoc on public methods.

Source: Coding guidelines

dspace-api/src/main/java/org/dspace/content/dao/clarin/ClarinLicenseLabelDAO.java (1)

26-26: ⚡ Quick win

Document the new public DAO contract with Javadoc.

Please add method-level Javadoc for findByLabel(...) to match API documentation requirements.

As per coding guidelines, "Provide Javadoc comments for all public classes and methods".

🤖 Prompt for AI Agents

Verify each finding against current code. Fix only still-valid issues, skip the
rest with a brief reason, keep changes minimal, and validate.

In
`@dspace-api/src/main/java/org/dspace/content/dao/clarin/ClarinLicenseLabelDAO.java`
at line 26, Add Javadoc for the public DAO method findByLabel in
ClarinLicenseLabelDAO: describe the purpose (find a ClarinLicenseLabel by its
label), document parameters (Context context, String label), document return
value (ClarinLicenseLabel or null if not found), and list thrown exceptions
(throws SQLException). Place the Javadoc immediately above the method
declaration in the ClarinLicenseLabelDAO interface and use standard tags
(`@param`, `@return`, `@throws`) to satisfy the API documentation requirement.

Source: Coding guidelines

dspace-api/src/main/java/org/dspace/content/clarin/ClarinLicenseLabelServiceImpl.java (1)

80-83: ⚡ Quick win

Add Javadoc for the new public service method.

Please add method documentation (or {@inheritdoc}) for findByLabel(...).

As per coding guidelines, "Provide Javadoc comments for all public classes and methods".

🤖 Prompt for AI Agents

Verify each finding against current code. Fix only still-valid issues, skip the
rest with a brief reason, keep changes minimal, and validate.

In
`@dspace-api/src/main/java/org/dspace/content/clarin/ClarinLicenseLabelServiceImpl.java`
around lines 80 - 83, Add Javadoc to the public service method findByLabel in
ClarinLicenseLabelServiceImpl: update the method declaration for public
ClarinLicenseLabel findByLabel(Context context, String label) to include a
Javadoc block (or {`@inheritDoc`}) describing the purpose, parameters (context,
label), return value, and SQLException thrown; reference the DAO call
clarinLicenseLabelDAO.findByLabel in the comment if helpful to explain behavior.

Source: Coding guidelines

dspace-api/src/main/java/org/dspace/content/dao/clarin/ClarinLicenseDAO.java (1)

31-31: ⚡ Quick win

Add Javadoc to the new public DAO method.

findByLabel(...) is public API surface and should be documented for contract clarity.

Suggested update

+    /**
+     * Find CLARIN licenses by associated label.
+     *
+     * `@param` context current DSpace context
+     * `@param` label license label
+     * `@return` matching licenses
+     * `@throws` SQLException if database access fails
+     */
     List<ClarinLicense> findByLabel(Context context, String label) throws SQLException;

As per coding guidelines, "Provide Javadoc comments for all public classes and methods".

🤖 Prompt for AI Agents

Verify each finding against current code. Fix only still-valid issues, skip the
rest with a brief reason, keep changes minimal, and validate.

In `@dspace-api/src/main/java/org/dspace/content/dao/clarin/ClarinLicenseDAO.java`
at line 31, Add Javadoc for the new public DAO method findByLabel in the
ClarinLicenseDAO interface: document the method's purpose (lookup licenses by
label), describe each parameter (Context context, String label), state the
return value (List<ClarinLicense> matching the label, possibly empty), and
document the SQLException thrown; ensure the Javadoc appears above the method
signature in ClarinLicenseDAO and follows project tag conventions (`@param`,
`@return`, `@throws`).

Source: Coding guidelines

dspace-api/src/main/java/org/dspace/content/clarin/ClarinLicenseServiceImpl.java (1)

101-104: ⚡ Quick win

Document findByLabel(...) with Javadoc.

This new public method should include method-level Javadoc (or {@inheritdoc}) to keep public API docs complete.

As per coding guidelines, "Provide Javadoc comments for all public classes and methods".

🤖 Prompt for AI Agents

Verify each finding against current code. Fix only still-valid issues, skip the
rest with a brief reason, keep changes minimal, and validate.

In
`@dspace-api/src/main/java/org/dspace/content/clarin/ClarinLicenseServiceImpl.java`
around lines 101 - 104, Add method-level Javadoc to the public method
findByLabel in ClarinLicenseServiceImpl: either add a descriptive Javadoc block
explaining purpose, parameters (Context context, String label), return value
(List<ClarinLicense>), and thrown SQLException, or use {`@inheritDoc`} if the
interface already documents it; ensure the Javadoc appears immediately above the
findByLabel method and references the parameters and exception so the public API
documentation remains complete.

Source: Coding guidelines

dspace-server-webapp/src/main/java/org/dspace/app/rest/repository/ClarinLicenseLabelRestRepository.java (1)

170-180: 💤 Low value

Add Javadoc for the private helper method.

While less critical than public methods, documenting the validation logic in checkLabelAndTitle would improve maintainability.

📝 Suggested Javadoc

+    /**
+     * Validates that the label and title are not blank and that the label length
+     * does not exceed MAX_LABEL_LENGTH.
+     *
+     * `@param` clarinLicenseLabelRest the REST object to validate
+     * `@throws` DSpaceBadRequestException if validation fails
+     */
     private void checkLabelAndTitle(ClarinLicenseLabelRest clarinLicenseLabelRest) {

🤖 Prompt for AI Agents

Verify each finding against current code. Fix only still-valid issues, skip the
rest with a brief reason, keep changes minimal, and validate.

In
`@dspace-server-webapp/src/main/java/org/dspace/app/rest/repository/ClarinLicenseLabelRestRepository.java`
around lines 170 - 180, Add Javadoc to the private helper checkLabelAndTitle in
ClarinLicenseLabelRestRepository describing its purpose (validate non-null/empty
trimmed label and title), the behavior (throws DSpaceBadRequestException on
missing/blank fields or when label exceeds MAX_LABEL_LENGTH), and note the
trimming of label via Optional and the use of MAX_LABEL_LENGTH; keep the comment
concise and place it immediately above the checkLabelAndTitle method.

🤖 Prompt for all review comments with AI agents

Verify each finding against current code. Fix only still-valid issues, skip the
rest with a brief reason, keep changes minimal, and validate.

Inline comments:
In
`@dspace-api/src/main/java/org/dspace/content/dao/impl/clarin/ClarinLicenseDAOImpl.java`:
- Around line 62-76: The public method findByLabel in ClarinLicenseDAOImpl is
missing Javadoc; add a standard Javadoc block above the method explaining the
purpose of findByLabel(Context context, String label), describing the parameters
(context and label), the return value (List<ClarinLicense>), and the thrown
SQLException, and include any behavioural notes (e.g., matching semantics or
null/empty handling) so the documentation meets project guidelines and documents
the contract of the overridden DAO method.

In
`@dspace-api/src/main/resources/org/dspace/storage/rdbms/sqlmigration/h2/V7.6_2026.06.01__license_label_constraint.sql`:
- Around line 17-18: The migration unconditionally drops/adds the UNIQUE
constraint on license_label.label which will fail if duplicate labels exist;
before adding license_label_label_unique run a preflight duplicate check (SELECT
label, COUNT(*) ... HAVING COUNT(*) > 1) and implement a deterministic
dedup/upsert step to remove or merge duplicates (e.g. keep the earliest id or
newest timestamp) for rows with the same label, then re-run the ALTER TABLE to
add the UNIQUE constraint; reference the table name license_label and the
constraint name license_label_label_unique when adding the preflight check and
deterministic cleanup migration.

In
`@dspace-api/src/test/java/org/dspace/builder/util/AbstractBuilderCleanupUtil.java`:
- Line 20: The cleanup map in AbstractBuilderCleanupUtil now includes
ClarinLicenseBuilder but is missing ClarinLicenseLabelBuilder, causing FK
deletion order issues; update the cleanup map in the AbstractBuilderCleanupUtil
class to add ClarinLicenseLabelBuilder immediately after the existing
ClarinLicenseBuilder entry so that ClarinLicenseBuilder (licenses) are removed
before ClarinLicenseLabelBuilder (labels), ensuring deletions obey the
foreign-key constraint referenced by ClarinLicenseLabelRestRepository; locate
the map population code (the entries involving ClarinLicenseBuilder) and insert
the ClarinLicenseLabelBuilder entry right after it.

In
`@dspace-server-webapp/src/main/java/org/dspace/app/rest/repository/ClarinLicenseLabelRestRepository.java`:
- Around line 112-148: Add a Javadoc comment to the public method put in
ClarinLicenseLabelRestRepository: document the purpose of the endpoint, describe
each parameter (Context context, HttpServletRequest request, String apiCategory,
String model, Integer id, JsonNode jsonNode), explain the return value
(ClarinLicenseLabelRest via converter.toRest), and list thrown exceptions
(SQLException, AuthorizeException, DSpaceBadRequestException,
ClarinLicenseLabelNotFoundException). Keep the description concise and mention
relevant behavior such as parsing with objectMapper.treeToValue, validation via
checkLabelAndTitle, uniqueness check via clarinLicenseLabelService.findByLabel,
and that the entity is updated with updateClarinLicenseLabel and
clarinLicenseLabelService.update.
- Around line 150-168: Add Javadoc to the public method delete(Context context,
Integer id) in ClarinLicenseLabelRestRepository: document the purpose of the
method (deletes a ClarinLicenseLabel by id), describe parameters `@param` context
and `@param` id, and list thrown exceptions with `@throws` for AuthorizeException,
ClarinLicenseLabelNotFoundException, DSpaceBadRequestException and any
runtime/SQL-related exceptions that can propagate (e.g., RuntimeException
wrapping SQLException); also include a brief note about the
`@PreAuthorize`("hasAuthority('ADMIN')") security requirement. Ensure the Javadoc
sits immediately above the delete method signature.

---

Nitpick comments:
In
`@dspace-api/src/main/java/org/dspace/content/clarin/ClarinLicenseLabelServiceImpl.java`:
- Around line 80-83: Add Javadoc to the public service method findByLabel in
ClarinLicenseLabelServiceImpl: update the method declaration for public
ClarinLicenseLabel findByLabel(Context context, String label) to include a
Javadoc block (or {`@inheritDoc`}) describing the purpose, parameters (context,
label), return value, and SQLException thrown; reference the DAO call
clarinLicenseLabelDAO.findByLabel in the comment if helpful to explain behavior.

In
`@dspace-api/src/main/java/org/dspace/content/clarin/ClarinLicenseServiceImpl.java`:
- Around line 101-104: Add method-level Javadoc to the public method findByLabel
in ClarinLicenseServiceImpl: either add a descriptive Javadoc block explaining
purpose, parameters (Context context, String label), return value
(List<ClarinLicense>), and thrown SQLException, or use {`@inheritDoc`} if the
interface already documents it; ensure the Javadoc appears immediately above the
findByLabel method and references the parameters and exception so the public API
documentation remains complete.

In
`@dspace-api/src/main/java/org/dspace/content/dao/clarin/ClarinLicenseDAO.java`:
- Line 31: Add Javadoc for the new public DAO method findByLabel in the
ClarinLicenseDAO interface: document the method's purpose (lookup licenses by
label), describe each parameter (Context context, String label), state the
return value (List<ClarinLicense> matching the label, possibly empty), and
document the SQLException thrown; ensure the Javadoc appears above the method
signature in ClarinLicenseDAO and follows project tag conventions (`@param`,
`@return`, `@throws`).

In
`@dspace-api/src/main/java/org/dspace/content/dao/clarin/ClarinLicenseLabelDAO.java`:
- Line 26: Add Javadoc for the public DAO method findByLabel in
ClarinLicenseLabelDAO: describe the purpose (find a ClarinLicenseLabel by its
label), document parameters (Context context, String label), document return
value (ClarinLicenseLabel or null if not found), and list thrown exceptions
(throws SQLException). Place the Javadoc immediately above the method
declaration in the ClarinLicenseLabelDAO interface and use standard tags
(`@param`, `@return`, `@throws`) to satisfy the API documentation requirement.

In
`@dspace-api/src/main/java/org/dspace/content/dao/impl/clarin/ClarinLicenseLabelDAOImpl.java`:
- Around line 34-35: Add a Javadoc comment to the public overridden method
findByLabel in class ClarinLicenseLabelDAOImpl: place a brief /** {`@inheritDoc`}
*/ directly above the method signature of public ClarinLicenseLabel
findByLabel(Context context, String label) throws SQLException to satisfy the
coding guideline requiring Javadoc on public methods.

In
`@dspace-server-webapp/src/main/java/org/dspace/app/rest/repository/ClarinLicenseLabelRestRepository.java`:
- Around line 170-180: Add Javadoc to the private helper checkLabelAndTitle in
ClarinLicenseLabelRestRepository describing its purpose (validate non-null/empty
trimmed label and title), the behavior (throws DSpaceBadRequestException on
missing/blank fields or when label exceeds MAX_LABEL_LENGTH), and note the
trimming of label via Optional and the use of MAX_LABEL_LENGTH; keep the comment
concise and place it immediately above the checkLabelAndTitle method.

🪄 Autofix (Beta)

Fix all unresolved CodeRabbit comments on this PR:

• Push a commit to this branch (recommended)
• Create a new PR with the fixes

---

ℹ️ Review info

⚙️ Run configuration

Configuration used: Organization UI

Review profile: CHILL

Plan: Pro

Run ID: 22405db3-31ea-40ca-aa5e-3395ae46e0db

📥 Commits

Reviewing files that changed from the base of the PR and between 82085cb and 91f94ce.

📒 Files selected for processing (16)

• dspace-api/src/main/java/org/dspace/content/clarin/ClarinLicenseLabelServiceImpl.java
• dspace-api/src/main/java/org/dspace/content/clarin/ClarinLicenseServiceImpl.java
• dspace-api/src/main/java/org/dspace/content/dao/clarin/ClarinLicenseDAO.java
• dspace-api/src/main/java/org/dspace/content/dao/clarin/ClarinLicenseLabelDAO.java
• dspace-api/src/main/java/org/dspace/content/dao/impl/clarin/ClarinLicenseDAOImpl.java
• dspace-api/src/main/java/org/dspace/content/dao/impl/clarin/ClarinLicenseLabelDAOImpl.java
• dspace-api/src/main/java/org/dspace/content/service/clarin/ClarinLicenseLabelService.java
• dspace-api/src/main/java/org/dspace/content/service/clarin/ClarinLicenseService.java
• dspace-api/src/main/resources/org/dspace/storage/rdbms/sqlmigration/h2/V7.6_2026.06.01__license_label_constraint.sql
• dspace-api/src/main/resources/org/dspace/storage/rdbms/sqlmigration/postgres/V7.6_2026.06.01__license_label_constraint.sql
• dspace-api/src/test/java/org/dspace/builder/util/AbstractBuilderCleanupUtil.java
• dspace-server-webapp/src/main/java/org/dspace/app/rest/exception/ClarinLicenseLabelNotFoundException.java
• dspace-server-webapp/src/main/java/org/dspace/app/rest/repository/ClarinLicenseLabelRestRepository.java
• dspace-server-webapp/src/test/java/org/dspace/app/rest/ClarinLicenseLabelRestRepositoryIT.java
• dspace-server-webapp/src/test/java/org/dspace/app/rest/ClarinWorkspaceItemRestRepositoryIT.java
• dspace-server-webapp/src/test/java/org/dspace/app/rest/ProvenanceServiceIT.java

[Copilot]

Pull request overview

This PR ports Issue 1343 to dtq-dev by expanding the CLARIN License Label REST support (adding PUT/DELETE), adding label-based lookup in the CLARIN License service/DAO layer, and enforcing stricter validation/constraints around license labels.

Changes:

• Add/update REST API behavior for ClarinLicenseLabel (PUT + DELETE), including label/title validation, max label length enforcement, uniqueness checks, and “in-use” deletion protection.
• Add DAO/service methods to find licenses by a given label (to support deletion checks and label-based retrieval).
• Add DB migration to enforce FK restrict semantics and a UNIQUE constraint on license_label.label, plus expanded integration tests.

Reviewed changes

Copilot reviewed 16 out of 16 changed files in this pull request and generated 4 comments.

Show a summary per file

File	Description
dspace-server-webapp/src/test/java/org/dspace/app/rest/ProvenanceServiceIT.java	Updates test helpers to create licenses with explicit labels and adjusts cleanup order for FK constraints.
dspace-server-webapp/src/test/java/org/dspace/app/rest/ClarinWorkspaceItemRestRepositoryIT.java	Updates workspace item REST integration tests to create licenses with explicit label values.
dspace-server-webapp/src/test/java/org/dspace/app/rest/ClarinLicenseLabelRestRepositoryIT.java	Adds integration coverage for create/update/delete scenarios, including validation and “label in use” deletion behavior.
dspace-server-webapp/src/main/java/org/dspace/app/rest/repository/ClarinLicenseLabelRestRepository.java	Implements PUT/DELETE endpoints and centralizes validation + update logic for license labels.
dspace-server-webapp/src/main/java/org/dspace/app/rest/exception/ClarinLicenseLabelNotFoundException.java	Adds a dedicated 404 exception for missing license labels.
dspace-api/src/test/java/org/dspace/builder/util/AbstractBuilderCleanupUtil.java	Registers ClarinLicenseBuilder for automated builder cleanup in tests.
dspace-api/src/main/resources/org/dspace/storage/rdbms/sqlmigration/postgres/V7.6_2026.06.01__license_label_constraint.sql	Adds/replaces FK + UNIQUE constraints for Postgres to enforce restrict semantics and label uniqueness.
dspace-api/src/main/resources/org/dspace/storage/rdbms/sqlmigration/h2/V7.6_2026.06.01__license_label_constraint.sql	Adds/replaces FK + UNIQUE constraints for H2 to enforce restrict semantics and label uniqueness.
dspace-api/src/main/java/org/dspace/content/service/clarin/ClarinLicenseService.java	Adds findByLabel() to the CLARIN license service API.
dspace-api/src/main/java/org/dspace/content/service/clarin/ClarinLicenseLabelService.java	Adds findByLabel() to the CLARIN license label service API.
dspace-api/src/main/java/org/dspace/content/dao/impl/clarin/ClarinLicenseLabelDAOImpl.java	Implements findByLabel() query for license labels via Criteria API.
dspace-api/src/main/java/org/dspace/content/dao/impl/clarin/ClarinLicenseDAOImpl.java	Implements findByLabel() query for licenses via join to license labels.
dspace-api/src/main/java/org/dspace/content/dao/clarin/ClarinLicenseLabelDAO.java	Adds findByLabel() to the DAO interface.
dspace-api/src/main/java/org/dspace/content/dao/clarin/ClarinLicenseDAO.java	Adds findByLabel() to the DAO interface.
dspace-api/src/main/java/org/dspace/content/clarin/ClarinLicenseServiceImpl.java	Wires the new findByLabel() through to the DAO.
dspace-api/src/main/java/org/dspace/content/clarin/ClarinLicenseLabelServiceImpl.java	Wires the new findByLabel() through to the DAO.

[milanmajchrak]

Please check the copilot comments