Improve code readability/understandability by separating flush event cache cleans to separate method

Author: Zsolt Gál

app/code/community/Inviqa/SymfonyContainer/Model/Observer.php

@@ -12,12 +12,16 @@ class Inviqa_SymfonyContainer_Model_Observer
 
     public function onCacheRefresh(Varien_Event_Observer $event)
     {
-        $eventType = $event->getType();
-        if ($eventType === ConfigurationBuilder::MODEL_ALIAS || is_null($eventType)) {
+        if ($event->getType() === ConfigurationBuilder::MODEL_ALIAS) {
             $this->clearCache();
         }
     }
 
+    public function onCacheFlush()
+    {
+        $this->clearCache();
+    }
+
     public function onPreDispatch(Varien_Event_Observer $event)
     {
         $controller = $event->getControllerAction();

app/code/community/Inviqa/SymfonyContainer/etc/config.xml

@@ -27,7 +27,7 @@
                     <inviqa_symfonyContainer>
                         <class>Inviqa_SymfonyContainer_Model_Observer</class>
                         <type>singleton</type>
-                        <method>onCacheRefresh</method>
+                        <method>onCacheFlush</method>
                     </inviqa_symfonyContainer>
                 </observers>
             </adminhtml_cache_flush_all>
@@ -36,7 +36,7 @@
                     <inviqa_symfonyContainer>
                         <class>Inviqa_SymfonyContainer_Model_Observer</class>
                         <type>singleton</type>
-                        <method>onCacheRefresh</method>
+                        <method>onCacheFlush</method>
                     </inviqa_symfonyContainer>
                 </observers>
             </adminhtml_cache_flush_system>