Skip to content
Snippets Groups Projects

Issue #3283810: Added permission access to pathauto checkbox

Open Issue #3283810: Added permission access to pathauto checkbox
issue/pathauto-3283810:3283810-access-to-pathauto into 8.x-1.x
3 unresolved threads
Open eduardo morales alberti requested to merge issue/pathauto-3283810:3283810-access-to-pathauto into 8.x-1.x
3 unresolved threads

Closes #3283810

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
tests/src/Functional/PathautoTaxonomyWebTest.php
80 81 $this->assertSession()->pageTextContains($name);
81 82
82 83 // Manually set the term's alias.
83 $manual_alias = '/tags/' . $term->id();
84 $manual_alias = "/tags/" . $term->id();
  • Sascha Grossenbacher
    Sascha Grossenbacher @berdir started a thread on the diff
    • src/PathautoWidget.php
    61 61 '#default_value' => $entity->path->pathauto,
    62 62 '#description' => $description,
    63 63 '#weight' => -1,
    64 '#disabled' => !(\Drupal::currentUser()->hasPermission('update content state pathauto')),
    • Sascha Grossenbacher
      Sascha Grossenbacher @berdir ·
      Developer

      this will need an update function to grant this permission to basically anyone I guess, possibly also need to consider a hook install or if not that then at least documentation updates that granting this permission is now basically required for most use cases.

      Or a setting to respect the permission that is off by default.

      The weight and implications means I'm really not sure if this should be part of this project. If you have this use case then it is easy enough to add in an alter, could be a separate module doing it too.

    • Please register or sign in to reply
  • Sascha Grossenbacher
    Sascha Grossenbacher @berdir started a thread on the diff
    • pathauto.permissions.yml
    4 4 notify of path changes:
    5 5 title: 'Notify of Path Changes'
    6 6 description: 'Determines whether or not users are notified.'
    7 update content state pathauto:
    8 title: 'Update content state pathauto'
    Please register or sign in to reply