From ebdb649bd7728649ebf2bfcc60a3da9838d59537 Mon Sep 17 00:00:00 2001
From: Taras Machyshyn <taras@letrium.com>
Date: Tue, 17 Feb 2015 11:02:01 +0200
Subject: [PATCH 1/8] Improved upgrade/install extenions feature

---
 application/Espo/Core/ExtensionManager.php    |  1 -
 application/Espo/Core/UpgradeManager.php      |  1 +
 .../Espo/Core/Upgrades/Actions/Base.php       | 33 ++++++++-
 .../Core/Upgrades/Actions/Base/Install.php    | 42 +++++++----
 .../Core/Upgrades/Actions/Base/Uninstall.php  | 65 ++++++++++++-----
 .../Upgrades/Actions/Extension/Install.php    | 22 +-----
 .../Upgrades/Actions/Extension/Uninstall.php  | 10 +++
 .../Core/Upgrades/Actions/Upgrade/Install.php | 11 +--
 application/Espo/Core/Utils/File/Manager.php  | 70 +++++++++++++++++++
 9 files changed, 196 insertions(+), 59 deletions(-)

diff --git a/application/Espo/Core/ExtensionManager.php b/application/Espo/Core/ExtensionManager.php
index f3c35baf7a..8613b3a958 100644
--- a/application/Espo/Core/ExtensionManager.php
+++ b/application/Espo/Core/ExtensionManager.php
@@ -28,7 +28,6 @@ class ExtensionManager extends Upgrades\Base
 
     protected $params = array(
         'packagePath' => 'data/upload/extensions',
-
         'backupPath' => 'data/.backup/extensions',
 
         'scriptNames' => array(
diff --git a/application/Espo/Core/UpgradeManager.php b/application/Espo/Core/UpgradeManager.php
index adae1d87f3..3e805b5b52 100644
--- a/application/Espo/Core/UpgradeManager.php
+++ b/application/Espo/Core/UpgradeManager.php
@@ -30,6 +30,7 @@ class UpgradeManager extends Upgrades\Base
 
     protected $params = array(
         'packagePath' => 'data/upload/upgrades',
+        'backupPath' => 'data/.backup/upgrades',
 
         'scriptNames' => array(
             'before' => 'BeforeUpgrade',
diff --git a/application/Espo/Core/Upgrades/Actions/Base.php b/application/Espo/Core/Upgrades/Actions/Base.php
index 52e7f7ad0a..2b0f530910 100644
--- a/application/Espo/Core/Upgrades/Actions/Base.php
+++ b/application/Espo/Core/Upgrades/Actions/Base.php
@@ -304,7 +304,7 @@ abstract class Base
     /**
      * Get a list of files defined in manifest.json
      *
-     * @return [type] [description]
+     * @return array
      */
     protected function getDeleteFileList()
     {
@@ -352,6 +352,18 @@ abstract class Base
         return $this->data['fileList'];
     }
 
+    protected function getRestoreFileList()
+    {
+        if (!isset($this->data['restoreFileList'])) {
+            $backupPath = $this->getPath('backupPath');
+            $backupFilePath = Util::concatPath($backupPath, self::FILES);
+
+            $this->data['restoreFileList'] = $this->getFileManager()->getFileList($backupFilePath, true, '', true, true);
+        }
+
+        return $this->data['restoreFileList'];
+    }
+
     protected function copy($sourcePath, $destPath, $recursively = false, array $fileList = null, $copyOnlyFiles = false)
     {
         try {
@@ -506,4 +518,23 @@ abstract class Base
     {
         return $this->getContainer()->get('dataManager')->clearCache();
     }
+
+    protected function checkIsWritable()
+    {
+        $fullFileList = array_merge($this->getDeleteFileList(), $this->getCopyFileList());
+
+        $result = $this->getFileManager()->isWritableList($fullFileList);
+        if (!$result) {
+            $permissionDeniedList = $this->getFileManager()->getLastPermissionDeniedList();
+            throw new Error("Permission denied in <br>". implode(", <br>", $permissionDeniedList));
+        }
+    }
+
+    protected function backupExistingFiles()
+    {
+        $fullFileList = array_merge($this->getDeleteFileList(), $this->getCopyFileList());
+
+        $backupPath = $this->getPath('backupPath');
+        return $this->copy('', array($backupPath, self::FILES), false, $fullFileList);
+    }
 }
\ No newline at end of file
diff --git a/application/Espo/Core/Upgrades/Actions/Base/Install.php b/application/Espo/Core/Upgrades/Actions/Base/Install.php
index 7a0f9dd93a..4d27ba9b87 100644
--- a/application/Espo/Core/Upgrades/Actions/Base/Install.php
+++ b/application/Espo/Core/Upgrades/Actions/Base/Install.php
@@ -23,6 +23,7 @@
 namespace Espo\Core\Upgrades\Actions\Base;
 
 use Espo\Core\Exceptions\Error;
+use Espo\Core\Utils\Util;
 
 class Install extends \Espo\Core\Upgrades\Actions\Base
 {
@@ -58,15 +59,15 @@ class Install extends \Espo\Core\Upgrades\Actions\Base
             $this->isAcceptable();
         }
 
-        $this->beforeRunAction();
+        //check permissions copied and deleted files
+        $this->checkIsWritable();
+
+        $this->backupExistingFiles();
 
         /* run before install script */
         $this->runScript('before');
 
-        /* remove files defined in a manifest */
-        if (!$this->deleteFiles()) {
-            $this->throwErrorAndRemovePackage('Permission denied to delete files.');
-        }
+        $this->beforeRunAction();
 
         /* copy files from directory "Files" to EspoCRM files */
         if (!$this->copyFiles()) {
@@ -74,15 +75,18 @@ class Install extends \Espo\Core\Upgrades\Actions\Base
         }
         $this->isCopied = true;
 
+        /* remove files defined in a manifest */
+        $this->deleteFiles();
+
         if (!$this->systemRebuild()) {
             $this->throwErrorAndRemovePackage('Error occurred while EspoCRM rebuild.');
         }
 
+        $this->afterRunAction();
+
         /* run before install script */
         $this->runScript('after');
 
-        $this->afterRunAction();
-
         $this->clearCache();
 
         /* delete unziped files */
@@ -93,15 +97,25 @@ class Install extends \Espo\Core\Upgrades\Actions\Base
 
     protected function restoreFiles()
     {
-        $backupPath = $this->getPath('backupPath');
-
-        $res = true;
-        if ($this->isCopied) {
-            $res &= $this->copy(array($backupPath, self::FILES), '', true);
-            $GLOBALS['log']->info('Restore: copy back');
+        if (!$this->isCopied) {
+            return;
         }
 
-        $res &= $this->getFileManager()->removeInDir($backupPath, true);
+        $GLOBALS['log']->info('Installer: Restore previous files.');
+
+        $backupPath = $this->getPath('backupPath');
+        $backupFilePath = Util::concatPath($backupPath, self::FILES);
+
+        $backupFileList = $this->getRestoreFileList();
+        $copyFileList = $this->getCopyFileList();
+        $deleteFileList = array_diff($copyFileList, $backupFileList);
+
+        $res = $this->copy($backupFilePath, '', true);
+        $res &= $this->getFileManager()->remove($deleteFileList, null, true);
+
+        if ($res) {
+            $this->getFileManager()->removeInDir($backupPath, true);
+        }
 
         return $res;
     }
diff --git a/application/Espo/Core/Upgrades/Actions/Base/Uninstall.php b/application/Espo/Core/Upgrades/Actions/Base/Uninstall.php
index 5c84377eb8..722e0e9287 100644
--- a/application/Espo/Core/Upgrades/Actions/Base/Uninstall.php
+++ b/application/Espo/Core/Upgrades/Actions/Base/Uninstall.php
@@ -37,34 +37,36 @@ class Uninstall extends \Espo\Core\Upgrades\Actions\Base
 
         $this->setProcessId($processId);
 
-        $this->beforeRunAction();
+        $this->checkIsWritable();
 
         /* run before install script */
         $this->runScript('beforeUninstall');
 
+        $this->beforeRunAction();
+
         $backupPath = $this->getPath('backupPath');
         if (file_exists($backupPath)) {
 
-            /* remove extension files, saved in fileList */
-            if (!$this->deleteFiles(true)) {
-                throw new Error('Permission denied to delete files.');
-            }
-
             /* copy core files */
             if (!$this->copyFiles()) {
-                throw new Error('Cannot copy files.');
+                throw new $this->throwErrorAndRemovePackage('Cannot copy files.');
+            }
+
+            /* remove extension files, saved in fileList */
+            if (!$this->deleteFiles(true)) {
+                throw new $this->throwErrorAndRemovePackage('Permission denied to delete files.');
             }
         }
 
         if (!$this->systemRebuild()) {
-            throw new Error('Error occurred while EspoCRM rebuild.');
+            throw new $this->throwErrorAndRemovePackage('Error occurred while EspoCRM rebuild.');
         }
 
+        $this->afterRunAction();
+
         /* run before install script */
         $this->runScript('afterUninstall');
 
-        $this->afterRunAction();
-
         $this->clearCache();
 
         /* delete backup files */
@@ -73,12 +75,6 @@ class Uninstall extends \Espo\Core\Upgrades\Actions\Base
         $GLOBALS['log']->debug('Uninstallation process ['.$processId.']: end run.');
     }
 
-    protected function getDeleteFileList()
-    {
-        $extensionEntity = $this->getExtensionEntity();
-        return $extensionEntity->get('fileList');
-    }
-
     protected function restoreFiles()
     {
         $packagePath = $this->getPath('packagePath');
@@ -131,4 +127,41 @@ class Uninstall extends \Espo\Core\Upgrades\Actions\Base
         throw new Error($errorMessage);
     }
 
+    protected function getCopyFileList()
+    {
+        if (!isset($this->data['fileList'])) {
+            $backupPath = $this->getPath('backupPath');
+            $filesPath = Util::concatPath($backupPath, self::FILES);
+
+            $this->data['fileList'] = $this->getFileManager()->getFileList($filesPath, true, '', true, true);
+        }
+
+        return $this->data['fileList'];
+    }
+
+    protected function getRestoreFileList()
+    {
+        if (!isset($this->data['restoreFileList'])) {
+            $packagePath = $this->getPackagePath();
+            $filesPath = Util::concatPath($packagePath, self::FILES);
+
+            if (!file_exists($filesPath)) {
+                $this->unzipArchive($packagePath);
+            }
+
+            $this->data['restoreFileList'] = $this->getFileManager()->getFileList($filesPath, true, '', true, true);
+        }
+
+        return $this->data['restoreFileList'];
+    }
+
+    protected function getDeleteFileList()
+    {
+        $packageFileList = $this->getRestoreFileList();
+        $backupFileList = $this->getCopyFileList();
+
+        $deleteFileList = array_diff($packageFileList, $backupFileList);
+
+        return $deleteFileList;
+    }
 }
diff --git a/application/Espo/Core/Upgrades/Actions/Extension/Install.php b/application/Espo/Core/Upgrades/Actions/Extension/Install.php
index dff99f54a0..1fb605cd8a 100644
--- a/application/Espo/Core/Upgrades/Actions/Extension/Install.php
+++ b/application/Espo/Core/Upgrades/Actions/Extension/Install.php
@@ -37,8 +37,6 @@ class Install extends \Espo\Core\Upgrades\Actions\Base\Install
             $this->uninstallExtension();
             $this->deleteExtension();
         }
-
-        $this->copyExistingFiles();
     }
 
     protected function afterRunAction()
@@ -51,12 +49,11 @@ class Install extends \Espo\Core\Upgrades\Actions\Base\Install
      *
      * @return bool
      */
-    protected function copyExistingFiles()
+    protected function backupExistingFiles()
     {
-        $fileList = $this->getCopyFileList();
-        $backupPath = $this->getPath('backupPath');
+        parent::backupExistingFiles();
 
-        $res = $this->copy('', array($backupPath, self::FILES), false, $fileList);
+        $backupPath = $this->getPath('backupPath');
 
         /** copy scripts files */
         $packagePath = $this->getPackagePath();
@@ -65,19 +62,6 @@ class Install extends \Espo\Core\Upgrades\Actions\Base\Install
         return $res;
     }
 
-    protected function restoreFiles()
-    {
-        $res = true;
-        if ($this->isCopied) {
-            $extensionFileList = $this->getCopyFileList();
-            $res &= $this->getFileManager()->remove($extensionFileList);
-        }
-
-        $res &= parent::restoreFiles();
-
-        return $res;
-    }
-
     protected function isNew()
     {
         $extensionEntity = $this->getExtensionEntity();
diff --git a/application/Espo/Core/Upgrades/Actions/Extension/Uninstall.php b/application/Espo/Core/Upgrades/Actions/Extension/Uninstall.php
index 41745c0979..d68fe183d2 100644
--- a/application/Espo/Core/Upgrades/Actions/Extension/Uninstall.php
+++ b/application/Espo/Core/Upgrades/Actions/Extension/Uninstall.php
@@ -68,4 +68,14 @@ class Uninstall extends \Espo\Core\Upgrades\Actions\Base\Uninstall
         $extensionEntity->set('isInstalled', false);
         $this->getEntityManager()->saveEntity($extensionEntity);
     }
+
+    protected function getRestoreFileList()
+    {
+        if (!isset($this->data['restoreFileList'])) {
+            $extensionEntity = $this->getExtensionEntity();
+            $this->data['restoreFileList'] = $extensionEntity->get('fileList');
+        }
+
+        return $this->data['restoreFileList'];
+    }
 }
\ No newline at end of file
diff --git a/application/Espo/Core/Upgrades/Actions/Upgrade/Install.php b/application/Espo/Core/Upgrades/Actions/Upgrade/Install.php
index 5d3b69a8c5..407ff85733 100644
--- a/application/Espo/Core/Upgrades/Actions/Upgrade/Install.php
+++ b/application/Espo/Core/Upgrades/Actions/Upgrade/Install.php
@@ -24,17 +24,12 @@ namespace Espo\Core\Upgrades\Actions\Upgrade;
 
 class Install extends \Espo\Core\Upgrades\Actions\Base\Install
 {
-    protected function systemRebuild()
+    protected function afterRunAction()
     {
         $manifest = $this->getManifest();
 
-        $res = $this->getConfig()->set('version', $manifest['version']);
-        if (method_exists($this->getConfig(), 'save')) {
-            $res = $this->getConfig()->save();
-        }
-        $res &= parent::systemRebuild();
-
-        return $res;
+        $this->getConfig()->set('version', $manifest['version']);
+        $this->getConfig()->save();
     }
 
     /**
diff --git a/application/Espo/Core/Utils/File/Manager.php b/application/Espo/Core/Utils/File/Manager.php
index e0cb8f4576..391ffabef6 100644
--- a/application/Espo/Core/Utils/File/Manager.php
+++ b/application/Espo/Core/Utils/File/Manager.php
@@ -29,6 +29,8 @@ class Manager
 {
     private $permission;
 
+    private $permissionDeniedList = array();
+
     public function __construct(\Espo\Core\Utils\Config $config = null)
     {
         $params = null;
@@ -774,5 +776,73 @@ return '.var_export($content, true).';
 ?>';
     }
 
+    /**
+     * Check if $paths are writable. Permission denied list are defined in getLastPermissionDeniedList()
+     *
+     * @param  array   $paths
+     *
+     * @return boolean
+     */
+    public function isWritableList(array $paths)
+    {
+        $permissionDeniedList = array();
+
+        $result = true;
+        foreach ($paths as $path) {
+            $rowResult = $this->isWritable($path);
+            if (!$rowResult) {
+                $permissionDeniedList[] = $path;
+            }
+            $result &= $rowResult;
+        }
+
+        if (!empty($permissionDeniedList)) {
+            $this->permissionDeniedList = $this->getPermissionUtils()->arrangePermissionList($permissionDeniedList);
+        }
+
+        return (bool) $result;
+    }
+
+    /**
+     * Get last permission denied list
+     *
+     * @return array
+     */
+    public function getLastPermissionDeniedList()
+    {
+        return $this->permissionDeniedList;
+    }
+
+    /**
+     * Check if $path is writable
+     *
+     * @param  string | array  $path
+     *
+     * @return boolean
+     */
+    public function isWritable($path)
+    {
+        $existFile = $this->getExistsPath($path);
+
+        return is_writable($existFile);
+    }
+
+    /**
+     * Get exists path. Ex. if check /var/www/espocrm/custom/someFile.php and this file doesn't extist, result will be /var/www/espocrm/custom
+     *
+     * @param  string | array $path
+     *
+     * @return string
+     */
+    protected function getExistsPath($path)
+    {
+        $fullPath = $this->concatPaths($path);
+
+        if (!file_exists($fullPath)) {
+            $fullPath = $this->getExistsPath($fullPath, PATHINFO_DIRNAME);
+        }
+
+        return $fullPath;
+    }
 }
 

From b6010a7ad723f6a201977dbfd7b24ad74e3c991a Mon Sep 17 00:00:00 2001
From: Taras Machyshyn <taras@letrium.com>
Date: Tue, 17 Feb 2015 12:03:55 +0200
Subject: [PATCH 2/8] added a new fileManager test, fixes

---
 application/Espo/Core/Utils/File/Manager.php |  2 +-
 tests/Espo/Core/Utils/File/ManagerTest.php   | 20 ++++++++++++++++++--
 2 files changed, 19 insertions(+), 3 deletions(-)

diff --git a/application/Espo/Core/Utils/File/Manager.php b/application/Espo/Core/Utils/File/Manager.php
index 391ffabef6..5a1e5a35cd 100644
--- a/application/Espo/Core/Utils/File/Manager.php
+++ b/application/Espo/Core/Utils/File/Manager.php
@@ -839,7 +839,7 @@ return '.var_export($content, true).';
         $fullPath = $this->concatPaths($path);
 
         if (!file_exists($fullPath)) {
-            $fullPath = $this->getExistsPath($fullPath, PATHINFO_DIRNAME);
+            $fullPath = $this->getExistsPath(pathinfo($fullPath, PATHINFO_DIRNAME));
         }
 
         return $fullPath;
diff --git a/tests/Espo/Core/Utils/File/ManagerTest.php b/tests/Espo/Core/Utils/File/ManagerTest.php
index f0a0db14ce..bc231ce1c5 100644
--- a/tests/Espo/Core/Utils/File/ManagerTest.php
+++ b/tests/Espo/Core/Utils/File/ManagerTest.php
@@ -1,4 +1,4 @@
-<?php
+<?php
 /************************************************************************
  * This file is part of EspoCRM.
  *
@@ -389,7 +389,23 @@ class ManagerTest extends \PHPUnit_Framework_TestCase
         }
     }
 
+    public function existsPathSet()
+    {
+        return array(
+          array( 'application/Espo/Core/Application.php', 'application/Espo/Core/Application.php', ),
+          array( 'application/Espo/Core/NotRealApplication.php', 'application/Espo/Core'),
+          array( array('application', 'Espo/Core', 'NotRealApplication.php'), 'application/Espo/Core'),
+          array( 'application/NoEspo/Core/Application.php', 'application'),
+          array( 'notRealPath/Espo/Core/Application.php', '.'),
+        );
+    }
 
+    /**
+     * @dataProvider existsPathSet
+     */
+    public function testGetExistsPath($input, $result)
+    {
+        $this->assertEquals($result, $this->reflection->invokeMethod('getExistsPath', array($input)) );
+    }
 }
 
-?>

From b5b8ab4a8ea032eb7d91b947a44033fdd63f6bf5 Mon Sep 17 00:00:00 2001
From: Taras Machyshyn <taras@letrium.com>
Date: Tue, 17 Feb 2015 13:02:02 +0200
Subject: [PATCH 3/8] added initialize(), finalize() methods to upgrade scripts

---
 application/Espo/Core/Upgrades/Actions/Base.php        | 10 ++++++++++
 application/Espo/Core/Upgrades/Actions/Base/Delete.php |  4 ++++
 .../Espo/Core/Upgrades/Actions/Base/Install.php        |  6 +++++-
 .../Espo/Core/Upgrades/Actions/Base/Uninstall.php      |  4 ++++
 application/Espo/Core/Upgrades/Actions/Base/Upload.php |  4 ++++
 .../Espo/Core/Upgrades/Actions/Upgrade/Install.php     |  2 +-
 6 files changed, 28 insertions(+), 2 deletions(-)

diff --git a/application/Espo/Core/Upgrades/Actions/Base.php b/application/Espo/Core/Upgrades/Actions/Base.php
index 2b0f530910..91a1d3bba1 100644
--- a/application/Espo/Core/Upgrades/Actions/Base.php
+++ b/application/Espo/Core/Upgrades/Actions/Base.php
@@ -504,6 +504,16 @@ abstract class Base
         $this->getActionManager()->setAction($currentAction);
     }
 
+    protected function initialize()
+    {
+
+    }
+
+    protected function finalize()
+    {
+
+    }
+
     protected function beforeRunAction()
     {
 
diff --git a/application/Espo/Core/Upgrades/Actions/Base/Delete.php b/application/Espo/Core/Upgrades/Actions/Base/Delete.php
index edcbcb66aa..c301a04228 100644
--- a/application/Espo/Core/Upgrades/Actions/Base/Delete.php
+++ b/application/Espo/Core/Upgrades/Actions/Base/Delete.php
@@ -32,6 +32,8 @@ class Delete extends \Espo\Core\Upgrades\Actions\Base
             throw new Error('Delete package package ID was not specified.');
         }
 
+        $this->initialize();
+
         $this->setProcessId($processId);
 
         $this->beforeRunAction();
@@ -41,6 +43,8 @@ class Delete extends \Espo\Core\Upgrades\Actions\Base
 
         $this->afterRunAction();
 
+        $this->finalize();
+
         $GLOBALS['log']->debug('Delete package process ['.$processId.']: end run.');
     }
 
diff --git a/application/Espo/Core/Upgrades/Actions/Base/Install.php b/application/Espo/Core/Upgrades/Actions/Base/Install.php
index 4d27ba9b87..85b4a42b99 100644
--- a/application/Espo/Core/Upgrades/Actions/Base/Install.php
+++ b/application/Espo/Core/Upgrades/Actions/Base/Install.php
@@ -50,6 +50,8 @@ class Install extends \Espo\Core\Upgrades\Actions\Base
 
         $this->setProcessId($processId);
 
+        $this->initialize();
+
         $this->isCopied = false;
 
         /** check if an archive is unzipped, if no then unzip */
@@ -76,7 +78,7 @@ class Install extends \Espo\Core\Upgrades\Actions\Base
         $this->isCopied = true;
 
         /* remove files defined in a manifest */
-        $this->deleteFiles();
+        $this->deleteFiles(true);
 
         if (!$this->systemRebuild()) {
             $this->throwErrorAndRemovePackage('Error occurred while EspoCRM rebuild.');
@@ -92,6 +94,8 @@ class Install extends \Espo\Core\Upgrades\Actions\Base
         /* delete unziped files */
         $this->deletePackageFiles();
 
+        $this->finalize();
+
         $GLOBALS['log']->debug('Installation process ['.$processId.']: end run.');
     }
 
diff --git a/application/Espo/Core/Upgrades/Actions/Base/Uninstall.php b/application/Espo/Core/Upgrades/Actions/Base/Uninstall.php
index 722e0e9287..910ad9dd29 100644
--- a/application/Espo/Core/Upgrades/Actions/Base/Uninstall.php
+++ b/application/Espo/Core/Upgrades/Actions/Base/Uninstall.php
@@ -37,6 +37,8 @@ class Uninstall extends \Espo\Core\Upgrades\Actions\Base
 
         $this->setProcessId($processId);
 
+        $this->initialize();
+
         $this->checkIsWritable();
 
         /* run before install script */
@@ -72,6 +74,8 @@ class Uninstall extends \Espo\Core\Upgrades\Actions\Base
         /* delete backup files */
         $this->deletePackageFiles();
 
+        $this->finalize();
+
         $GLOBALS['log']->debug('Uninstallation process ['.$processId.']: end run.');
     }
 
diff --git a/application/Espo/Core/Upgrades/Actions/Base/Upload.php b/application/Espo/Core/Upgrades/Actions/Base/Upload.php
index 2d10db12ec..03ebae176e 100644
--- a/application/Espo/Core/Upgrades/Actions/Base/Upload.php
+++ b/application/Espo/Core/Upgrades/Actions/Base/Upload.php
@@ -38,6 +38,8 @@ class Upload extends \Espo\Core\Upgrades\Actions\Base
 
         $GLOBALS['log']->debug('Installation process ['.$processId.']: start upload the package.');
 
+        $this->initialize();
+
         $packagePath = $this->getPackagePath();
         $packageArchivePath = $this->getPackagePath(true);
 
@@ -55,6 +57,8 @@ class Upload extends \Espo\Core\Upgrades\Actions\Base
 
         $this->isAcceptable();
 
+        $this->finalize();
+
         $GLOBALS['log']->debug('Installation process ['.$processId.']: end upload the package.');
 
         return $processId;
diff --git a/application/Espo/Core/Upgrades/Actions/Upgrade/Install.php b/application/Espo/Core/Upgrades/Actions/Upgrade/Install.php
index 407ff85733..026b9ff6a7 100644
--- a/application/Espo/Core/Upgrades/Actions/Upgrade/Install.php
+++ b/application/Espo/Core/Upgrades/Actions/Upgrade/Install.php
@@ -24,7 +24,7 @@ namespace Espo\Core\Upgrades\Actions\Upgrade;
 
 class Install extends \Espo\Core\Upgrades\Actions\Base\Install
 {
-    protected function afterRunAction()
+    protected function finalize()
     {
         $manifest = $this->getManifest();
 

From b5d2eb93aa532da785d080816144e4a4b252f4bb Mon Sep 17 00:00:00 2001
From: Taras Machyshyn <taras@letrium.com>
Date: Tue, 17 Feb 2015 13:46:37 +0200
Subject: [PATCH 4/8] Upgrades: impoved beforeRunAction(), afterRunAction()

---
 application/Espo/Core/Upgrades/Actions/Base/Install.php   | 8 ++++----
 application/Espo/Core/Upgrades/Actions/Base/Uninstall.php | 8 ++++----
 application/Espo/Core/Upgrades/Actions/Base/Upload.php    | 4 ++++
 3 files changed, 12 insertions(+), 8 deletions(-)

diff --git a/application/Espo/Core/Upgrades/Actions/Base/Install.php b/application/Espo/Core/Upgrades/Actions/Base/Install.php
index 85b4a42b99..512b6362b8 100644
--- a/application/Espo/Core/Upgrades/Actions/Base/Install.php
+++ b/application/Espo/Core/Upgrades/Actions/Base/Install.php
@@ -66,11 +66,11 @@ class Install extends \Espo\Core\Upgrades\Actions\Base
 
         $this->backupExistingFiles();
 
+        $this->beforeRunAction();
+
         /* run before install script */
         $this->runScript('before');
 
-        $this->beforeRunAction();
-
         /* copy files from directory "Files" to EspoCRM files */
         if (!$this->copyFiles()) {
             $this->throwErrorAndRemovePackage('Cannot copy files.');
@@ -84,11 +84,11 @@ class Install extends \Espo\Core\Upgrades\Actions\Base
             $this->throwErrorAndRemovePackage('Error occurred while EspoCRM rebuild.');
         }
 
-        $this->afterRunAction();
-
         /* run before install script */
         $this->runScript('after');
 
+        $this->afterRunAction();
+
         $this->clearCache();
 
         /* delete unziped files */
diff --git a/application/Espo/Core/Upgrades/Actions/Base/Uninstall.php b/application/Espo/Core/Upgrades/Actions/Base/Uninstall.php
index 910ad9dd29..0c39b42b16 100644
--- a/application/Espo/Core/Upgrades/Actions/Base/Uninstall.php
+++ b/application/Espo/Core/Upgrades/Actions/Base/Uninstall.php
@@ -41,11 +41,11 @@ class Uninstall extends \Espo\Core\Upgrades\Actions\Base
 
         $this->checkIsWritable();
 
+        $this->beforeRunAction();
+
         /* run before install script */
         $this->runScript('beforeUninstall');
 
-        $this->beforeRunAction();
-
         $backupPath = $this->getPath('backupPath');
         if (file_exists($backupPath)) {
 
@@ -64,11 +64,11 @@ class Uninstall extends \Espo\Core\Upgrades\Actions\Base
             throw new $this->throwErrorAndRemovePackage('Error occurred while EspoCRM rebuild.');
         }
 
-        $this->afterRunAction();
-
         /* run before install script */
         $this->runScript('afterUninstall');
 
+        $this->afterRunAction();
+
         $this->clearCache();
 
         /* delete backup files */
diff --git a/application/Espo/Core/Upgrades/Actions/Base/Upload.php b/application/Espo/Core/Upgrades/Actions/Base/Upload.php
index 03ebae176e..1effbd029d 100644
--- a/application/Espo/Core/Upgrades/Actions/Base/Upload.php
+++ b/application/Espo/Core/Upgrades/Actions/Base/Upload.php
@@ -40,6 +40,8 @@ class Upload extends \Espo\Core\Upgrades\Actions\Base
 
         $this->initialize();
 
+        $this->beforeRunAction();
+
         $packagePath = $this->getPackagePath();
         $packageArchivePath = $this->getPackagePath(true);
 
@@ -57,6 +59,8 @@ class Upload extends \Espo\Core\Upgrades\Actions\Base
 
         $this->isAcceptable();
 
+        $this->afterRunAction();
+
         $this->finalize();
 
         $GLOBALS['log']->debug('Installation process ['.$processId.']: end upload the package.');

From 7453eb583ad0d6e753e56123fcdfdcbb72b7805f Mon Sep 17 00:00:00 2001
From: Taras Machyshyn <taras@letrium.com>
Date: Tue, 17 Feb 2015 15:43:51 +0200
Subject: [PATCH 5/8] impoved Uprades code

---
 .../Upgrades/Actions/Extension/Delete.php     | 27 ++++++-----------
 .../Upgrades/Actions/Extension/Uninstall.php  | 30 +++++--------------
 2 files changed, 17 insertions(+), 40 deletions(-)

diff --git a/application/Espo/Core/Upgrades/Actions/Extension/Delete.php b/application/Espo/Core/Upgrades/Actions/Extension/Delete.php
index d33fee95ea..c595c98901 100644
--- a/application/Espo/Core/Upgrades/Actions/Extension/Delete.php
+++ b/application/Espo/Core/Upgrades/Actions/Extension/Delete.php
@@ -38,26 +38,17 @@ class Delete extends \Espo\Core\Upgrades\Actions\Base\Delete
         return $this->extensionEntity;
     }
 
-    /**
-     * Set Extension Entity
-     *
-     * @param \Espo\Entities\Extension $extensionEntity
-     */
-    protected function setExtensionEntity(\Espo\Entities\Extension $extensionEntity)
+    protected function getExtensionEntity()
     {
-        $this->extensionEntity = $extensionEntity;
-    }
-
-    protected function beforeRunAction()
-    {
-        $processId = $this->getProcessId();
-
-        /** get extension entity */
-        $extensionEntity = $this->getEntityManager()->getEntity('Extension', $processId);
-        if (!isset($extensionEntity)) {
-            throw new Error('Extension Entity not found.');
+        if (!isset($this->extensionEntity)) {
+            $processId = $this->getProcessId();
+            $this->extensionEntity = $this->getEntityManager()->getEntity('Extension', $processId);
+            if (!isset($this->extensionEntity)) {
+                throw new Error('Extension Entity not found.');
+            }
         }
-        $this->setExtensionEntity($extensionEntity);
+
+        return $this->extensionEntity;
     }
 
     protected function afterRunAction()
diff --git a/application/Espo/Core/Upgrades/Actions/Extension/Uninstall.php b/application/Espo/Core/Upgrades/Actions/Extension/Uninstall.php
index d68fe183d2..a310bc319b 100644
--- a/application/Espo/Core/Upgrades/Actions/Extension/Uninstall.php
+++ b/application/Espo/Core/Upgrades/Actions/Extension/Uninstall.php
@@ -35,29 +35,15 @@ class Uninstall extends \Espo\Core\Upgrades\Actions\Base\Uninstall
      */
     protected function getExtensionEntity()
     {
-        return $this->extensionEntity;
-    }
-
-    /**
-     * Set Extension Entity
-     *
-     * @param \Espo\Entities\Extension $extensionEntity [description]
-     */
-    protected function setExtensionEntity(\Espo\Entities\Extension $extensionEntity)
-    {
-        $this->extensionEntity = $extensionEntity;
-    }
-
-    protected function beforeRunAction()
-    {
-        $processId = $this->getProcessId();
-
-        /** get extension entity */
-        $extensionEntity = $this->getEntityManager()->getEntity('Extension', $processId);
-        if (!isset($extensionEntity)) {
-            throw new Error('Extension Entity not found.');
+        if (!isset($this->extensionEntity)) {
+            $processId = $this->getProcessId();
+            $this->extensionEntity = $this->getEntityManager()->getEntity('Extension', $processId);
+            if (!isset($this->extensionEntity)) {
+                throw new Error('Extension Entity not found.');
+            }
         }
-        $this->setExtensionEntity($extensionEntity);
+
+        return $this->extensionEntity;
     }
 
     protected function afterRunAction()

From 4e6b300477fc7915346875b1f1a1f044b7d81bdd Mon Sep 17 00:00:00 2001
From: Taras Machyshyn <taras@letrium.com>
Date: Tue, 17 Feb 2015 16:39:51 +0200
Subject: [PATCH 6/8] Upgrades: improved restore functionality

---
 .../Espo/Core/Upgrades/Actions/Base/Uninstall.php    | 12 ++++++++++--
 1 file changed, 10 insertions(+), 2 deletions(-)

diff --git a/application/Espo/Core/Upgrades/Actions/Base/Uninstall.php b/application/Espo/Core/Upgrades/Actions/Base/Uninstall.php
index 0c39b42b16..bd835e8e35 100644
--- a/application/Espo/Core/Upgrades/Actions/Base/Uninstall.php
+++ b/application/Espo/Core/Upgrades/Actions/Base/Uninstall.php
@@ -22,8 +22,9 @@
 
 namespace Espo\Core\Upgrades\Actions\Base;
 
-use Espo\Core\Exceptions\Error,
-    Espo\Core\Utils\Util;
+use Espo\Core\Exceptions\Error;
+use Espo\Core\Utils\Util;
+use Espo\Core\Utils\Json;
 
 class Uninstall extends \Espo\Core\Upgrades\Actions\Base
 {
@@ -89,6 +90,13 @@ class Uninstall extends \Espo\Core\Upgrades\Actions\Base
         }
 
         $res = $this->copy($filesPath, '', true);
+
+        $manifestJson = $this->getFileManager()->getContents(array($packagePath, $this->manifestName));
+        $manifest = Json::decode($manifestJson, true);
+        if (!empty($manifest['delete'])) {
+            $res &= $this->getFileManager()->remove($manifest['delete'], null, true);
+        }
+
         $res &= $this->getFileManager()->removeInDir($packagePath, true);
 
         return $res;

From e09c101f8255f08cafc3a6a0f25ed1f4ddbe2a87 Mon Sep 17 00:00:00 2001
From: Taras Machyshyn <taras@letrium.com>
Date: Tue, 17 Feb 2015 16:45:21 +0200
Subject: [PATCH 7/8] Upgrades: fixed an error

---
 application/Espo/Core/Upgrades/Actions/Extension/Delete.php | 5 -----
 1 file changed, 5 deletions(-)

diff --git a/application/Espo/Core/Upgrades/Actions/Extension/Delete.php b/application/Espo/Core/Upgrades/Actions/Extension/Delete.php
index c595c98901..8476b1c852 100644
--- a/application/Espo/Core/Upgrades/Actions/Extension/Delete.php
+++ b/application/Espo/Core/Upgrades/Actions/Extension/Delete.php
@@ -33,11 +33,6 @@ class Delete extends \Espo\Core\Upgrades\Actions\Base\Delete
      *
      * @return \Espo\Entities\Extension
      */
-    protected function getExtensionEntity()
-    {
-        return $this->extensionEntity;
-    }
-
     protected function getExtensionEntity()
     {
         if (!isset($this->extensionEntity)) {

From 7b0b3d05ad86820ee2f8c75766f5282a4f997dd6 Mon Sep 17 00:00:00 2001
From: yuri <yurak@list.ru>
Date: Tue, 17 Feb 2015 16:46:13 +0200
Subject: [PATCH 8/8] fix zend mail

---
 application/Espo/Core/Mail/Storage/Message.php | 9 +++++++++
 1 file changed, 9 insertions(+)

diff --git a/application/Espo/Core/Mail/Storage/Message.php b/application/Espo/Core/Mail/Storage/Message.php
index 5ac07f0ce9..f4ab45fff0 100644
--- a/application/Espo/Core/Mail/Storage/Message.php
+++ b/application/Espo/Core/Mail/Storage/Message.php
@@ -24,6 +24,15 @@ namespace Espo\Core\Mail\Storage;
 
 class Message extends \Zend\Mail\Storage\Message
 {
+    public function __isset($name)
+    {
+        $headers = $this->getHeaders();
+        if (empty($headers) || !is_object($headers)) {
+            return false;
+        }
+        return $this->getHeaders()->has($name);
+    }
+
     public function isMultipart()
     {
         if (!isset($this->contentType)) {