Commit e6793cab authored by catch's avatar catch
Browse files

Issue #3318992 by wells, smustgrave, larowlan: Increase access level required... 

Issue #3318992 by wells, smustgrave, larowlan: Increase access level required for content permissions rebuilds
parent c326b923

core/modules/node/node.module

+1 −1
Original line number Diff line number Diff line
@@ -42,7 +42,7 @@ function node_help($route_name, RouteMatchInterface $route_match) { 
  // for rebuild. We don't need to issue the message on the confirm form, or

  // while the rebuild is being processed.

  if ($route_name != 'node.configure_rebuild_confirm' && $route_name != 'system.batch_page.html' && $route_name != 'help.page.node' && $route_name != 'help.main'

    && \Drupal::currentUser()->hasPermission('access administration pages') && node_access_needs_rebuild()) {

    && \Drupal::currentUser()->hasPermission('administer nodes') && node_access_needs_rebuild()) {

    if ($route_name == 'system.status') {

      $message = t('The content access permissions need to be rebuilt.');

    }

core/modules/node/node.routing.yml

+1 −1
Original line number Diff line number Diff line
@@ -164,4 +164,4 @@ node.configure_rebuild_confirm: 
  defaults:

    _form: '\Drupal\node\Form\RebuildPermissionsForm'

  requirements:

    _permission: 'access administration pages'
 
    _permission: 'administer nodes'

core/modules/node/tests/src/Functional/NodeAccessRebuildNodeGrantsTest.php

+1 −0
Original line number Diff line number Diff line
@@ -40,6 +40,7 @@ protected function setUp(): void { 
      'administer site configuration',

      'access administration pages',

      'access site reports',

      'administer nodes',

    ]);

    $this->drupalLogin($this->adminUser);