fix dncrews#7: date format parsed from query to builder - eslinted - …

…v1.5.5
oguennec · Jul 13, 2016 · eedeef2 · eedeef2
1 parent 923822b
commit eedeef2
Show file tree

Hide file tree

Showing 13 changed files with 257 additions and 119 deletions.
diff --git a/README.md b/README.md
@@ -19,11 +19,14 @@ It's still pretty early on, as it doesn't support a whole lot of use-cases, but
 Notice: this plugin requires:
   - the [Angular Recursion](https://github.com/marklagendijk/angular-recursion) module.
   - the [Angular directives for Bootstrap](https://github.com/angular-ui/bootstrap) module to display the Calendar (ui.bootstrap.datepicker)
+  - the [Angular Moment](https://github.com/urish/angular-moment) module.
 
 ### Installation
 First you'll need to download the [dist](https://github.com/dncrews/angular-elastic-builder/tree/master/dist) files and include this JS file to your app (don't forget to substitute `x.x.x` with the current version number), along with the RecursionHelper, if you're not already using it.
 ```html
 <script type="text/javascript" src="angular-ui-bootstrap/ui-bootstrap-tpls.js"></script>
+<script src="moment/moment.js"></script>
+<script src="angular-moment/angular-moment.js"></script>
 <script type="text/javascript" src="angular-recursion/angular-recursion.min.js"></script>
 <script type="text/javascript" src="angular-elastic-builder/angular-elastic-builder.min.js"></script>
 ```

diff --git a/bower.json b/bower.json
@@ -1,10 +1,22 @@
 {
-  "name": "angular-elastic-builder",
-  "homepage": "https://github.com/dncrews/angular-elastic-builder",
+  "name": "angular-elastic-builder-oguennec",
+  "version": "1.5.5",
+  "description": "Angular Module for building an Elasticsearch Query",
+  "homepage": "https://github.com/oguennec/angular-elastic-builder",
   "authors": [
-    "Dan Crews <[email protected]>"
+    {
+      "name": "Dan Crews",
+      "email": "[email protected]"
+    },
+    {
+      "name": "Olivier Guennec",
+      "email": "[email protected]"
+    }
   ],
-  "description": "Angular Module for building an Elasticsearch Query",
+  "repository": {
+    "type": "git",
+    "url": "https://github.com/oguennec/angular-elastic-builder.git"
+  },
   "main": "dist/angular-elastic-builder.js",
   "keywords": [
     "elasticsearch",
@@ -17,5 +29,12 @@
     "bower_components",
     "test",
     "tests"
-  ]
+  ],
+  "devDependencies": {
+    "angular-moment": "^0.10.3",
+    "angular-recursion": "^1.0.5",
+    "angular-ui-bootstrap": "^1.3.3",
+    "express": "^4.13.4",
+    "moment": "^2.13.0"
+  }
 }
diff --git a/dist/angular-elastic-builder.js b/dist/angular-elastic-builder.js
diff --git a/dist/angular-elastic-builder.min.js b/dist/angular-elastic-builder.min.js
diff --git a/examples/index.html b/examples/index.html
@@ -24,6 +24,8 @@ <h3>Query</h3>
 
     <script src="https://ajax.googleapis.com/ajax/libs/angularjs/1.3.15/angular.min.js"></script>
     <script src="angular-ui-bootstrap/ui-bootstrap-tpls.js"></script>
+    <script src="moment/moment.js"></script>
+    <script src="angular-moment/angular-moment.js"></script>
     <script src="angular-recursion/angular-recursion.min.js"></script>
     <script src="js/angular-elastic-builder.js"></script>
     <script src="js/exampleApp.js"></script>

diff --git a/examples/index.js b/examples/index.js
@@ -7,6 +7,8 @@ app.use(express.static(__dirname));
 app.use('/js', express.static(path.join(__dirname, '../dist')));
 app.use('/angular-recursion', express.static(path.join(__dirname, '../node_modules/angular-recursion')));
 app.use('/angular-ui-bootstrap', express.static(path.join(__dirname, '../node_modules/angular-ui-bootstrap/dist')));
+app.use('/moment', express.static(path.join(__dirname, '../node_modules/moment')));
+app.use('/angular-moment', express.static(path.join(__dirname, '../node_modules/angular-moment')));
 
 app.listen(process.env.PORT || 3000, function() {
   console.log('listening on ' + ( process.env.PORT || 3000));

diff --git a/examples/js/exampleApp.js b/examples/js/exampleApp.js
@@ -13,7 +13,8 @@
         'and': [
           {
             'term': {
-              'test.date': '2016-04-08T09:16:48'
+              "test.date": "2016-04-07T23:59:59",
+              "format": "yyyy-MM-ddTHH:mm:ss"
             }
           },
           {
@@ -25,8 +26,9 @@
           },
           {
             'range': {
-              'test.number': {
-                'lt': 850
+              'test.otherdate': {
+                "lt": "2016/04/07",
+                "format": "yyyy/MM/dd"
               }
             }
           }
@@ -59,8 +61,8 @@
       {
         'range': {
           'test.otherdate': {
-            'gte': 'now',
-            'lte': 'now+7d'
+            "gte": "now-7d",
+            "lte": "now"
           }
         }
       }

diff --git a/package.json b/package.json
@@ -1,18 +1,23 @@
 {
-  "name": "angular-elastic-builder",
-  "version": "1.5.1",
+  "name": "angular-elastic-builder-oguennec",
+  "version": "1.5.5",
   "description": "Angular Module for building an Elasticsearch Query",
   "author": "Dan Crews <[email protected]>",
+  "contributors": [{
+    "name": "Olivier Guennec",
+    "email": "[email protected]"
+  }],
   "license": "MIT",
   "keywords": [
     "angular",
     "elasticsearch"
   ],
   "repository": {
     "type": "git",
-    "url": "https://github.com/dncrews/angular-elastic-builder.git"
+    "url": "https://github.com/oguennec/angular-elastic-builder.git"
   },
   "devDependencies": {
+    "angular-moment": "^0.10.3",
     "angular-recursion": "^1.0.5",
     "angular-ui-bootstrap": "^1.2.5",
     "del": "^1.2.1",
@@ -24,7 +29,8 @@
     "gulp-eslint": "^2.0.0",
     "gulp-header": "^1.2.2",
     "gulp-rename": "^1.2.2",
-    "gulp-uglifyjs": "^0.6.1"
+    "gulp-uglifyjs": "^0.6.1",
+    "moment": "^2.13.0"
   },
   "scripts": {
     "example": "node examples",

diff --git a/src/directives/RuleTypes.js b/src/directives/RuleTypes.js
@@ -13,7 +13,8 @@
 
   app.directive('elasticType', [
 
-    function() {
+    '$filter', 'moment',
+    function($filter, moment) {
       return {
         scope: {
           type: '=elasticType',
@@ -60,18 +61,13 @@
             return ~needs.indexOf(scope.rule.subType);
           };
 
-          scope.today = function() {
-            scope.rule.date = new Date();
-          };
-          scope.today();
-
           scope.clear = function() {
             scope.rule.date = null;
           };
 
           scope.dateOptions = {
             dateDisabled: disabled,
-            formatYear: 'yy',
+            formatYear: 'yyyy',
             maxDate: new Date(2018, 1, 13),
             minDate: new Date(),
             startingDay: 1,
@@ -88,20 +84,45 @@
             scope.popup1.opened = true;
           };
 
-          scope.setDate = function(year, month, day) {
-            scope.rule.date = new Date(year, month - 1, day);
+
+          scope.formatDate = function (date) {
+            if (scope.rule.format) {
+              var inputFormat = scope.rule.format;
+              //moment requires yyyy and DD in upper case but ui.datepicker and Elastic require lower case
+              var momentFormat = inputFormat.replace(/(yyyy|yy|d|dd|ddd)/g, function(str) {
+                return str.toUpperCase();
+              });
+
+              var fDate = moment(date, momentFormat);
+
+              return fDate.toDate(momentFormat);
+
+            }
+          };
+
+          scope.formatDate($filter, scope.rule.value);
+
+          scope.setDate = function() {
+            if (scope.rule.format) {
+              scope.rule.date = scope.formatDate(scope.rule.value);
+              scope.date = scope.rule.date;
+            }
           };
 
+          scope.setDate();
+
           scope.formats = [
             'yyyy-MM-ddTHH:mm:ss',
-            'yyyy-MM-ddTHH:mm:ssZ',
             'yyyy-MM-dd',
+            'yyyyMMdd',
             'dd-MMMM-yyyy',
             'yyyy/MM/dd',
             'shortDate',
           ];
-          scope.rule.dateFormat = scope.formats[0];
-          scope.format = scope.rule.dateFormat;
+
+          if (scope.rule.hasOwnProperty('format')) {
+            scope.format = scope.rule.format;
+          }
 
           scope.altInputFormats = ['M!/d!/yyyy'];
 

diff --git a/src/module.js b/src/module.js
@@ -12,6 +12,8 @@
   angular.module('angular-elastic-builder', [
     'RecursionHelper',
     'ui.bootstrap',
+    'angularMoment',
   ]);
+  //angular.module('angular-elastic-builder').constant('moment', require('moment-timezone'));
 
 })(window.angular);