diff options
author | Thomas Steur <thomas.steur@googlemail.com> | 2014-07-08 07:40:25 +0400 |
---|---|---|
committer | Thomas Steur <thomas.steur@googlemail.com> | 2014-07-08 07:40:25 +0400 |
commit | c01d57bc17f0e9a07122539b9c5ee60872ff5a28 (patch) | |
tree | 177d48a72ad3b86bda7168388686b49237ceb2fb /core/Plugin/Dimension | |
parent | 4a9a872095b4ba8a5c416249b37647fdd9dc6e69 (diff) |
this is a very complicated one. Problem was when updating from 2.4.0 to 2.5.0-b1 the updater wanted to update all dimensions (meaning alter all columns in log_visit, link_action and conversion) to the same column type. This was happening because the system did not know those dimensions were already installed from a previous Piwik version. There was an update script that was supposed to tell Piwik those components are actually already installed since we only moved them from core to plugins but it cannot work as it is an update as well and therefore not executed before the actual update check. I tried many solutions to overcome this issue including reverting all the columns to the initial MySql Schema but even then there are problems as some plugins like DevicesDetection are not defined in MySql Schema but in the plugin. It is especially complicated since users might update from 2.4 to a future version where the column type of one of those dimension changes and we need to make sure to actually execute an alter update if one of those dimension changes. In such a case we cannot directly mark the component as successfully recorded. The conclusion was for me it is only possible to solve this problem by listing all dimensions that were moved from core to plugins including their version at that time hard coded...
Diffstat (limited to 'core/Plugin/Dimension')
-rw-r--r-- | core/Plugin/Dimension/VisitDimension.php | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/core/Plugin/Dimension/VisitDimension.php b/core/Plugin/Dimension/VisitDimension.php index f87f09acc8..cf7204bdd5 100644 --- a/core/Plugin/Dimension/VisitDimension.php +++ b/core/Plugin/Dimension/VisitDimension.php @@ -45,7 +45,7 @@ abstract class VisitDimension extends Dimension return $changes; } - public function update($visitColumns, $conversionColumns) + public function update($conversionColumns) { if (!$this->columnType) { return array(); |