Removing TeamInvitationQueryAccessSubscriber for AppGroup member edit/add (#1034)

Author: shishir-intelli

modules/apigee_edge_teams/apigee_edge_teams.services.yml

@@ -117,7 +117,7 @@ services:
 
   apigee_edge_teams.team_invitation_query_access_subscriber:
     class: Drupal\apigee_edge_teams\EventSubscriber\TeamInvitationQueryAccessSubscriber
-    arguments: ['@entity_type.manager']
+    arguments: ['@apigee_edge.controller.organization', '@entity_type.manager']
     tags:
       - { name: event_subscriber }
 

modules/apigee_edge_teams/src/EventSubscriber/TeamInvitationQueryAccessSubscriber.php

@@ -20,6 +20,7 @@
 
 namespace Drupal\apigee_edge_teams\EventSubscriber;
 
+use Drupal\apigee_edge\Entity\Controller\OrganizationControllerInterface;
 use Drupal\Core\Entity\EntityTypeManagerInterface;
 use Drupal\entity\QueryAccess\QueryAccessEvent;
 use Symfony\Component\EventDispatcher\EventSubscriberInterface;
@@ -29,6 +30,13 @@
  */
 class TeamInvitationQueryAccessSubscriber implements EventSubscriberInterface {
 
+  /**
+   * The organization controller service.
+   *
+   * @var \Drupal\apigee_edge\Entity\Controller\OrganizationControllerInterface
+   */
+  private $orgController;
+
   /**
    * The entity type manager service.
    *
@@ -39,10 +47,13 @@ class TeamInvitationQueryAccessSubscriber implements EventSubscriberInterface {
   /**
    * TeamInvitationQueryAccessSubscriber constructor.
    *
+   * @param \Drupal\apigee_edge\Entity\Controller\OrganizationControllerInterface $org_controller
+   *   The organization controller service.
    * @param \Drupal\Core\Entity\EntityTypeManagerInterface $entity_type_manager
    *   The entity type manager service.
    */
-  public function __construct(EntityTypeManagerInterface $entity_type_manager) {
+  public function __construct(OrganizationControllerInterface $org_controller, EntityTypeManagerInterface $entity_type_manager) {
+    $this->orgController = $org_controller;
     $this->entityTypeManager = $entity_type_manager;
   }
 
@@ -62,6 +73,10 @@ public static function getSubscribedEvents() {
    *   The event.
    */
   public function onQueryAccess(QueryAccessEvent $event) {
+    // AppGroup members information is stored in Database tables.
+    if ($this->orgController->isOrganizationApigeeX()) {
+      return;
+    }
     // Add a condition to check for a valid team.
     // We query team from storage instead of check for a null team field because
     // the team might have been deleted on the remote server.

modules/apigee_edge_teams/tests/src/Functional/ApigeeX/TeamInvitationsTest.php

@@ -159,20 +159,13 @@ public function testMultipleInvitations() {
       $this->teamB->decorated(),
     ];
 
-    $inCache = FALSE;
     foreach ($teams as $team) {
-      if (!$inCache) {
-        $this->queueAppGroupResponse($team->decorated());
-      }
+      $this->queueAppGroupResponse($team->decorated());
       $this->drupalGet(Url::fromRoute('entity.team.add_members', [
         'team' => $team->id(),
       ]));
 
       $this->assertSession()->pageTextContains('Invite members');
-
-      $this->queueAppGroupsResponse($appgroups);
-      $this->queueAppGroupsResponse($appgroups);
-      $this->queueDevsInCompanyResponse([]);
       $this->submitForm([
         'developers' => $this->accountUser->getEmail(),
       ], 'Invite members');
@@ -184,7 +177,6 @@ public function testMultipleInvitations() {
       ]);
 
       $this->assertSession()->pageTextContains($successMessage);
-      $inCache = TRUE;
     }
   }