API page: https://api.drupal.org/api/drupal/core%21lib%21Drupal%21Core%21Form%21Co...

The comment for that method is the following.

// By default, render the form using theme_system_config_form().

Theme hook functions don't exist anymore. The comment should reference the template file, which actually exists.

CommentFileSizeAuthor
#5 update-comment-2780405-5.patch582 bytesapaderno
#2 update-comment-2780405-2.patch582 bytesapaderno
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

kiamlaluno created an issue. See original summary.

apaderno’s picture

apaderno
he/him
Primary language Italian
Location Brescia, 🇮🇹 🇪🇺
CreditAttribution: apaderno as a volunteer commented
Status: Active » Needs review
FileSize
update-comment-2780405-2.patch582 bytes
apaderno’s picture

apaderno
he/him
Primary language Italian
Location Brescia, 🇮🇹 🇪🇺
CreditAttribution: apaderno as a volunteer commented
Title: ConfigFormBase::buildForm() comment contain a reference to a theme hook, which a function that doesn't exist » ConfigFormBase::buildForm() comment contain a reference to a theme hook, which is a function that doesn't exist

Version: 8.2.x-dev » 8.3.x-dev

Drupal 8.2.0-beta1 was released on August 3, 2016, which means new developments and disruptive changes should now be targeted against the 8.3.x-dev branch. For more information see the Drupal 8 minor version schedule and the Allowed changes during the Drupal 8 release cycle.

apaderno’s picture

apaderno
he/him
Primary language Italian
Location Brescia, 🇮🇹 🇪🇺
CreditAttribution: apaderno as a volunteer commented
FileSize
update-comment-2780405-5.patch582 bytes
joachim’s picture

joachim CreditAttribution: joachim at Torchbox commented
Status: Needs review » Reviewed & tested by the community
alexpott’s picture

alexpott
he/they
Primary language English
Location 🇪🇺🌍
CreditAttribution: alexpott at Chapter Three commented
Status: Reviewed & tested by the community » Fixed

Committed 249a9a8 and pushed to 8.1.x, 8.2.x, 8.3.x. Thanks!

  • alexpott committed fb0b048 on 8.3.x 
    Issue #2780405 by kiamlaluno: ConfigFormBase::buildForm() comment...

  • alexpott committed 32c90ff on 8.2.x 
    Issue #2780405 by kiamlaluno: ConfigFormBase::buildForm() comment...

  • alexpott committed 249a9a8 on 8.1.x 
    Issue #2780405 by kiamlaluno: ConfigFormBase::buildForm() comment...

Status: Fixed » Closed (fixed)

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