Merge remote-tracking branch 'origin/3.1'

This commit is contained in:
Simon Welsh 2013-07-05 11:56:31 +12:00
commit 4b57a343a2
2 changed files with 4 additions and 2 deletions

View File

@ -183,7 +183,8 @@ abstract class SearchFilter extends Object {
if($candidateClass == 'DataObject') {
// fallback to the provided name in the event of a joined column
// name (as the candidate class doesn't check joined records)
return $this->fullName;
$parts = explode('.', $this->fullName);
return '"' . implode('"."', $parts) . '"';
}
return "\"$candidateClass\".\"$this->name\"";

View File

@ -625,7 +625,8 @@ class DataListTest extends SapphireTest {
));
$this->assertEquals(2, $list->count());
$this->assertEquals(array('Joe', 'Bob'), $list->column('Name'));
$values = $list->column('Name');
$this->assertEquals(array_intersect($values, array('Joe', 'Bob')), $values);
}
public function testFilterAndExcludeById() {