drupal 8.4.2

This is a patch release of Drupal 8 and is ready for use on production sites. See the CHANGELOG.txt for a summary of changes and improvements since the last minor release. Learn more about Drupal 8.

This release downgrades the ckeditor library included with Drupal core from version 4.7.3 to 4.7.2, otherwise it is identical to Drupal 8.4.1.

This release only contains bug fixes, along with documentation and testing improvements.

If you have not already upgraded to 8.4.0, read the Drupal 8.4.0 release notes before upgrading to 8.4.2.

The following important migrate issues were resolved in 8.4.1 (in addition to the dozens of other fixes listed at the end of this post):

• #2912120: Migrations with a highwater property are not applying correct orderBy() on first run
• #2914649: [D7] Vocabulary migration: vid incorrectly mapped from vocabulary name instead of machine_name
• #2887142: NodeType source plugin should include comment information
• #2897254: URLs without http:// are broken after migration from d6 or d7

Known issues

There are no known regressions in this release.

Search the issue queue for all known issues.

Changes since 8.4.0:

• #2919983 by xjm: Restore the 8.4.0 phpcs.xml.dist on 8.4.x
• #2919130 by masipila, heddn: Add masipila to MAINTAINERS.txt for migrate subsystem
• #2864995 followup by amateescu, xjm: Don't disrupt QueryInterface in a patch release.
• \Revert "Issue #2915673 by borisson_, alexpott, Gábor Hojtsy: Ignore coding standards in sites/* PHP files as they are for configuration and not real code"
• #2915673 by borisson_, alexpott, Gábor Hojtsy: Ignore coding standards in sites/* PHP files as they are for configuration and not real code
• #2914668 by Jo Fitzgerald, shashikant_chauhan, heddn, quietone: MigrateNodeDeriverTest::testNoTranslations() does not install node module
• #2913460 by Eric_A: drupal/core-class-finder missing from replace section
• #2913459 by Eric_A: drupal/core-class-finder missing in ComposerIntegrationTest
• #2825204 by dawehner, BR0kEN, xjm, pcambra, Wim Leers, tim.plunkett, tstoeckler, damiankloip, larowlan, effulgentsia, alexpott: REST views: authentication is broken
• #2915490 by Wim Leers: Modernize & harden HalLinkManagerTest
• #2905227 by maxocub, yoroy, jjpoole, heddn, phenaproxima, vulcanr, quietone, wturrell, xjm: Migrate UI: Improve 'Review Upgrade' page UX
• #2912120 by phenaproxima, heddn, Berdir, quietone, maxocub: Migrations with a highwater property are not applying correct orderBy() on first run
• Revert "Issue #2843765 by vaplas: EntityResource: Provide comprehensive test coverage for EntityViewDisplay entity"
• Revert "Issue #2701393 by tim.plunkett, Wim Leers, thpoul, xjm: Switching between editors on the format configuration causes errors upon save"
• Revert "Issue #2912399 by wengerk, Wim Leers: Extend the CKEditorIntegrationTest for DrupalImage"
• #2888905 by vaplas, Lendude: ConvertConfigTranslationUiTest WTB to JTB & BTB for config_translation module
• #2843765 by vaplas: EntityResource: Provide comprehensive test coverage for EntityViewDisplay entity
• #2909164 by drunken monkey, dawehner: Fatal error with stub container in DependencySerializationTrait::__wakeup()
• #2912399 by wengerk, Wim Leers: Extend the CKEditorIntegrationTest for DrupalImage
• #2701393 by tim.plunkett, Wim Leers, thpoul, xjm: Switching between editors on the format configuration causes errors upon save
• #2915767 by heddn: CckFieldPluginBase doesn't implement MigrateCckFieldInterface
• #2864995 by amateescu, Dinesh18, Sam152: Allow entity query to query the latest revision
• #2897254 by Jo Fitzgerald, heddn, rakesh.gectcr, maxocub, masipila, phenaproxima, larowlan: URLs without http:// are broken after migration from d6 or d7
• #2887311 by naveenvalecha, vaplas, Lendude: Convert BlockContentTypeTest web tests to browser tests for block_content module
• #2809483 by michielnugter, ApacheEx, Lendude, dawehner: Convert AJAX part of \Drupal\field\Tests\FormTest::testFieldFormJSAddMore to JavascriptTestBase
• #2870439 by vaplas, andypost, Jo Fitzgerald, jonathan1055, Lendude, dawehner, alexpott, GoZ: Convert web tests to browser tests for config module
• #2887813 by vaplas: Convert web tests to browser tests for content_translation module Part 2
• #2908600 by neclimdul: YamlTest writes temporary files but never uses them
• #2870457 by ApacheEx, andypost, dawehner, naveenvalecha, vaplas, nlisgo, Wim Leers, Lendude, alexpott: Convert web tests to browser tests for page_cache module
• #2910081 by mpdonadio, neclimdul, gambry, jhedstrom, alexpott: DateTimePlus calls should be chainable
• #2914033 by alexpott, borisson_: Complete deprecation of install_ensure_config_directory()
• #2915900 by David_Rothstein: MAINTAINERS.txt doesn't indicate that the database API is in need of a maintainer
• #2909163 by mfernea: Fix 'Drupal.Commenting.InlineComment.WrongStyle' coding standard
• #2916642 by quietone, heddn: Private and Public file migrations use the same name
• #2870456 by ApacheEx, Lendude: Convert web tests to browser tests for Settings Tray module
• #2573807 by 20th, Sam152, msgph, sherakama, heikki, jcnventura, Wim Leers: Fatal error: Call to a member function getTimestamp()
• #2843781 by vaplas, Wim Leers: EntityResource: Provide comprehensive test coverage for EntityViewMode entity
• #2909183 by Wim Leers, tedbow: Add path alias (PathItem) field PATCH test coverage
• #2901722 by mfernea: Fix 'Squiz.Functions.FunctionDeclarationArgumentSpacing' coding standard
• #2843780 by vaplas, jamesdesq, Wim Leers: EntityResource: Provide comprehensive test coverage for EntityFormMode entity
• #2578485 by Mile23, mayeco, TamCalderwood, alduya, Gravypower, Andy_D, alexpott, slde_rorschach: Composer::preAutoloadDump fails with no specified classmap
• #2915664 by alexpott, dawehner: Sites installed by InstallerTestBase should have a timezone of 'Australia/Sydney'
• #2916025 by Wim Leers: Rename $denormalized to $normalized in \Drupal\hal\Normalizer\FieldItemNormalizer::normalizedFieldValues()
• Update CHANGELOG with more accurate Drush compatibility information.
• #2572787 by alexpott, mfernea, attiks, andriyun, andypost, RoSk0, hussainweb, klausi, martin107: Fix 'Drupal.WhiteSpace.CloseBracketSpacing' coding standard
• #2911165 by mfernea, Jo Fitzgerald: LoggingTest.php - Weird comments
• #2901572 by Jo Fitzgerald, bluccas, mfernea, klausi: Fix 'Drupal.Commenting.PostStatementComment' coding standard
• #2908864 by Wim Leers, eleleka: Update CKEditor library to 4.7.3
• #2711353 by Jo Fitzgerald, rakesh.gectcr, quietone, chx, Yogesh Pawar, Pavan B S, biguzis, mikeryan, vasi, catch, iMiksu, dawehner, Berdir, benjy: Migrate never unsets existing data for content entitites
• #2907420 by DyanneNova, tedbow: The Off-Canvas CSS reset prevents SVGs from displaying
• Revert "Issue #2894068 by Jo Fitzgerald, davidsickmiller, Yogesh Pawar, heddn, quietone: datetime_type is not set correctly when migrating datetime fields from D7"
• #2356913 by jmolivas, mgifford, maurizio.ganovelli, zhuber: Fix rebuild_token_calculator
• #2881348 by benjifisher, guncha25, Jo Fitzgerald, anya_m, Dinesh18, pritish.kumar, andypost, alexpott, James Nesbitt, larowlan, znerol, dawehner: SessionCacheContext calls getId() on null
• #2894068 by Jo Fitzgerald, davidsickmiller, Yogesh Pawar, heddn, quietone: datetime_type is not set correctly when migrating datetime fields from D7
• #2909368 by mfernea, reflie, zaporylie: Fix 'Drupal.Commenting.VariableComment.InlineVariableName' coding standard
• #2892377 by Wim Leers, Gábor Hojtsy: Document relationship of entities, entity revisions and entity translations
• #2805449 by joelpittet: Comment out 'extension_discovery_scan_tests' in example.settings.local.php
• #2680097 by maxocub, quietone, vg3095, heddn, xjm: Update Migrate entries in MAINTAINERS.txt
• #2914738 by _gradient_: Fix typo in MapperInterface::getInstance() description
• #2862670 by Jo Fitzgerald, pk188, heddn, edysmp, shashikant_chauhan, dhruveshdtripathi, pritish.kumar, quietone, maxocub, mikeryan: Add documentation to SourcePluginBase source plugin
• #2913957 by geertvd, mpdonadio: Fix invalid casing in date argument's namespace
• #2914408 by fgm: Incorrect code/endcode in API files
• #2912666 by shashikant_chauhan, tobiberlin, dawehner: Clarify param in KernelTestBase::installConfig()
• #2898373 by Mile23, Jo Fitzgerald, joachim, dawehner, Gábor Hojtsy, Lendude: Gaps in docs for KernelTestBase
• #2909367 by iKb, zaporylie, mfernea: Fix 'Drupal.Commenting.VariableComment.DuplicateVar' coding standard
• #2909373 by karthikkumarbodu, xjm, mfernea, dawehner: Views TaxonomyIndexTidDepth has a weird commented IN condition
• #2901789 by mfernea, andriyun, borisson_: Fix 'PEAR.Functions.FunctionCallSignature' coding standard
• #2572699 by rajeshwari10, andriyun, mfernea, pashupathi nath gajawada, marvin_B8, attiks, andypost, alexpott: Fix 'Drupal.ControlStructures.InlineControlStructure' coding standard
• #2901744 by mfernea: Fix 'PSR2.Namespaces' coding standard
• #2901726 by mfernea, gmario, finn.lewis, robertoperuzzo, rachel_norfolk: Fix 'Squiz.Functions.MultiLineFunctionDeclaration' coding standard
• #2873750 by eheller: Add Change record to @deprecated for install_ensure_config_directory()
• #2887142 by maxocub, phenaproxima, Jo Fitzgerald, Adita, heddn, andypost: NodeType source plugin should include comment information
• #2911280 by alexpott, andypost, dawehner: RectangleTest.php takes a very long time to scan for coding standards
• #2851468 by tameeshb, dhirendra.mishra, alexpott, joelpittet: Wrong usage of file_prepare_directory in hook_install documentation
• #2857789 by vegantriathlete, snte, narnua, xjm, lauriii: Correct spelling/non-standard use of "therefor"
• #2902190 by chenderson: Docblock on FileUsageTemporaryDeletionConfigurationUpdateTest::testUpdateHookN() is incorrect
• #2911164 by mfernea, Jo Fitzgerald: Undo accidental commenting of message in MetadataGeneratorTest
• #2911166 by mfernea, Jo Fitzgerald: Undo accidental commenting of message in EntityDefinitionUpdateTest
• #2873768 by dhopki12, WidgetsBurritos, harsha012, Mile23, catch: Add Change record to @deprecated for *.modules constants
• #2744663 by jholding, rivimey, John Cook, hctom, mahalingam_cs: Batch missing title on screen
• #2901478 by Sam152, Jo Fitzgerald, timmillwood: Test ModerationStateWidget::isApplicable with a BaseFieldOverride
• #2877480 by Yogesh Pawar, hardik.p, eojthebrave, Cyberschorsch: Update #cache example code on Render API overview to use Node::getCacheTags instead of hard coding
• #2914649 by masipila, dipakmdhrm, phenaproxima, maxocub: [D7] Vocabulary migration: vid incorrectly mapped from vocabulary name instead of machine_name
• #2857843 by vaplas, mpdonadio, Jo Fitzgerald, gambry: Random fail in Drupal\KernelTests\Core\Entity\ContentEntityChangedTest::testChanged
• #2914249 by maxocub, dillix, masipila, phenaproxima: Translated forum vocabulary migration creates duplicate forum vocabularies
• #2903007 by rakesh.gectcr, dipakmdhrm, maxocub, dillix, masipila, quietone, phenaproxima: [D7] Forum containers are migrated as forums
• #2902018 by harsha012, Meenakshi Gupta, vijaycs85, Sam152, timmillwood, amateescu: Use array_column instead of array_map where possible in the Workflows module
• #2894765 by bendev, Sam152, arunkumark, cedric_a: BlockContentModerationHandler::enforceRevisionsEntityFormAlter sets incorrect keys in form
• #2864613 by vaplas, Jo Fitzgerald, andypost, dawehner: Convert ViewEditTest in views_ui module to BrowserTestBase
• #2907282 by BramDriesen, joachim: EntityTypeBundleInfoInterface::getBundleInfo() param should say 'ID'
• #2759757 by gambry, claudiu.cristea, heldercor, dawehner, -enzo-, TimRutherford, amateescu, joachim: EntityQuery wrong SQL with two reference fields conditions targetting same entity type
• #2901943 by amateescu, Berdir: Content entity form validation does not respect the #limit_validation_errors property from field widgets
• #2913445 by Ashley George: Incorrect description of cache invalidation tags
• #2910005 by catch, samuel.mortenson: JavaScript errors thrown when viewing non-latest default revision of entity
• #2895685 followup by xjm: Remove outdated CHANGELOG references to the alpha.