From 7fa037e75b7fd3f4d8f525ddfb3250d6aa7ca3e0 Mon Sep 17 00:00:00 2001
From: Gil Moskowitz <gmoskowitz@xtuple.com>
Date: Wed, 11 Jun 2014 12:19:32 -0400
Subject: [PATCH] make filenames more descriptive

---
 node-datasource/routes/export.js | 18 +++++++++++-------
 1 file changed, 11 insertions(+), 7 deletions(-)

diff --git a/node-datasource/routes/export.js b/node-datasource/routes/export.js
index 80269c2bf..831497e37 100644
--- a/node-datasource/routes/export.js
+++ b/node-datasource/routes/export.js
@@ -150,21 +150,25 @@ regexp:true, undef:true, strict:true, trailing:true, white:true */
       } else {
         var resultAsCsv,
           filename = "export",
-          type;
+          type,
+          number = result.data && result.data.data && result.data.data[0]
+                && (result.data.data[0].number || result.data.data[0].name),
+          attr = requestDetails.query
+              && requestDetails.query.details
+              && requestDetails.query.details.attr
+          ;
         try {
-          // try to name the file after the record type
           type = requestDetails.type;
-          // suffix() would be better than substring() but doesn't exist here yet
-          filename = type.replace("ListItem", "Export");
+          filename = type.replace("ListItem", "Export")
+                   + (attr && number ? "-" + number : "")
+                   + (attr           ? "-" + attr   : "")
+                   ;
 
         } catch (error) {
           // "export" will have to do.
         }
 
         try {
-          var attr = requestDetails.query
-                   && requestDetails.query.details
-                   && requestDetails.query.details.attr;
           if (attr) {
             resultAsCsv = jsonToCsv(result.data.data[0][attr]);
           } else {
-- 
2.39.2