#D8 Accelerate London

⚑️ Live updates comments, jobs, and issues, tagged with #D8 Accelerate London will update issues and activities on this page.

Issues

The last 100 updated issues.

Activities

The last 7 days of comments and CI jobs.

  • Might be worth a follow-up to try to remove the overrides, then we wouldn't need to update them in Drupal 12 except for when we delete them.

    There are a lot of them:

    core/modules/block/src/BlockListBuilder.php
      public function getDefaultOperations(EntityInterface $entity) {
    core/modules/block_content/src/BlockContentTypeListBuilder.php
      public function getDefaultOperations(EntityInterface $entity) {
    core/modules/comment/src/CommentTypeListBuilder.php
      public function getDefaultOperations(EntityInterface $entity) {
    core/modules/field_ui/src/FieldConfigListBuilder.php
      public function getDefaultOperations(EntityInterface $entity) {
    core/modules/filter/src/FilterFormatListBuilder.php
      public function getDefaultOperations(EntityInterface $entity) {
    core/modules/image/src/ImageStyleListBuilder.php
      public function getDefaultOperations(EntityInterface $entity) {
    core/modules/menu_ui/src/MenuListBuilder.php
      public function getDefaultOperations(EntityInterface $entity) {
    core/modules/node/src/NodeTypeListBuilder.php
      public function getDefaultOperations(EntityInterface $entity) {
    core/modules/responsive_image/src/ResponsiveImageStyleListBuilder.php
      public function getDefaultOperations(EntityInterface $entity) {
    core/modules/search/src/SearchPageListBuilder.php
      public function getDefaultOperations(EntityInterface $entity) {
    core/modules/shortcut/src/ShortcutSetListBuilder.php
      public function getDefaultOperations(EntityInterface $entity) {
    core/modules/taxonomy/src/VocabularyListBuilder.php
      public function getDefaultOperations(EntityInterface $entity) {
    core/modules/user/src/RoleListBuilder.php
      public function getDefaultOperations(EntityInterface $entity) {
    core/modules/views_ui/src/ViewListBuilder.php
      public function getDefaultOperations(EntityInterface $entity) {
    

    Two content entity list builders too:

    core/modules/menu_link_content/src/MenuLinkListBuilder.php
      public function getDefaultOperations(EntityInterface $entity) {
    core/modules/menu_link_content/src/MenuLinkListBuilder.php
      public function getDefaultOperations(EntityInterface $entity) {
    

    They do various things, so I don't think they can be removed.

    Of relevance to this issue, these implementations of getDefaultOperations() include access checks inside them, so I wonder if cacheability needs to be captured.
    core/modules/field_ui/src/FieldConfigListBuilder.php
    core/modules/taxonomy/src/VocabularyListBuilder.php
    core/modules/workspaces/src/WorkspaceListBuilder.php

  • πŸ‡¦πŸ‡ΊAustralia acbramley

    Do you know why?

    It looks like it might have been accidental. See the following commit where public function getOperations was changed to public function getDefaultOperations

    https://git.drupalcode.org/project/drupal/-/commit/b504423ed07e9bb437e96...

    Added πŸ› getDefaultOperations() in ConfigEntityListBuilder and sub classes should be protected Active

  • πŸ‡¬πŸ‡§United Kingdom catch

    ConfigEntityListBuilder and all its subclasses override this and make getDefaultOperations() public. Wild.

    Do you know why? Might be worth a follow-up to try to remove the overrides, then we wouldn't need to update them in Drupal 12 except for when we delete them.

    All the list builder classes will need to have this parameter added in D12, but not sure if all the list builder classes need to be updated now to indicate the change?

    It would probably help anyone subclassing the subclasses if we did that, I don't think we've had a similar situation yet - the commented out added parameters is pretty new in itself.

Production build 0.71.5 2024