Skip to content
Snippets Groups Projects
Verified Commit 495174e5 authored by Alex Pott's avatar Alex Pott
Browse files

Issue #3114365 by mohit_aghera, david.muffley, danflanagan8, Greg__, wsantell,...

Issue #3114365 by mohit_aghera, david.muffley, danflanagan8, Greg__, wsantell, Lendude, larowlan, xjm: Vocabulary name not shown in View for Anonymous Users

(cherry picked from commit 8d5b54ef)
parent 7686fb5a
No related branches found
No related tags found
25 merge requests!11769Issue #3517987: Add option to contextual filters to encode slashes in query parameter.,!11185Issue #3477324 by andypost, alexpott: Fix usage of str_getcsv() and fgetcsv() for PHP 8.4,!10602Issue #3438769 by vinmayiswamy, antonnavi, michelle, amateescu: Sub workspace does not clear,!10301Issue #3469309 by mstrelan, smustgrave, moshe weitzman: Use one-time login...,!10187Issue #3487488 by dakwamine: ExtensionMimeTypeGuesser::guessMimeType must support file names with "0" (zero) like foo.0.zip,!9944Issue #3483353: Consider making the createCopy config action optionally fail...,!9929Issue #3445469 by pooja_sharma, smustgrave: Add additional test coverage for...,!9787Resolve issue 3479427 - bootstrap barrio issue under Windows,!9742Issue #3463908 by catch, quietone: Split OptionsFieldUiTest into two,!9526Issue #3458177 by mondrake, catch, quietone, godotislate, longwave, larowlan,...,!8738Issue #3424162 by camilledavis, dineshkumarbollu, smustgrave: Claro...,!8704Make greek characters available in ckeditor5,!8597Draft: Issue #3442259 by catch, quietone, dww: Reduce time of Migrate Upgrade tests...,!8533Issue #3446962 by kim.pepper: Remove incorrectly added...,!8517Issue #3443748 by NexusNovaz, smustgrave: Testcase creates false positive,!8325Update file Sort.php,!8095Expose document root on install,!7930Resolve #3427374 "Taxonomytid viewsargumentdefault plugin",!7627Issue #3439440 by nicxvan, Binoli Lalani, longwave: Remove country support from DateFormatter,!7445Issue #3440169: When using drupalGet(), provide an associative array for $headers,!7401#3271894 Fix documented StreamWrapperInterface return types for realpath() and dirname(),!7384Add constraints to system.advisories,!6502Draft: Resolve #2938524 "Plach testing issue",!38582585169-10.1.x,!3226Issue #2987537: Custom menu link entity type should not declare "bundle" entity key
Pipeline #132433 passed with warnings
Pipeline: drupal

#132460

    Pipeline: drupal

    #132455

      Pipeline: drupal

      #132448

        +1
        ......@@ -69,7 +69,7 @@ class TermTest extends ResourceTestBase {
        protected function setUpAuthorization($method) {
        switch ($method) {
        case 'GET':
        $this->grantPermissionsToTestedRole(['access content']);
        $this->grantPermissionsToTestedRole(['access content', 'view vocabulary labels']);
        break;
        case 'POST':
        ......
        ......@@ -14,11 +14,23 @@
        */
        class VocabularyAccessControlHandler extends EntityAccessControlHandler {
        /**
        * {@inheritdoc}
        */
        protected $viewLabelOperation = TRUE;
        /**
        * {@inheritdoc}
        */
        protected function checkAccess(EntityInterface $entity, $operation, AccountInterface $account) {
        switch ($operation) {
        case 'view label':
        return AccessResult::allowedIfHasPermissions($account, [
        'view vocabulary labels',
        'access taxonomy overview',
        'administer taxonomy',
        ], 'OR');
        case 'access taxonomy overview':
        case 'view':
        return AccessResult::allowedIfHasPermissions($account, ['access taxonomy overview', 'administer taxonomy'], 'OR');
        ......
        ......@@ -14,5 +14,8 @@ delete all taxonomy revisions:
        view all taxonomy revisions:
        title: 'View all term revisions'
        view vocabulary labels:
        title: 'View vocabulary labels'
        permission_callbacks:
        - Drupal\taxonomy\TaxonomyPermissions::permissions
        ......@@ -4,6 +4,7 @@
        use Drupal\Core\Render\RenderContext;
        use Drupal\Tests\taxonomy\Traits\TaxonomyTestTrait;
        use Drupal\Tests\user\Traits\UserCreationTrait;
        use Drupal\Tests\views\Kernel\ViewsKernelTestBase;
        use Drupal\user\Entity\User;
        use Drupal\views\Tests\ViewTestData;
        ......@@ -18,6 +19,7 @@
        class TaxonomyFieldVidTest extends ViewsKernelTestBase {
        use TaxonomyTestTrait;
        use UserCreationTrait;
        /**
        * Modules to enable.
        ......@@ -116,6 +118,34 @@ public function testViewsHandlerVidField() {
        $this->assertEquals($expected, $actual, 'Displayed vocabulary name should match that loaded from the term.');
        $this->assertEquals('bbb', $vocabulary->id(), 'First result should be vocabulary "bbb", due to DESC sorting.');
        // Test with user without 'view vocabulary labels' permission.
        $this->setUpCurrentUser();
        $actual = $renderer->executeInRenderContext(new RenderContext(), function () use ($view) {
        return $view->field['vid']->advancedRender($view->result[0]);
        });
        $expected = '';
        $this->assertEquals($expected, $actual);
        // Test with user with 'view vocabulary labels' permissions.
        $this->setUpCurrentUser([], ['view vocabulary labels']);
        $actual = $renderer->executeInRenderContext(new RenderContext(), function () use ($view) {
        return $view->field['vid']->advancedRender($view->result[0]);
        });
        $expected = $vocabulary->label();
        $this->assertEquals($expected, $actual);
        // Test with user with 'administer taxonomy' and 'access taxonomy overview'
        // permissions. Label should be displayed for either permission.
        $this->setUpCurrentUser([], [
        'administer taxonomy',
        'access taxonomy overview',
        ]);
        $actual = $renderer->executeInRenderContext(new RenderContext(), function () use ($view) {
        return $view->field['vid']->advancedRender($view->result[0]);
        });
        $expected = $vocabulary->label();
        $this->assertEquals($expected, $actual);
        }
        }
        0% Loading or .
        You are about to add 0 people to the discussion. Proceed with caution.
        Finish editing this message first!
        Please register or to comment