CKEditor should allow plugins to override default config
Problem/MotivationCurrently, Drupal\ckeditor\Plugin\Editor\CKEditor::getJSSettings() uses a + operator to combine the configuration arrays provided by plugins. In the event there are identical keys,...
View ArticleIncorrect data type for the expected parameter
Problem/MotivationIncorrect data type for the $expected parameter. * @dataProvider providerFilterAttributes * * @param string $html * Input HTML. * @param array $expected * The expected output string....
View ArticleRemove outdated TODO in \Drupal\tracker\Plugin\Menu\UserTrackerTab
Problem/MotivationIf currentUser() is not going to be moved to \Drupal\Core\Plugin\PluginBase, remove the TODO in \Drupal\tracker\Plugin\Menu\UserTrackerTab.Proposed resolutionRemove the TODO in...
View Article[Regression] login link has no destination=drupalSettings.path, so dumps you...
Problem/MotivationFollow-up to #2503755: Switch from user login block to login menu link and search block in standard profilelogin link has no destination=[current.path], so dumps you on the...
View ArticleMultiple usages of FieldPluginBase::getEntity do not check for NULL, leading...
Problem/MotivationLike the already committed #2913971: Views regression: 8.4.x EntityField can't handle a null row value from a non-required relationship, there are multiple places where the return...
View ArticleAdd optional support for CKEditor 5 in D9 so we can remove CKE 4 from Drupal 10
Problem/MotivationDrupal 8 is running on CKEditor 4.x. CKEditor 5.x has gone stable and looks like it has enhancements to help improve the authoring experience.CKEditor 5 also no longer relies on...
View ArticleX-Drupal-Cache-Tags / X-Drupal-Cache-Context header tags can exceed HTTP...
ORIGINAL TITLE: http.response.debug_cacheability_headers TRUE could cause a WSOD - AH01070: Error parsing script headersI was getting a WSOD with apache 2.4.29_1 and php-fpm 7.1.14 with the following...
View ArticleRemove redirect to front page after submitting contact form
Problem/MotivationCurrently we are redirecting to <front> after a user submits the contact form. It seems like this was introduced in 744309f9 which is #2125253: Convert $form_state['redirect']...
View ArticleDrupal.Message.clear throws exception if messageWrapper not initialized
Issue SummaryI ran into this error using the following code in an attempt to clear any existing messages before adding another via the new JS Messages API.var messages = new Drupal.Message();...
View ArticleCKEditor paste from word/plaintext stopped working after 8.4
It looks like Drupal 8.4 brought an update to CKEditor from 4.6 to 4.7 where essentially CKEditor ended functionality for the paste buttons, seemingly because they were a security issue. Clicking on...
View ArticleIncorrect use of UnprocessableHttpEntityException in...
The EntityResource controller class has a deserialize() method which takes care of building the entity object from the submitted request data.At some point, if either UnexpectedValueException or...
View Articleurl() should return / when asked for the URL of the frontpage
I've been very confused with the addition of canonical links in Drupal 7. The default behaviour seems to be to assign the actual node url of the node being displayed on the front page as the canonical...
View ArticleDeprecate file_default_scheme() and just use config
Problem/MotivationWe want to deprecated procedural functions in file.inc and replace with methods on FileSystem or StreamWrapperManager.file_default_scheme() just wraps...
View ArticleMove file uri/scheme functions from file.inc and FileSystem to...
Problem/MotivationSee #2229865: [meta] Modernize File/StreamWrapper API, we need to move file scheme functions out of file.inc. StreamWrapperManager is a good place for them, but some were already...
View ArticleProperties of FieldType 'comment' should be declared computed
The comment FieldType defines several properties :statuscidlast_comment_timestamplast_comment_namelast_comment_uidcomment_countAside from status, all other properties are not meant to be updated from...
View ArticleImprove exception messages when plugins cannot be found
Problem/MotivationCurrently, when a plugin cannot be found, we get this obtuse error message:Drupal\Component\Plugin\Exception\PluginException: The plugin (%plugin) did not specify an instance class in...
View ArticleCreate Migration Lookup service
Problem/MotivationThere are a 4-5 process plugins in core which use the following pattern: /** * {@inheritdoc} */ public function __construct(array $configuration, $plugin_id, $plugin_definition,...
View ArticleReplace calls to format_string() with Drupal\Component\Render\FormattableMarkup
In #2221771: Mark all simple wrappers in bootstrap.inc and common.inc as deprecated the function format_string() is deprecated. Let's remove all calls to it.
View ArticlePublisher ID (Authored By) not setting default value
We have a Media Type "Image" that, unlike its other media type brethren, shows the field "Publisher ID", and it doesn't pre-populate to the current user. Additionally, the other media types show...
View ArticleNotice: Undefined index: #include_fallback
Updated a small, simple, brochure site from 8.6.12 to 8.7.0.composer update drupal/core webflo/drupal-core-require-dev --with-dependencies had no complaints.update.php produced three identical...
View Article