Code cleanup

Signed-off-by: Andrew Welch <andrew@nystudio107.com>
nystudio107 · Jul 30, 2018 · b0c4e6b · b0c4e6b
1 parent df324c5
commit b0c4e6b
Show file tree

Hide file tree

Showing 4 changed files with 21 additions and 16 deletions.
diff --git a/src/EagerBeaver.php b/src/EagerBeaver.php
@@ -2,7 +2,8 @@
 /**
  * Eager Beaver plugin for Craft CMS 3.x
  *
- * Allows you to eager load elements from auto-injected Entry elements on demand from your templates.
+ * Allows you to eager load elements from auto-injected Entry elements on
+ * demand from your templates.
  *
  * @link      https://nystudio107.com
  * @copyright Copyright (c) 2017 nystudio107
@@ -73,8 +74,4 @@ function (Event $event) {
             __METHOD__
         );
     }
-
-    // Protected Methods
-    // =========================================================================
-
 }
diff --git a/src/services/EagerBeaverService.php b/src/services/EagerBeaverService.php
@@ -43,17 +43,17 @@ class EagerBeaverService extends Component
     public function eagerLoadElements($elements, $with)
     {
         // Bail if there aren't even any elements
-        if (!$elements || empty($elements)) {
+        if (empty($elements)) {
             return;
         }
 
-        if (!is_array($elements)) {
+        if (!\is_array($elements)) {
             $elements = [$elements];
         }
         // We are assuming all of these elements are of the same type
         /** @var Element $element */
         $element = $elements[0];
-        $elementType = get_class($element);
+        $elementType = \get_class($element);
         Craft::$app->elements->eagerLoadElements($elementType, $elements, $with);
     }
 }
diff --git a/src/twigextensions/EagerBeaverTwigExtension.php b/src/twigextensions/EagerBeaverTwigExtension.php
@@ -2,7 +2,8 @@
 /**
  * Eager Beaver plugin for Craft CMS 3.x
  *
- * Allows you to eager load elements from auto-injected Entry elements on demand from your templates.
+ * Allows you to eager load elements from auto-injected Entry elements on
+ * demand from your templates.
  *
  * @link      https://nystudio107.com
  * @copyright Copyright (c) 2017 nystudio107
@@ -12,7 +13,6 @@
 
 use nystudio107\eagerbeaver\EagerBeaver;
 
-use Craft;
 use craft\base\ElementInterface;
 
 /**
@@ -48,8 +48,12 @@ public function getFunctions()
     /**
      * Eager-loads additional elements onto a given set of elements.
      *
-     * @param ElementInterface[] $elements    The root element models that should be updated with the eager-loaded elements
-     * @param string|array       $with        Dot-delimited paths of the elements that should be eager-loaded into the root elements
+     * @param ElementInterface[] $elements The root element models that should
+     *                                     be updated with the eager-loaded
+     *                                     elements
+     * @param string|array       $with     Dot-delimited paths of the elements
+     *                                     that should be eager-loaded into the
+     *                                     root elements
      *
      * @return void
      */

diff --git a/src/variables/EagerBeaverVariable.php b/src/variables/EagerBeaverVariable.php
@@ -2,7 +2,8 @@
 /**
  * Eager Beaver plugin for Craft CMS 3.x
  *
- * Allows you to eager load elements from auto-injected Entry elements on demand from your templates.
+ * Allows you to eager load elements from auto-injected Entry elements on
+ * demand from your templates.
  *
  * @link      https://nystudio107.com
  * @copyright Copyright (c) 2017 nystudio107
@@ -12,7 +13,6 @@
 
 use nystudio107\eagerbeaver\EagerBeaver;
 
-use Craft;
 use craft\base\ElementInterface;
 
 /**
@@ -32,8 +32,12 @@ class EagerBeaverVariable
     /**
      * Eager-loads additional elements onto a given set of elements.
      *
-     * @param ElementInterface[] $elements    The root element models that should be updated with the eager-loaded elements
-     * @param string|array       $with        Dot-delimited paths of the elements that should be eager-loaded into the root elements
+     * @param ElementInterface[] $elements The root element models that should
+     *                                     be updated with the eager-loaded
+     *                                     elements
+     * @param string|array       $with     Dot-delimited paths of the elements
+     *                                     that should be eager-loaded into the
+     *                                     root elements
      *
      * @return void
      */