Moved default CacheAdvice from Porter -> ImportSpecification.

Author: Bilge

src/Porter.php

@@ -35,11 +35,6 @@ class Porter
      */
     private $providerFactory;
 
-    /**
-     * @var CacheAdvice
-     */
-    private $defaultCacheAdvice;
-
     /**
      * @var int
      */
@@ -50,11 +45,6 @@ class Porter
      */
     private $fetchExceptionHandler;
 
-    public function __construct()
-    {
-        $this->defaultCacheAdvice = CacheAdvice::SHOULD_NOT_CACHE();
-    }
-
     /**
      * Imports data according to the design of the specified import specification.
      *
@@ -108,7 +98,7 @@ private function fetch(ProviderResource $resource, CacheAdvice $cacheAdvice = nu
     {
         $provider = $this->getProvider($resource->getProviderClassName(), $resource->getProviderTag());
 
-        $this->applyCacheAdvice($provider, $cacheAdvice ?: $this->defaultCacheAdvice);
+        $this->applyCacheAdvice($provider, $cacheAdvice);
 
         if (($records = \ScriptFUSION\Retry\retry(
             $this->getMaxFetchAttempts(),

src/Specification/ImportSpecification.php

@@ -26,14 +26,21 @@ class ImportSpecification
     private $context;
 
     /**
-     * @var CacheAdvice
+     * @var CacheAdvice|null
      */
     private $cacheAdvice;
 
+    /**
+     * @var CacheAdvice
+     */
+    private $defaultCacheAdvice;
+
     public function __construct(ProviderResource $resource)
     {
         $this->resource = $resource;
+
         $this->clearTransformers();
+        $this->defaultCacheAdvice = CacheAdvice::SHOULD_NOT_CACHE();
     }
 
     public function __clone()
@@ -141,7 +148,7 @@ final public function setContext($context)
      */
     final public function getCacheAdvice()
     {
-        return $this->cacheAdvice;
+        return $this->cacheAdvice ?: $this->defaultCacheAdvice;
     }
 
     /**