diff --git a/workflow/engine/classes/class.patch.php b/workflow/engine/classes/class.patch.php index 5a0cf840f..ec10a6099 100644 --- a/workflow/engine/classes/class.patch.php +++ b/workflow/engine/classes/class.patch.php @@ -10,16 +10,21 @@ G::LoadClass("System"); * */ -class patch +abstract class patch +{ + static protected $isPathchable = false; + abstract static public function isApplicable(); + abstract static public function execute(); +} + +class p11835 extends patch { - static private $isPathchable = false; - /* * Note.- Use before upgrade DB. * Check if the table TASK has the field TAS_GROUP_VARIABLE * @return boolean */ - static public function is_11835Applicable() + static public function isApplicable() { patch::$isPathchable = false; $con = Propel::getConnection("workflow"); @@ -30,7 +35,6 @@ class patch if ($row ['Field'] == "TAS_GROUP_VARIABLE") { $version = System::getVersion (); $version = explode('-',$version); - //$pos = strpos($version,'2.5.1-testing'); if ($version[0] == '2.5.1') { echo "Version " . $version[0] . " Patch\n"; patch::$isPathchable = true; @@ -48,7 +52,7 @@ class patch * if the current task has asignated users, means SELF_SERVICE only, * otherwise leave TAS_GROUP_VARIABLE as it is. */ - static public function execute_11835() + static public function execute() { //Check if this is the version to apply the patch $count = 0; @@ -80,4 +84,3 @@ class patch } } - diff --git a/workflow/engine/classes/class.wsTools.php b/workflow/engine/classes/class.wsTools.php index 7a5ed4253..5655ec311 100755 --- a/workflow/engine/classes/class.wsTools.php +++ b/workflow/engine/classes/class.wsTools.php @@ -602,11 +602,11 @@ class workspaceTools { G::LoadClass("patch"); $this->initPropel( true ); - patch::is_11835Applicable(); + p11835::isApplicable(); $systemSchema = System::getSystemSchema(); $this->upgradeSchema( $systemSchema ); $this->upgradeData(); - patch::execute_11835(); + p11835::execute(); return true; }