tonyair/silverstripe-framework
1
0
Fork 0
mirror of https://github.com/silverstripe/silverstripe-framework synced 2024-07-04 02:19:32 +02:00
Code Issues Projects Releases Wiki Activity

Merge pull request #4736 from Cheddam/listdecorator-totalitems-fix

Browse Source 
Corrected TotalItems() method to use Count(). (fixes #4646)
This commit is contained in:
Loz Calver 2015-11-03 12:24:50 +00:00
commit 7896ce6a90
2 changed files with 28 additions and 1 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

2
model/ListDecorator.php
View File

@ -81,7 +81,7 @@ abstract class SS_ListDecorator extends ViewableData implements SS_List, SS_Sort
}
public function TotalItems() {
return $this->list->TotalItems();
return $this->list->Count();
}
public function Count() {

27
tests/model/GroupedListTest.php
View File

@ -123,4 +123,31 @@ class GroupedListTest extends SapphireTest {
}
}
public function testTotalItems() {
$list = GroupedList::create(
ArrayList::create(
array(
ArrayData::create(array(
'Name' => 'AAA',
'Number' => '111',
)),
ArrayData::create(array(
'Name' => 'BBB',
'Number' => '111',
)),
ArrayData::create(array(
'Name' => 'AAA',
'Number' => '222',
)),
ArrayData::create(array(
'Name' => 'BBB',
'Number' => '111',
))
)
)
);
$this->assertEquals(4, $list->TotalItems());
}
}