Also referencing Issues:
#820816: #type text_format not compatible with system_settings_form($automatic_defaults = TRUE)
and
#266246: Add smart defaults for system_settings_form

on line 1584 of systems.admin.inc, system_settings_form is being passed a second parameter when the api does not show support for it. After looking into the patches in the above listed issues I see this was fixed in other functions but looks like it was missed on system_cron_settings.

CommentFileSizeAuthor
system_settings_form_incorrect_api_usage.patch474 bytesmegensel
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

droplet’s picture

droplet CreditAttribution: droplet commented
Version: 7.x-dev » 8.x-dev
Status: Needs review » Reviewed & tested by the community

Minor change, bots happy, im happy.

rfay’s picture

rfay
Primary language English
Location Palisade, CO, USA
CreditAttribution: rfay commented
Issue tags: +Needs backport to D7

Marking needs backport.

Damien Tournoud’s picture

Damien Tournoud CreditAttribution: Damien Tournoud commented
Issue tags: -Needs backport to D7

This was missed when #266246: Add smart defaults for system_settings_form was reverted. I'm in favor.

Damien Tournoud’s picture

Damien Tournoud CreditAttribution: Damien Tournoud commented
Issue tags: +Needs backport to D7

Restoring tag.

Dries’s picture

Dries CreditAttribution: Dries commented
Status: Reviewed & tested by the community » Fixed

Good catch. Committed to 7.x and 8.x. Thanks!

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