Follow-up to #2845499: Content Cloud Beta

Problem/Motivation

Lingotek Content Cloud broke the build.

Proposed resolution

Fix the errors. Follow ups will be needed for non-critical issues.

Remaining tasks

None

User interface changes

None.

API changes

None

Data model changes

None

CommentFileSizeAuthor
#2 2846203-2.patch1.54 KBpenyaskito

Comments

penyaskito created an issue. See original summary.

penyaskito’s picture

penyaskito
he/him
Primary language Spanish
Location 🧑🏽‍🌾 Seville 💃 Andalusia, UTC+2 🇪🇺
commented
Status: Fixed » Needs review
StatusFileSize
new 2846203-2.patch1.54 KB

Fixed tests. Ensured drupal_flush_all_caches() is only called once. Fixed how to access the menu link manager service.

penyaskito’s picture

penyaskito
he/him
Primary language Spanish
Location 🧑🏽‍🌾 Seville 💃 Andalusia, UTC+2 🇪🇺
commented
Title: Content Cloud Beta » Fix build after Content Cloud Beta was committed
Category: Feature request » Bug report
penyaskito’s picture

penyaskito
he/him
Primary language Spanish
Location 🧑🏽‍🌾 Seville 💃 Andalusia, UTC+2 🇪🇺
commented
Status: Needs review » Reviewed & tested by the community

Tests are back to green.

  • penyaskito committed 55653e2 on 8.x-1.x 
    Issue #2846203 by penyaskito: Fix build after Content Cloud Beta was...
penyaskito’s picture

penyaskito
he/him
Primary language Spanish
Location 🧑🏽‍🌾 Seville 💃 Andalusia, UTC+2 🇪🇺
commented
Status: Reviewed & tested by the community » Fixed

Committed 55653e2 and pushed to 8.x-1.x. Thanks!
Fixed some PHPCS reports in commit.

Status: Fixed » Closed (fixed)

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