Skip to content
Snippets Groups Projects

Issue #3384600: Deprecate and stop using EntityPermissionsRouteProviderWithCheck

Merge request reports

Loading
Loading

Activity

Filter activity
  • Approvals
  • Assignees & reviewers
  • Comments (from bots)
  • Comments (from users)
  • Commits & branches
  • Edits
  • Labels
  • Lock status
  • Mentions
  • Merge request status
  • Tracking
159 159 *
160 160 * @return \Drupal\Core\Access\AccessResultInterface
161 161 * The access result.
162 *
163 * @deprecated in drupal:10.2.0 and is removed from drupal:11.0.0. Use
  • 159 159 *
    160 160 * @return \Drupal\Core\Access\AccessResultInterface
    161 161 * The access result.
    162 *
    163 * @deprecated in drupal:10.2.0 and is removed from drupal:11.0.0. Use
    164 * a permissions check in the route definition instead.
    165 * @see https://www.drupal.org/node/3384745
    162 166 */
    163 167 public function access(Route $route, RouteMatchInterface $route_match, $bundle = NULL): AccessResultInterface {
    168 trigger_error(__CLASS__ . ':access() is deprecated in drupal:10.2.0 and is removed from drupal:11.0.0. Use a permissions check on the route definition instead. See https://www.drupal.org/node/3384745');
  • spokje added 30 commits

    added 30 commits

    Compare with previous version

  • spokje added 1 commit

    added 1 commit

    • 5b41a529 - changed deprecated/removed version numbers

    Compare with previous version

  • spokje added 1 commit

    added 1 commit

    Compare with previous version

  • spokje added 22 commits

    added 22 commits

    Compare with previous version

  • 93 93 // Get the names of all config entities that depend on $this->bundle.
    94 94 $config_name = $this->bundle->getConfigDependencyName();
    95 95 $config_entities = $this->configManager
    96 ->getConfigEntitiesToChangeOnDependencyRemoval('config', [$config_name]);
    96 ->findConfigEntityDependenciesAsEntities('config', [$config_name]);
    97 97 $config_names = array_map(
    98 98 function ($dependent_config) {
    99 99 return $dependent_config->getConfigDependencyName();
    100 }, $config_entities['delete'] ?? []
    100 }, $config_entities
  • 60 61 $module_handler = $this->prophesize(ModuleHandlerInterface::class)->reveal();
    61 62 $module_extension_list = $this->prophesize(ModuleExtensionList::class)->reveal();
    62 63 $prophecy = $this->prophesize(ConfigManagerInterface::class);
    63 $prophecy->getConfigEntitiesToChangeOnDependencyRemoval('config', ['node.type.article'])
    64 $prophecy->findConfigEntityDependenciesAsEntities('config', ['node.type.article'])
    64 65 ->willReturn([
    65 'delete' => [
    66 66 new ConfigEntityDependency('core.entity_view_display.node.article.full'),
    67 67 new ConfigEntityDependency('field.field.node.article.body'),
    68 ],
  • added 82 commits

    Compare with previous version

  • added 30 commits

    Compare with previous version

  • Brian Suttis added 1 commit

    added 1 commit

    Compare with previous version

  • Brian Suttis added 27 commits

    added 27 commits

    Compare with previous version

  • Brian Suttis added 13 commits

    added 13 commits

    Compare with previous version

  • Brian Suttis added 1 commit

    added 1 commit

    • 0a000930 - Update comment to remove references to EntityPermissionsRouteProviderWithCheck

    Compare with previous version

  • 13 13 /**
    14 14 * Provides routes for the entity permissions form.
    15 15 *
    16 * Use this class or EntityPermissionsRouteProviderWithCheck as a route
    17 * provider for an entity type such as Vocabulary. Either one will provide
    18 * routes for the entity permissions form. The
    19 * EntityPermissionsRouteProviderWithCheck class provides a custom access check:
    20 * it denies access if there are no entity-specific permissions. If you know
    21 * that each entity has permissions, or if the check is too expensive, then use
    22 * this class.
    16 * Use this class as a route provider for an entity type such as Vocabulary.
  • Brian Suttis added 14 commits

    added 14 commits

    • 0a000930...fbe398b5 - 5 commits from branch project:11.x
    • 3ab92ac8 - commit patch, avoid phpstan error
    • 68c4447c - fix typo
    • 561106dd - don't ignore
    • b435a887 - Fix failing test, use findConfigEntityDependenciesAsEntities
    • 6df5e551 - changed deprecated/removed version numbers
    • 3d0fe009 - Added `E_USER_DEPRECATED`
    • d84e99af - Update test
    • 3eed802b - Add expectDeprecation test
    • be20ad96 - Update comment to remove references to EntityPermissionsRouteProviderWithCheck

    Compare with previous version

  • Loading
  • Loading
  • Loading
  • Loading
  • Loading
  • Loading
  • Loading
  • Loading
  • Loading
  • Loading
  • Please register or sign in to reply
    Loading