Fix ng-admin issue #738

exportToCsv function was failing to correctly (re)load data in a new filtered page. This was due to an incorrect search parameters given to the ReadQueries getAll function. Using $stateParams' search attribute (instead of scope.search()) solves the issue.
marmelab · Dec 7, 2015 · 2801917 · 2801917 · yaoelvon · Jan 25, 2016
1 parent 23a6bac
commit 2801917
Showing 1 changed file with 2 additions and 3 deletions.
diff --git a/src/javascripts/ng-admin/Crud/button/maExportToCsvButton.js b/src/javascripts/ng-admin/Crud/button/maExportToCsvButton.js
@@ -4,8 +4,7 @@ export default function maExportToCsvButton ($stateParams, Papa, notification, A
         scope: {
             entity: '&',
             label: '@',
-            datastore: '&',
-            search: '&'
+            datastore: '&'
         },
         link: function(scope) {
             scope.label = scope.label || 'Export';
@@ -30,7 +29,7 @@ export default function maExportToCsvButton ($stateParams, Papa, notification, A
                 var nonOptimizedReferencedData;
                 var optimizedReferencedData;
 
-                ReadQueries.getAll(exportView, -1, scope.search(), $stateParams.sortField, $stateParams.sortDir)
+                ReadQueries.getAll(exportView, -1, $stateParams.search, $stateParams.sortField, $stateParams.sortDir)
                     .then(response => {
                         rawEntries = response.data;
                         return rawEntries;