Comment on translated article - get redirected to node in original language...
Step to reproduce:enable content translation moduleadd second language and configure article as translatableAdd new article node and add translationpost comments to translated articleShould be...
View ArticleDo not use basic_html text format for 'No log messages available.'...
How to reproduce:Install site on Minimal installation profile.Enable Views module.Go to Recent log messages (admin/reports/dblog) page.Click Delete link for a clear log.Press Confirm on the next step...
View Article[regression] New event dispatch: a migrated entity is about to be saved
Problem/MotivationTrue, there is a MigrateEvents::PRE_ROW_SAVE event that allows subscribers to react before the row is passed to the destination plugin, but this is not enough. For the case when the...
View ArticleAdd helper functions to MigrateDrupal7TestBase
Problem/MotivationIssue #2554321: Clean up Migrate's test suite introduces helper methods in MigrateDrupal6TestBase, that allow to perform bundled migrations in tests, such as migrateUsers(),...
View ArticleFixes for MigrateUpgradeForm
Postponed on #2687843: Add back incremental migrations through the UIProblem/MotivationA review of the patch in #2687843: Add back incremental migrations through the UI comment #98 included some points...
View ArticleMigrate Drupal source plugins should use dependency injection
Follow-up from #2791041: Migrate source plugins should use dependency injectionThey currently do not, and it causes a lot of weird acrobatics and unsavory hacking in Migrate's tests. The way around...
View ArticleProvide a interface for common methods of sources and destinations
Problem/MotivationSources and destinations are sharing some common methods like getIds() and fields(). They are referring exactly the same things but each on the opposite endpoint of the pipe.Proposed...
View ArticleReuse option in FileCopy migrate process plugin not work with remote files
I have tested that code:process: uri: plugin: file_copy reuse: true source: - source_file_path_remote - destination_file_path In the transform method of file_copy process plugin the download process...
View ArticleClik here to view.
display: block for details/summary hides drop arrows in Firefox...
Problem/MotivationIn older versions of Firefox, details elements in the Seven theme in Drupal 8.x, such as on admin/config/system/site-information , looked like this screenshot -- the summary line...
View ArticleClean Up HTML & improve accessibility during migration
While migrating content to Drupal 8, it would be great if the HTML could be cleaned up a bit. All part of Authoring Tools Accessibility Guidelines Principle B.1: Fully automatic processes produce...
View ArticleEvaluate Accessibility Durring Migration
Would be nice to evaluate the accessibility of pages you are importing using something like aXe or Pa11y. https://github.com/dequelabs/axe-corehttp://pa11y.org/There's no reason we couldn't eliminate...
View ArticleUpdate UI standard page: Dropdown
Problem/MotivationThe UI standard page for Dropdownn needs to be updated https://www.drupal.org/docs/develop/user-interface-standards/dropbuttonProposed resolutionRewrite the page and add missing...
View ArticleMigrate plugin base classes should implement ConfigurablePluginInterface
Migration plugins throughout core and contrib keep having to do this sort of thing:$delimiter = isset($this->configuration['delimiter']) ? $this->configuration['delimiter'] : ''; Implementing...
View ArticleClik here to view.
Validat file path and db on Credential form
From https://www.drupal.org/node/2281691#comment-10286177...webchick:OBSERVATION There doesn't seem to be any validation around the file path field? Or at least I wasn't able to trigger it despite...
View ArticleAggregated css/js asset files can be garbage collected while cached pages...
We recently noticed a situation where a cached page on a reverse-proxy (in our case, cloudflare) was referring to a deleted aggregated js file, and so was getting a 404 response on that file, so the...
View ArticleProblems creating a d6 user profile field storage with an name longer than 32...
Problem/MotivationField names are limited to 32 characters, but profile field names may be longer on older sites. A Drupal 6 site with profile field names longer than 32 characters cannot be fully...
View ArticleRepresentative Node Views fails due to invalid SQL
The Taxonomy and User Representative Node (GroupwiseMax) using a RepresentativeView does not work because the existing code generates SQLSTATE[42000]: Syntax error or access violation and...
View ArticleLog message if static_map plugin skips the row
Problem/MotivationIt is hard to notice that static_map was the reason a migration row was skipped.Proposed resolutionLog a message to MigrateSkipRowException to clearly explain why the row is...
View Articleinvalid conf file warnings when skip_permissions_hardening is on
Problem/MotivationAt the moment when $settings['skip_permissions_hardening'] = TRUE, the system module's system_requirements unconditionally issues a warning about the site's config files' permissions...
View ArticleMove module_set_weight() into ModuleHandler::setWeight(), add...
Wondering why there is no corresponding module_get_weight() for module_set_weight(). The typical use case is to set my module's weight to one higher than another module, but I don't necessarily know...
View Article
