Merge pull request #6820 from JeroenED/bugfix/notification-order
authorPhilipp <admin+Github@philipp.info>
Thu, 7 Mar 2019 13:19:05 +0000 (14:19 +0100)
committerGitHub <noreply@github.com>
Thu, 7 Mar 2019 13:19:05 +0000 (14:19 +0100)
Bugfix/notification order

src/Core/NotificationsManager.php

index 42abc31..f9a286b 100644 (file)
@@ -73,7 +73,7 @@ class NotificationsManager extends BaseObject
 
                $dbFilter = array_merge($filter, ['uid' => local_user()]);
 
-               $stmtNotifies = DBA::select('notify', [], $dbFilter, $order, $params);
+               $stmtNotifies = DBA::select('notify', [], $dbFilter, $params);
 
                if (DBA::isResult($stmtNotifies)) {
                        return $this->_set_extra(DBA::toArray($stmtNotifies));
@@ -429,6 +429,7 @@ class NotificationsManager extends BaseObject
                }
 
                $params = [];
+               $params['order'] = ['date' => 'DESC'];
                $params['limit'] = [$start, $limit];
 
                $stmtNotifies = DBA::select('notify',