MAGETWO-44798: [Varnish4] Page cache is not disabled if Varnish is used

Author: Joan He

app/code/Magento/Backend/Controller/Adminhtml/Cache/MassDisable.php

@@ -26,11 +26,11 @@ public function executeInternal()
             }
             $this->_validateTypes($types);
             foreach ($types as $code) {
+                $this->_cacheTypeList->cleanType($code);
                 if ($this->_cacheState->isEnabled($code)) {
                     $this->_cacheState->setEnabled($code, false);
                     $updatedTypes++;
                 }
-                $this->_cacheTypeList->cleanType($code);
             }
             if ($updatedTypes > 0) {
                 $this->_cacheState->persist();

app/code/Magento/Backend/Controller/Adminhtml/Cache/MassRefresh.php

@@ -27,7 +27,6 @@ public function executeInternal()
             $this->_validateTypes($types);
             foreach ($types as $type) {
                 $this->_cacheTypeList->cleanType($type);
-                $this->_eventManager->dispatch('adminhtml_cache_refresh_type', ['type' => $type]);
                 $updatedTypes++;
             }
             if ($updatedTypes > 0) {

app/code/Magento/PageCache/Model/Cache/Type.php

@@ -19,11 +19,27 @@ class Type extends \Magento\Framework\Cache\Frontend\Decorator\TagScope
      */
     const CACHE_TAG = 'FPC';
 
+    /**
+     * @var \Magento\Framework\Event\ManagerInterface
+     */
+    private $eventManager;
+
     /**
      * @param \Magento\Framework\App\Cache\Type\FrontendPool $cacheFrontendPool
+     * @param \Magento\Framework\Event\ManagerInterface $eventManager
      */
-    public function __construct(\Magento\Framework\App\Cache\Type\FrontendPool $cacheFrontendPool)
-    {
+    public function __construct(
+        \Magento\Framework\App\Cache\Type\FrontendPool $cacheFrontendPool,
+        \Magento\Framework\Event\ManagerInterface $eventManager
+
+    ) {
         parent::__construct($cacheFrontendPool->get(self::TYPE_IDENTIFIER), self::CACHE_TAG);
+        $this->eventManager = $eventManager;
+    }
+
+    public function clean($mode = \Zend_Cache::CLEANING_MODE_ALL, array $tags = [])
+    {
+        $this->eventManager->dispatch('adminhtml_cache_refresh_type');
+        return parent::clean($mode, $tags);
     }
 }

app/code/Magento/PageCache/Test/Unit/Model/Cache/TypeTest.php

@@ -0,0 +1,51 @@
+<?php
+/**
+ * Copyright © 2015 Magento. All rights reserved.
+ * See COPYING.txt for license details.
+ */
+namespace Magento\PageCache\Test\Unit\Model\Cache;
+
+class TypeTest extends \PHPUnit_Framework_TestCase
+{
+    /** @var \Magento\PageCache\Model\Cache\Type */
+    protected $model;
+
+    /** @var \PHPUnit_Framework_MockObject_MockObject | \Magento\Framework\Event\ManagerInterface */
+    protected $eventManagerMock;
+
+    /** @var \PHPUnit_Framework_MockObject_MockObject | \Magento\Framework\App\Cache\Type\FrontendPool */
+    protected $cacheFrontendPoolMock;
+
+    public function setUp()
+    {
+        $this->eventManagerMock = $this->getMockBuilder('Magento\Framework\Event\ManagerInterface')
+            ->disableOriginalConstructor()
+            ->getMock();
+        $this->cacheFrontendPoolMock = $this->getMockBuilder('Magento\Framework\App\Cache\Type\FrontendPool')
+            ->disableOriginalConstructor()
+            ->getMock();
+        $cacheFrontend = $this->getMockBuilder('Magento\Framework\Cache\FrontendInterface')
+            ->disableOriginalConstructor()
+            ->getMock();
+        $this->cacheFrontendPoolMock->expects($this->once())
+            ->method('get')
+            ->willReturn($cacheFrontend);
+        $objectManager = new \Magento\Framework\TestFramework\Unit\Helper\ObjectManager($this);
+        $this->model = $objectManager->getObject(
+            'Magento\PageCache\Model\Cache\Type',
+            [
+                'eventManager' => $this->eventManagerMock,
+                'cacheFrontendPool' => $this->cacheFrontendPoolMock,
+            ]
+        );
+    }
+
+    public function testClean()
+    {
+        $this->eventManagerMock->expects($this->once())
+            ->method('dispatch')
+            ->with('adminhtml_cache_refresh_type');
+
+        $this->model->clean();
+    }
+}