Problem/Motivation

API page: https://api.drupal.org/api/drupal/core%21modules%21views%21views.api.php...

See https://api.drupal.org/api/drupal/core%21modules%21views%21views.api.php...

This should contain the actual hooks, surely.

Proposed resolution

Remove

Remaining tasks

Patch

CommentFileSizeAuthor
#7 3028837-7.patch560 bytesquietone
#4 3028837-4.patch417 bytesquietone

Comments

joachim created an issue. See original summary.

Version: 8.6.x-dev » 8.8.x-dev

Drupal 8.6.x will not receive any further development aside from security fixes. Bug reports should be targeted against the 8.8.x-dev branch from now on, and new development or disruptive changes should be targeted against the 8.9.x-dev branch. For more information see the Drupal 8 and 9 minor version schedule and the Allowed changes during the Drupal 8 and 9 release cycles.

Version: 8.8.x-dev » 8.9.x-dev

Drupal 8.8.7 was released on June 3, 2020 and is the final full bugfix release for the Drupal 8.8.x series. Drupal 8.8.x will not receive any further development aside from security fixes. Sites should prepare to update to Drupal 8.9.0 or Drupal 9.0.0 for ongoing support.

Bug reports should be targeted against the 8.9.x-dev branch from now on, and new development or disruptive changes should be targeted against the 9.1.x-dev branch. For more information see the Drupal 8 and 9 minor version schedule and the Allowed changes during the Drupal 8 and 9 release cycles.

quietone’s picture

quietone commented
Version: 8.9.x-dev » 9.4.x-dev
Issue summary: View changes
Status: Active » Needs review
Issue tags: +Bug Smash Initiative
StatusFileSize
new 3028837-4.patch417 bytes

I compared view.api.php with other api.php files and looked at

@defgroup, @addtogroup, @ingroup, @{, @}: Groups and topics and I think this can be removed. iIt doesn't seem to offer any value to the documentation.

Not running tests because this is a doc only change.

joachim’s picture

joachim commented
Status: Needs review » Reviewed & tested by the community

LGTM.

alexpott’s picture

alexpott
he/they
Primary language English
Location 🇪🇺🌍
commented
Status: Reviewed & tested by the community » Needs work

This looks like a hang-over from the D7 views module when it is not part of core. The views hooks are now listed on the core hooks page which make sense to me - see https://api.drupal.org/api/drupal/core%21core.api.php/group/hooks/9.2.x

We should also remove the final 

/**
 * @}
 */

So the number of closing and opening API tags are the same.

quietone’s picture

quietone commented
Status: Needs work » Needs review
StatusFileSize
new 3028837-7.patch560 bytes

Ah, I should have seen that.

joachim’s picture

joachim commented
Status: Needs review » Reviewed & tested by the community

  • catch committed 9f6cef7 on 9.4.x 
    Issue #3028837 by quietone, joachim, alexpott: views_hooks group has...

  • catch committed 3eccfbd on 9.3.x 
    Issue #3028837 by quietone, joachim, alexpott: views_hooks group has...
catch’s picture

catch
he/him
Primary language English
commented
Version: 9.4.x-dev » 9.3.x-dev
Status: Reviewed & tested by the community » Fixed

Committed/pushed to 9.4.x and cherry-picked to 9.3.x, thanks!

Status: Fixed » Closed (fixed)

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