There are several topics/defgroups that should be removed from the code base, because they are not really relevant to developers or not enough to merit having a topic.
To do this, you need to:
a) Find the @defgroup doc block (see https://www.drupal.org/node/1354#defgroup for syntax) and remove it completely.
b) See if there are any lines in the same file or other files saying @ingroup (group name) or @addtogroup (group name) or
/**
* @} End of "defgroup (group name)".
*/
and remove those too.
List of topics to be removed:
1) https://api.drupal.org/api/drupal/core%21modules%21file%21file.module/gr...
2) https://api.drupal.org/api/drupal/core!includes!common.inc/group/http_ha...
3) https://api.drupal.org/api/drupal/core!includes!menu.inc/group/menu_tree...
4) https://api.drupal.org/api/drupal/core!modules!update!update.manager.inc...
5) https://api.drupal.org/api/drupal/core!modules!update!update.manager.inc...
6) https://api.drupal.org/api/drupal/core!modules!update!update.manager.inc...
| Comment | File | Size | Author |
|---|---|---|---|
| #6 | drupal8-defgroups-removed-2299691-6.patch | 8.82 KB | pushpinderchauhan |
| #6 | interdiff-2299691-5-6.txt | 928 bytes | pushpinderchauhan |
Comments
Comment #1
jhodgdonOne more:
7) https://api.drupal.org/api/drupal/core%21modules%21system%21system.api.p...
Comment #2
pushpinderchauhan commentedRemoved from mention files, if it's correct then can look into other files. Please review.
Comment #3
jhodgdonIt looks to me as though you've removed a LOT of defgroups here. We only want those specific 7 defgroups listed above to be removed. In common.inc, the only one is http_handling. Thanks!
Comment #4
pushpinderchauhan commented@jhodgdon, thankyou for your feedback. I hope in this patch, removed correct defgroups. Please review.
Comment #5
jhodgdonLooks good, thanks!
There are a couple of @see/@link references to tupdate_api in system.api.php that need to be removed also (just remove the whole sentence with the @link and the whole line with the @see).
The rest is perfect!
Comment #6
pushpinderchauhan commented@jhodgdon, thanks for your feedback.
Please review updated patch.
Comment #7
jhodgdonLooks good, thanks! I'm glad you figured out what I meant (I had a typo in comment #5 and it said "tupdate_api" incorrectly).
Comment #8
webchickBEGONE! ;) Thanks for helping to clean this up and make the important info more obvious.
Committed and pushed to 8.x. Thanks!