Skip to content

Commit c8a627f

Browse files
authored
Merge pull request #1856 from Sekiphp/grid-page-size
Grid page size: replaced similar lines to foreach
2 parents 99e5c46 + c972d23 commit c8a627f

File tree

2 files changed

+10
-5
lines changed
  • app
    • code/core/Mage/Adminhtml/Block/Widget
    • design/adminhtml/default/default/template/widget

2 files changed

+10
-5
lines changed

app/code/core/Mage/Adminhtml/Block/Widget/Grid.php

Lines changed: 6 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -1725,4 +1725,10 @@ public function getRowUrl($item)
17251725
$res = parent::getRowUrl($item);
17261726
return ($res ? $res : '#');
17271727
}
1728+
1729+
public function getLimitOptions(): array
1730+
{
1731+
return [20, 30, 50, 100, 200];
1732+
}
1733+
17281734
}

app/design/adminhtml/default/default/template/widget/grid.phtml

Lines changed: 4 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -27,6 +27,7 @@
2727
<?php
2828
/**
2929
* Template for Mage_Adminhtml_Block_Widget_Grid
30+
* @var Mage_Adminhtml_Block_Widget_Grid $this
3031
*
3132
* getId()
3233
* getCollection()
@@ -79,11 +80,9 @@ $numColumns = sizeof($this->getColumns());
7980
<span class="separator">|</span>
8081
<?php echo $this->__('View') ?>
8182
<select name="<?php echo $this->getVarNameLimit() ?>" onchange="<?php echo $this->getJsObjectName() ?>.loadByElement(this)">
82-
<option value="20"<?php if($this->getCollection()->getPageSize()==20): ?> selected="selected"<?php endif ?>>20</option>
83-
<option value="30"<?php if($this->getCollection()->getPageSize()==30): ?> selected="selected"<?php endif ?>>30</option>
84-
<option value="50"<?php if($this->getCollection()->getPageSize()==50): ?> selected="selected"<?php endif ?>>50</option>
85-
<option value="100"<?php if($this->getCollection()->getPageSize()==100): ?> selected="selected"<?php endif ?>>100</option>
86-
<option value="200"<?php if($this->getCollection()->getPageSize()==200): ?> selected="selected"<?php endif ?>>200</option>
83+
<?php foreach ($this->getLimitOptions() as $pageSize): ?>
84+
<option value="<?php echo $pageSize; ?>"<?php if($this->getCollection()->getPageSize() == $pageSize): ?> selected="selected"<?php endif ?>><?php echo $pageSize; ?></option>
85+
<?php endforeach ?>
8786
</select>
8887
<?php echo $this->__('per page') ?><span class="separator">|</span>
8988
<?php echo $this->__('Total %d records found', $this->getCollection()->getSize()) ?>

0 commit comments

Comments
 (0)