Problem/Motivation

content_translation_status allows/allowed NULL values. This *should* have been fixed by content_translation_update_8400() already, but somehow that didn't quite work for paragraphs_library.

The result is that the update function fails to make the field NOT NULL because there are NULL values.

Proposed resolution

Explicitly set NULL values to 1.

Remaining tasks

User interface changes

API changes

Data model changes

CommentFileSizeAuthor
#2 paragraphs-library-content-translation-status-null-2947259-2.patch1.48 KBBerdir
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

Berdir created an issue. See original summary.

Berdir’s picture

Berdir
Primary language German
Location Switzerland
CreditAttribution: Berdir commented
Status: Active » Needs review
FileSize
paragraphs-library-content-translation-status-null-2947259-2.patch1.48 KB

Status: Needs review » Needs work

The last submitted patch, 2: paragraphs-library-content-translation-status-null-2947259-2.patch, failed testing. View results
- codesniffer_fixes.patch Interdiff of automated coding standards fixes only.

Berdir’s picture

Berdir
Primary language German
Location Switzerland
CreditAttribution: Berdir commented
Status: Needs work » Needs review

  • Berdir committed a275a9e on 8.x-1.x 
    Issue #2947259 by Berdir: Library status field upgrade path can fail if...
Berdir’s picture

Berdir
Primary language German
Location Switzerland
CreditAttribution: Berdir commented
Status: Needs review » Fixed

Committed.

Status: Fixed » Closed (fixed)

Automatically closed - issue fixed for 2 weeks with no activity.

zanematthew’s picture

zanematthew CreditAttribution: zanematthew commented

I've have installed this update, and the issue persists.