Commit 5afd638b authored by Gábor Hojtsy's avatar Gábor Hojtsy Committed by Chris Wells
Browse files

Issue #3258247 by Gábor Hojtsy, chrisfromredfin, gaurav.kapoor, hmendes: Add... 

Issue #3258247 by Gábor Hojtsy, chrisfromredfin, gaurav.kapoor, hmendes: Add permissions to the project browser route
parent 3863467e

project_browser.routing.yml

+3 −12
Changes for project_browser.routing.yml: 3 added lines, 12 removed lines.
Original line number Diff line number Diff line
@@ -5,10 +5,7 @@ project_browser.api_project_get_all: 
    _controller: '\Drupal\project_browser\Controller\DrupalOrgProxyController::getAllProjects'

    _title: 'Get all projects'

  requirements:

    #_permission: 'manage own subscriptions+administer subscriptions'

    #_user_is_logged_in: 'TRUE'

    # @todo Undo this access bypass!

    _access: 'TRUE'

    _permission: 'administer modules'

  #options:

  #  no_cache: 'TRUE'

project_browser.api_project_get_releases:
@@ -18,10 +15,7 @@ project_browser.api_project_get_releases: 
    _controller: '\Drupal\project_browser\Controller\DrupalOrgProxyController::getProjectReleases'

    _title: 'Get all project releases'

  requirements:

    #_permission: 'manage own subscriptions+administer subscriptions'

    #_user_is_logged_in: 'TRUE'

    # @todo Undo this access bypass!

    _access: 'TRUE'

    _permission: 'administer modules'

  #options:

  #  no_cache: 'TRUE'

project_browser.browse:
@@ -30,7 +24,4 @@ project_browser.browse: 
    _controller: '\Drupal\project_browser\Controller\BrowserController::browse'

    _title: 'Browse Drupal.org projects'

  requirements:

    #_permission: 'manage own subscriptions+administer subscriptions'

    #_user_is_logged_in: 'TRUE'

    # @todo Undo this access bypass!

    _access: 'TRUE'
 
    _permission: 'administer modules'

tests/src/FunctionalJavascript/ProjectBrowserUiTest.php

+1 −1
Changes for tests/src/FunctionalJavascript/ProjectBrowserUiTest.php: 1 added line, 1 removed line.
Original line number Diff line number Diff line
@@ -30,7 +30,7 @@ class ProjectBrowserUiTest extends WebDriverTestBase { 
    parent::setUp();



    $this->drupalLogin($this->drupalCreateUser([

      // @todo Update once project_browser.browse has requirements.

      'administer modules'

    ]));

  }