In previous versions of Drupal, the Form Builder Functions topic in api.drupal.org gave a handy list of form builder functions.

In Drupal 8, we pretty much don't have these any more, so
https://api.drupal.org/api/drupal/core!includes!form.inc/group/forms/8
is pretty much empty.

So, we should remove this topic and all references to it.

To find the documentation that defines it, and the references to it, look for any of the following in Drupal 8 code:

@defgroup forms
@ingroup forms
@addtogroup forms
@link forms
@see forms

The entire doc block with the @defgroup in it should be removed. The other references should just be removed from their doc blocks.

CommentFileSizeAuthor
#1 drupal-2290325-1.patch6.37 KBcs_shadow
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

cs_shadow’s picture

Status: Active » Needs review
FileSize
6.37 KB
cs_shadow’s picture

Removed instances of @defgroup forms and @ingroup forms. No instances of @addtogroup forms, @link forms and @see forms.

joachim’s picture

We do still have form builders in Drupal though -- they're just classes now, right?

That said, I have always wondered what the point was of having a topic that listed all of Drupal's forms.

cs_shadow’s picture

@joachim, Yes. Form builders are classes in D8. This is the FormBuilder class: \Drupal\Core\Form\FormBuilder (https://api.drupal.org/api/drupal/core%21lib%21Drupal%21Core%21Form%21Fo...)

jhodgdon’s picture

Status: Needs review » Reviewed & tested by the community

Looks good, thanks! And yes, the point of this issue is that the topic is not needed. You can look at the class hierarchic of FormBuilder to discover form classes instead.

jhodgdon’s picture

Status: Reviewed & tested by the community » Fixed

Thanks again! Committed to 8.x.

  • jhodgdon committed 90a3498 on 8.x
    Issue #2290325 by cs_shadow: Get rid of form builder functions topic and...

Status: Fixed » Closed (fixed)

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