Problem/Motivation

When a CM Document is saved, validation should prevent save if there is already a CM document for that entity or entity:field combination.

The system should never have two documentations for the same thing.

blocked by https://www.drupal.org/project/content_model_documentation/issues/3334959

Steps to reproduce

Proposed resolution

The approach should be similar to what is done in the allow_only_one module.

However that module can not be used directly because cm_document entity type is not supported by that module and there os no good reason to extend that module to use it.

Remaining tasks

User interface changes

API changes

Data model changes

Issue fork content_model_documentation-3339247

Command icon Show commands

Start within a Git clone of the project using the version control instructions.

Or, if you do not have SSH keys set up on git.drupalcode.org:

About issue forks

Comments

swirt created an issue. See original summary.

swirt’s picture

swirt
he/him
Primary language English
commented
Title: Add validation to make ensure only one » Add validation to make ensure only one doc per entity
swirt’s picture

swirt
he/him
Primary language English
commented
Issue summary: View changes
swirt’s picture

swirt
he/him
Primary language English
commented

This is less critical now that https://www.drupal.org/project/content_model_documentation/issues/3346158 has been resolved.

Validation is still important, but not as critical.

Additional note: Validation will have to exempt applying to site.note, site.process and site.principle as these can have multiples of the same type creates as they are not entity specific.

swirt’s picture

swirt
he/him
Primary language English
commented
Title: Add validation to make ensure only one doc per entity » Add validation to enforce only one doc per entity being documented
swirt’s picture

swirt
he/him
Primary language English
commented
Priority: Major » Normal

JayDarnell made their first commit to this issue’s fork.

JayDarnell opened merge request !4

jaydarnell’s picture

jaydarnell
he/him
Primary language English
Location Guelph, Ontario
commented

I believe I have this one sorted @swirt

  • swirt committed cd43bc13 on 1.0.x authored by JayDarnell 
    Issue #3339247: add validation to enforce one doc per entity type
swirt’s picture

swirt
he/him
Primary language English
commented
Status: Active » Fixed

Thank you for this contribution @JayDarnell.

This will go out with 1.0.5

swirt’s picture

swirt
he/him
Primary language English
commented
Status: Fixed » Closed (fixed)