If a custom beforeSubmit() function on a Drupal.ajax object returns false,...
I was working with a file upload field (<input type ="file" />) in a Webform form today, and noticed that clicking on the "Upload" submit input before I had selected a file didn't prevent the...
View ArticleWhitelist, instead of blacklist, sensitive files and directories
Text editors create temporary or ".bak" files, version control systems create "hidden" directories and files. SQL backups are left littering the filesystem, translations, you name it. Database...
View ArticleHelper issue for "Comment field"
Ignore this one for reviews, this is just to post new periodic patches for the conversion of comment to field Original issue #731724: Decouple comment.module from nodeRelated issues:#1900132: [Meta]...
View ArticleUNION queries don't support PagerDefault
The following fails:<?php $query1 = db_select('a_blog', 'cb')         ->fields('cb', array('title', 'changed_date')); $query2 = db_select('a_gallery', 'cg')           ->fields('cg',...
View Articlemenu_navigation_links() generate a error
I'm try listing a menu navigation links with menu_navigation_links() function, but is generate a follow error:Notice: Undefined index: localized_options in menu_navigation_links() (line 1860 of...
View ArticleUpdate tag list for locale_string_is_safe() like filter_xss_admin()
Similarly to #732992: Allow filter_xss_admin() to accept HTML5 tags, can we update the tag list for locale_string_is_safe() to include safe HTML5 tags? Patch to follow.
View ArticleReturns FALSE if the file is not found.
API page: http://api.drupal.org/api/drupal/includes%21bootstrap.inc/function/drupa...Enter a descriptive title (above) relating to drupal_get_filename, then describe the problem you have found:There is...
View ArticleReturns FALSE if the file is not found.
API page: http://api.drupal.org/api/drupal/includes%21common.inc/function/drupal_g...Enter a descriptive title (above) relating to drupal_get_path, then describe the problem you have found: There is no...
View ArticleDrupal without DB but with site config throws uninformative error
Repeatable: Always Steps to repeat:Set up a fresh install of Drupal 8.Drop the site database.Visit the site in your browser.Expected results: An error explaining the root cause of the situation (Error...
View ArticleCM permission settings by MTimeProtectedFastFileStorage and FileStorage make...
Problem/MotivationWhen installing Drupal from the shell and configure through the site (or the other way around) the configuration directories are not writable for either the shell user or the web...
View ArticleContextual Links above Overlay
Hello guys,The contextual links have a z-index of 999 in order to be above every element. That causes some problems as the ones seen in the screenshot. To fix this, either: a) The overlay parent and...
View ArticleSkip hidden menu items
The following issue is against Drupal 8. It is a known performance issue and there is a working patch that is still not in Drupal core:http://drupal.org/node/1710656Just before...
View Article[meta] hook_help() text is seriously out of date and/or missing in most modules
There are quite a lot of new modules in Drupal 8.x without any hook_help() text at all. Also, many of the existing modules' functionality has changed or been split up, and the hook_help() text does not...
View ArticleUpdate edit modules implementation of hook_toolbar; refresh the design to...
The hook hook_toolbar was updated in #1847198-73: Update the structure returned by hook_toolbar().With these changes, we can finally improve the design of the Edit mode toggle. Here is a screenshot...
View ArticleSeparate Action Links from hook_menu()
Spinoff from #1889790: Allow modules to register links for menus, breadcrumbs and tabs (if not with hook_menu):We rip action links out of hook_menu entirely, using essentially the model from...
View ArticleRemove unused code from toolbar_pre_render_item that throws a warning on...
Problem/MotivationWhile working on #1741498: Add a mobile preview bar to Drupal core, I noticed that passing a custom render array to toolbar for a tab raised the following warning:Notice: Undefined...
View ArticleAllow #ajax['dialog'] to contain options other than 'modal'
Spin-off from #1870764: Add an ajax command which makes it easy to use the dialog API in complex cases. That issue is trying to accomplish several things related to better integrating Drupal's AJAX...
View ArticleREST DELETE does not log a watchdog message
DELETE operation does not leave behind a watchdog message like a web delete does. Please check PATCH/PUT as well.
View ArticleAdd CURL example for GET to admin/config/services/rest
I think we should add a json (not json-ld) example to the rest_help() for GET.
View ArticleUse modals in operations column of language settings config page
Follow up for #1807692: Introduce a column synchronization capability to translate alt and titles through the core image field widgetProblem/Motivationparts of the operations column should be modals...
View Article