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

MINOR Better isset checks on Versioned#allVersions() due to changes in SQLQuery

Browse Source
This commit is contained in:
Stig Lindqvist 2011-09-27 15:14:01 +13:00 committed by Stig Lindqvist
parent 8d7bfd4ffa
commit 75c0fcd052
1 changed files with 6 additions and 3 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

9
model/Versioned.php
View File

@ -679,7 +679,7 @@ class Versioned extends DataExtension {
* Return a list of all the versions available.
* @param string $filter
*/
function allVersions($filter = "", $sort = "", $limit = "", $join = "", $having = "") {
public function allVersions($filter = "", $sort = "", $limit = "", $join = "", $having = "") {
// Make sure the table names are not postfixed (e.g. _Live)
$oldMode = self::get_reading_mode();
self::reading_stage('Stage');
@ -687,8 +687,11 @@ class Versioned extends DataExtension {
$query = $this->owner->extendedSQL($filter, $sort, $limit, $join, $having);
foreach($query->from as $table => $tableJoin) {
if($tableJoin[0] == '"') $baseTable = str_replace('"','',$tableJoin);
else if (substr($tableJoin,0,5) != 'INNER') $query->from[$table] = "LEFT JOIN \"$table\" ON \"$table\".\"RecordID\" = \"{$baseTable}_versions\".\"RecordID\" AND \"$table\".\"Version\" = \"{$baseTable}_versions\".\"Version\"";
if(is_string($tableJoin) && $tableJoin[0] == '"') {
$baseTable = str_replace('"','',$tableJoin);
} elseif(is_string($tableJoin) && substr($tableJoin,0,5) != 'INNER') {
$query->from[$table] = "LEFT JOIN \"$table\" ON \"$table\".\"RecordID\" = \"{$baseTable}_versions\".\"RecordID\" AND \"$table\".\"Version\" = \"{$baseTable}_versions\".\"Version\"";
}
$query->renameTable($table, $table . '_versions');
}