quick phpcs

adrianodias8 · Mar 31, 2020 · eeacbf5 · eeacbf5
1 parent 43ce59e
commit eeacbf5
Show file tree

Hide file tree

Showing 5 changed files with 9 additions and 18 deletions.
diff --git a/modules/entity_activity_tracker_group/src/Plugin/CreditGroupBase.php b/modules/entity_activity_tracker_group/src/Plugin/CreditGroupBase.php
@@ -71,7 +71,7 @@ protected function getRelatedEntity(ContentEntityInterface $entity) {
         if (isset($this->pluginDefinition['credit_related'])) {
           if ($this->pluginDefinition['credit_related'] == 'group') {
             $node = $entity->getCommentedEntity();
-            if (empty($node)){
+            if (empty($node)) {
               return FALSE;
             }
             $group_contents = $this->entityTypeManager->getStorage('group_content')->loadByEntity($node);
@@ -84,7 +84,7 @@ protected function getRelatedEntity(ContentEntityInterface $entity) {
           }
           if ($this->pluginDefinition['credit_related'] == 'group_content') {
             $node = $entity->getCommentedEntity();
-            if (empty($node)){
+            if (empty($node)) {
               return FALSE;
             }
             $group_contents = $this->entityTypeManager->getStorage('group_content')->loadByEntity($node);
@@ -123,8 +123,8 @@ protected function getGroup(GroupContentInterface $group_content) {
     $group = $group_content->getGroup();
 
     // Prevent further execution if no group was found.
-    if (empty($group)){
-      \Drupal::logger('entity_activity_tracker')->error($this->t('Couldn\'t find Group!'));
+    if (empty($group)) {
+      \Drupal::logger('entity_activity_tracker')->error($this->t("Couldn't find Group!"));
       return FALSE;
     }
 

diff --git a/modules/entity_activity_tracker_group/src/Plugin/CreditGroupMembershipBase.php b/modules/entity_activity_tracker_group/src/Plugin/CreditGroupMembershipBase.php
@@ -20,9 +20,9 @@ protected function getRelatedEntity(ContentEntityInterface $entity) {
       $group = $this->getGroup($group_content_related_entity);
 
       // Prevent further execution if no group was found.
-      if (empty($group)){
+      if (empty($group)) {
 
-        \Drupal::logger('entity_activity_tracker')->error($this->t('Couldn\'t find Group!'));
+        \Drupal::logger('entity_activity_tracker')->error($this->t("Couldn't find Group!"));
         return FALSE;
       }
 

diff --git a/src/Plugin/ActivityProcessor/EntityCreate.php b/src/Plugin/ActivityProcessor/EntityCreate.php
@@ -170,7 +170,7 @@ public function canProcess(Event $event) {
     // load it again.
     $entity = $event->getEntity();
     $exists = $this->entityTypeManager->getStorage($entity->getEntityTypeId())->load($entity->id());
-    if (empty($exists) && $event->getDispatcherType() == ActivityEventInterface::ENTITY_INSERT){
+    if (empty($exists) && $event->getDispatcherType() == ActivityEventInterface::ENTITY_INSERT) {
       return ActivityProcessorInterface::SKIP;
     }
     return ActivityProcessorInterface::PROCESS;

diff --git a/src/Plugin/ActivityProcessor/EntityEdit.php b/src/Plugin/ActivityProcessor/EntityEdit.php
@@ -117,7 +117,7 @@ public function canProcess(Event $event) {
     // load it again.
     $entity = $event->getEntity();
     $exists = $this->entityTypeManager->getStorage($entity->getEntityTypeId())->load($entity->id());
-    if (empty($exists)){
+    if (empty($exists)) {
       return ActivityProcessorInterface::SKIP;
     }
     return ActivityProcessorInterface::PROCESS;

diff --git a/src/Plugin/ActivityProcessorCreditRelatedBase.php b/src/Plugin/ActivityProcessorCreditRelatedBase.php
@@ -3,7 +3,6 @@
 namespace Drupal\entity_activity_tracker\Plugin;
 
 use Drupal\Core\Entity\ContentEntityInterface;
-use Drupal\entity_activity_tracker\ActivityRecord;
 use Drupal\entity_activity_tracker\Event\ActivityEventInterface;
 use Drupal\entity_activity_tracker\Event\EntityActivityInsertEvent;
 use Symfony\Component\EventDispatcher\Event;
@@ -50,13 +49,6 @@ public function processActivity(Event $event) {
         }
         break;
 
-//      Do nothing for related entities regarding the delete operation.
-//      case ActivityEventInterface::ENTITY_DELETE:
-//        /** @var \Drupal\entity_activity_tracker\ActivityRecord $activity_record */
-//        $activity_record = $this->activityRecordStorage->getActivityRecordByEntity($event->getEntity());
-//        $this->activityRecordStorage->deleteActivityRecord($activity_record);
-//        break;
-
       case ActivityEventInterface::TRACKER_DELETE:
         $tracker = $event->getTracker();
         // Get ActivityRecords from this tracker.
@@ -125,7 +117,6 @@ public function canProcess(Event $event) {
 
       default:
         return ActivityProcessorInterface::PROCESS;
-        break;
     }
   }
 
@@ -145,7 +136,7 @@ protected function getRelatedEntity(ContentEntityInterface $entity) {
           }
           if ($this->pluginDefinition['credit_related'] == 'user') {
             $user = $entity->getOwner();
-            // prevent schedule for anonymous users.
+            // Prevent schedule for anonymous users.
             return $user->id() != 0 ? $user : FALSE;
           }
         }