Skip to content
Snippets Groups Projects
Verified Commit 8d5b54ef 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
parent 61018bfe
No related branches found
No related tags found
29 merge requests!11131[10.4.x-only-DO-NOT-MERGE]: Issue ##2842525 Ajax attached to Views exposed filter form does not trigger callbacks,!9470[10.3.x-only-DO-NOT-MERGE]: #3331771 Fix file_get_contents(): Passing null to parameter,!8540Issue #3457061: Bootstrap Modal dialog Not closing after 10.3.0 Update,!8528Issue #3456871 by Tim Bozeman: Support NULL services,!8373Issue #3427374 by danflanagan8, Vighneshh: taxonomy_tid ViewsArgumentDefault...,!7526Expose roles in response,!7352Draft: Resolve #3203489 "Set filename as",!3878Removed unused condition head title for views,!3818Issue #2140179: $entity->original gets stale between updates,!3742Issue #3328429: Create item list field formatter for displaying ordered and unordered lists,!3731Claro: role=button on status report items,!3651Issue #3347736: Create new SDC component for Olivero (header-search),!3531Issue #3336994: StringFormatter always displays links to entity even if the user in context does not have access,!3355Issue #3209129: Scrolling problems when adding a block via layout builder,!3154Fixes #2987987 - CSRF token validation broken on routes with optional parameters.,!3133core/modules/system/css/components/hidden.module.css,!2964Issue #2865710 : Dependencies from only one instance of a widget are used in display modes,!2812Issue #3312049: [Followup] Fix Drupal.Commenting.FunctionComment.MissingReturnType returns for NULL,!2794Issue #3100732: Allow specifying `meta` data on JSON:API objects,!2378Issue #2875033: Optimize joins and table selection in SQL entity query implementation,!2062Issue #3246454: Add weekly granularity to views date sort,!1105Issue #3025039: New non translatable field on translatable content throws error,!1073issue #3191727: Focus states on mobile second level navigation items fixed,!10223132456: Fix issue where views instances are emptied before an ajax request is complete,!877Issue #2708101: Default value for link text is not saved,!617Issue #3043725: Provide a Entity Handler for user cancelation,!579Issue #2230909: Simple decimals fail to pass validation,!560Move callback classRemove outside of the loop,!555Issue #3202493
Pipeline #132474 passed
Pipeline: drupal

#132475

    ......@@ -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