tonyair/silverstripe-framework
1
0
Fork 0
mirror of https://github.com/silverstripe/silverstripe-framework synced 2024-10-22 14:05:37 +02:00
Code Issues Projects Releases Wiki Activity

BUG Better filtering of versionable tables during SQL augmentation

Browse Source
This commit is contained in:
Damian Mooyman 2016-03-07 12:47:04 +13:00
parent 08c6aca9a4
commit 3d99ed24ac
1 changed files with 23 additions and 8 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

31
model/versioning/Versioned.php
View File

@ -264,7 +264,7 @@ class Versioned extends DataExtension implements TemplateGlobalProvider {
case 'archive':
$date = $dataQuery->getQueryParam('Versioned.date');
foreach($query->getFrom() as $table => $dummy) {
if(!DB::get_schema()->hasTable($table . '_versions')) {
if(!$this->isTableVersioned($table)) {
continue;
}
@ -303,14 +303,10 @@ class Versioned extends DataExtension implements TemplateGlobalProvider {
$stage = $dataQuery->getQueryParam('Versioned.stage');
if($stage && ($stage != $this->defaultStage)) {
foreach($query->getFrom() as $table => $dummy) {
// Only rewrite table names that are actually part of the subclass tree
// This helps prevent rewriting of other tables that get joined in, in
// particular, many_many tables
if(class_exists($table) && ($table == $this->owner->class
|| is_subclass_of($table, $this->owner->class)
|| is_subclass_of($this->owner->class, $table))) {
$query->renameTable($table, $table . '_' . $stage);
if(!$this->isTableVersioned($table)) {
continue;
}
$query->renameTable($table, $table . '_' . $stage);
}
}
break;
@ -342,6 +338,10 @@ class Versioned extends DataExtension implements TemplateGlobalProvider {
case 'all_versions':
case 'latest_versions':
foreach($query->getFrom() as $alias => $join) {
if(!$this->isTableVersioned($alias)) {
continue;
}
if($alias != $baseTable) {
// Make sure join includes version as well
$query->setJoinFilter(
@ -394,6 +394,21 @@ class Versioned extends DataExtension implements TemplateGlobalProvider {
}
}
/**
* Determine if the given versioned table is a part of the sub-tree of the current dataobject
* This helps prevent rewriting of other tables that get joined in, in particular, many_many tables
*
* @param string $table
* @return bool True if this table should be versioned
*/
protected function isTableVersioned($table) {
if(!class_exists($table)) {
return false;
}
$baseClass = ClassInfo::baseDataClass($this->owner);
return is_a($table, $baseClass, true);
}
/**
* For lazy loaded fields requiring extra sql manipulation, ie versioning.
*