-
Notifications
You must be signed in to change notification settings - Fork 18
speed up recycle bin and sort by timestamp #763
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Nice, looks mostly good to me :)
@@ -1,20 +1,24 @@ | |||
<?php echo $this->pageHeader('Uuid recycle bin')->render(); ?> | |||
<?php echo $this->pageHeader('Uuid recycle bin')->setSubtitle($this->translate('Page') . $this->elements->getCurrentPageNumber())->render(); ?> |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
missing space between Page
and current page number :)
* <code> | ||
* $uuids = $um->findByTrashed(true); | ||
* foreach($uuids as $uuid) | ||
* $elements = $um->findTrashed(); |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Missing $page
argument
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
* $elements = $um->findTrashed(); | |
* $elements = $um->findTrashed($page); |
@@ -1,20 +1,24 @@ | |||
<?php echo $this->pageHeader('Uuid recycle bin')->render(); ?> | |||
<?php echo $this->pageHeader('Uuid recycle bin')->setSubtitle($this->translate('Page') . $this->elements->getCurrentPageNumber())->render(); ?> |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
<?php echo $this->pageHeader('Uuid recycle bin')->setSubtitle($this->translate('Page') . $this->elements->getCurrentPageNumber())->render(); ?> | |
<?php echo $this->pageHeader('Uuid recycle bin')->setSubtitle($this->translate('Page') . ' ' . $this->elements->getCurrentPageNumber())->render(); ?> |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
Use paginator, so fewer elements get normalized (speeding up page load time) - closes #756
Also sorting by timestamp for more convenient access to currently deleted elements.