Merge pull request #8 from walterdolce/feature/fix-warning-when-attempting-file-removal

Jon Acker · Jon Acker · commit b4d6b893a6fd · 2015-11-24T15:17:13.000Z
Prevent PHP Warning when attempting to delete container cache file
diff --git a/app/code/community/Inviqa/SymfonyContainer/Model/Observer.php b/app/code/community/Inviqa/SymfonyContainer/Model/Observer.php
@@ -12,7 +12,10 @@ class Inviqa_SymfonyContainer_Model_Observer
     public function onCacheRefresh(Varien_Event_Observer $event)
     {
         if (ConfigurationBuilder::MODEL_ALIAS === $event->getType()) {
-            unlink(Mage::getBaseDir('cache') . '/' . ConfigurationBuilder::CACHED_CONTAINER);
+            $filePath = Mage::getBaseDir('cache') . '/' . ConfigurationBuilder::CACHED_CONTAINER;
+            if (file_exists($filePath)) {
+                unlink($filePath);
+            }
         }
     }
     public function onPreDispatch(Varien_Event_Observer $event)

Original file line number	Diff line number	Diff line change
`@@ -12,7 +12,10 @@ class Inviqa_SymfonyContainer_Model_Observer`
`12`	`12`	`public function onCacheRefresh(Varien_Event_Observer $event)`
`13`	`13`	`{`
`14`	`14`	`if (ConfigurationBuilder::MODEL_ALIAS === $event->getType()) {`
`15`		`- unlink(Mage::getBaseDir('cache') . '/' . ConfigurationBuilder::CACHED_CONTAINER);`
	`15`	`+ $filePath = Mage::getBaseDir('cache') . '/' . ConfigurationBuilder::CACHED_CONTAINER;`
	`16`	`+ if (file_exists($filePath)) {`
	`17`	`+ unlink($filePath);`
	`18`	`+ }`
`16`	`19`	`}`
`17`	`20`	`}`
`18`	`21`	`public function onPreDispatch(Varien_Event_Observer $event)`