Merge pull request #9 from ndu2/develop

new gtt for invoices

Author: ndu2

documentation.md

@@ -282,7 +282,7 @@ Defaults to `table`. `csv` and `markdown` can be printed to stdout, `pdf` and `x
 There are additional options for the invoice output as given in the following example:
 
 ```shell
-gtt report --output=invoice --file=invoice.md --from 2021-02-01 --to 2021-02-28 --closed  --invoiceCurrencyMaxUnit 1 --invoiceTitle "Rechnung" --invoiceAddress "Firma" "Mr. X" "Strasse" "10000 Ort" "Land" --invoiceCurrency "EUR" --invoiceCurrencyPerHour "50" --invoiceVAT "0.15" --invoiceDate "1.03.2021"
+gtt report --output=invoice --file=invoice.md --from 2021-02-01 --to 2021-02-28 --closed  --invoiceCurrencyMaxUnit 1 --invoiceTitle "Rechnung" --invoiceAddress "Firma" "Mr. X" "Strasse" "10000 Ort" "Land" --invoiceCurrency "EUR" --invoiceCurrencyPerHour "50" --invoiceVAT "0.15" --invoiceDate "1.03.2021" --invoicePositionText "Position Text"
 ```
 
 For paper invoice, further process the output with a css, see the folder preview (styles.css, invoice.pdf)
@@ -658,7 +658,6 @@ invoiceSettings:
  opening:
  - Satz 1.
  - Satz 2.
- positionText: Positionstext
  closing:
  - Grussformel
  - 

src/gtt-config.js

@@ -13,4 +13,4 @@ if (program.opts().local) {
     config.assertLocalConfig();
 }
 
-Fs.open(program.opts().local ? config.local : config.global);
+Fs.open(program.opts().local ? config.local : config.global);

src/gtt-list.js

@@ -0,0 +1,39 @@
+const program = require('commander');
+const colors = require('colors');
+const moment = require('moment');
+const Table = require('cli-table');
+
+
+const Config = require('./include/file-config');
+const Cli = require('./include/cli');
+const Tasks = require('./include/tasks');
+
+program
+    .arguments('[project]')
+    .option('--verbose', 'show verbose output')
+    .option('-c, --closed', 'show closed issues (instead of opened only)')
+    .option('--my', 'show only issues assigned to me')
+    .parse(process.argv);
+
+Cli.verbose = program.verbose;
+
+let config = new Config(process.cwd()),
+    tasks = new Tasks(config),
+    type = program.type ? program.type : 'issue',
+    project = program.args[0];
+
+tasks.list(project, program.closed ? 'closed' : 'opened', program.my)
+  .then(issues => {
+    let table = new Table({
+      style : {compact : true, 'padding-left' : 1}
+    });
+    if (issues.length == 0) {
+      console.log("No issues found.");
+    }
+    issues.forEach(issue => {
+      table.push([issue.iid.toString().magenta, issue.title.green + "\n" + issue.data.web_url.gray, issue.state])
+    })
+    console.log(table.toString());
+  })
+  .catch(error => Cli.error(error));
+

src/gtt-log.js

@@ -39,4 +39,4 @@ tasks.log()
             });
         }
     )
-    .catch(error => Cli.error(error));
+    .catch(error => Cli.error(error));

src/gtt-report.js

@@ -72,6 +72,7 @@ program
     .option('--invoiceVAT <number>', 'vat decimal (20% = 0.2)')
     .option('--invoiceDate <number>', 'date string')
     .option('--invoiceCurrencyMaxUnit <number>', 'rouning invoice total, e.g. 0.01, 0.05 or 1')
+    .option('--invoicePositionText <text>', 'invoice position text')
     .parse(process.argv);
 
 // init helpers
@@ -138,6 +139,7 @@ config
     .set('invoiceVAT', program.opts().invoiceVAT)
     .set('invoiceDate', program.opts().invoiceDate)
     .set('invoiceCurrencyMaxUnit', program.opts().invoiceCurrencyMaxUnit)
+    .set('invoicePositionText', program.opts().invoicePositionText)
     .set('_createDump', program.opts().output === 'dump');
 
 // date shortcuts

src/gtt.js

@@ -12,6 +12,7 @@ program
     .command('stop', 'stop monitoring time')
     .command('resume [project]', 'resume monitoring time for last stopped record')
     .command('cancel', 'cancel and discard active monitoring time')
+    .command('list [project]', 'list all open issues')
     .command('log', 'log recorded time records')
     .command('sync', 'sync local time records to GitLab')
     .command('edit [id]', 'edit time record by the given id')

src/models/hasTimes.js

@@ -15,6 +15,7 @@ class hasTimes extends Base {
     constructor(config) {
         super(config);
         this.times = [];
+        this.timesWarnings = [];
     }
 
     /**
@@ -58,6 +59,7 @@ class hasTimes extends Base {
      */
     getTimes() {
         let times = [],
+            timesWarnings = [],
             timeSpent = 0,
             totalTimeSpent = 0,
             timeUsers = {},
@@ -127,18 +129,22 @@ class hasTimes extends Base {
             !(created.isSameOrAfter(moment(this.config.get('from'))) && created.isSameOrBefore(moment(this.config.get('to'))))
             ) return resolve();
 
+            // warn about difference, but do not correct as gitlab API
+            // stats forget the times after an issue is moved to another project.
             let difference = this.data.time_stats.total_time_spent - totalTimeSpent,
                 note = Object.assign({noteable_type: this._typeSingular}, this.data);
-
-            times.unshift(new Time(Time.toHumanReadable(difference, this.config.get('hoursPerDay')), null, note, this, this.config));
-
+            note.timeWarning = {};
+            note.timeWarning['stats'] = this.data.time_stats.total_time_spent;
+            note.timeWarning['notes'] = totalTimeSpent;
+            timesWarnings.push(new Time(Time.toHumanReadable(difference, this.config.get('hoursPerDay')), null, note, this, this.config));
             resolve();
         }));
 
         promise.then(() => {
             _.each(timeUsers, (time, name) => this[`time_${name}`] = Time.toHumanReadable(time, this.config.get('hoursPerDay'), timeFormat));
             this.timeSpent = timeSpent;
-            this.times = times
+            this.times = times;
+            this.timesWarnings = timesWarnings;
         });
 
         return promise;

src/models/issue.js

@@ -29,6 +29,23 @@ class issue extends hasTimes {
         return promise;
     }
 
+    list(project, state, my) {
+      return new Promise((resolve, reject) => {
+        let promise;
+        const query = `scope=${my ? "assigned-to-me" : "all"}&state=${state}`;
+        if (project) {
+          promise = this.get(`projects/${encodeURIComponent(project)}/issues?${query}`);
+        } else {
+          promise = this.get(`issues/?${query}`);
+        }
+        promise.then(response => {
+          const issues = response.body.map(issue => new this.constructor(this.config, issue))
+          resolve(issues)
+        });
+        promise.catch(error => reject(error))
+      })
+    }
+
     /*
      * properties
      */

src/models/report.js

@@ -79,7 +79,10 @@ class report extends Base {
      */
     getMergeRequests() {
         let promise = this.all(`projects/${this.project.id}/merge_requests${this.params()}`);
-        promise.then(mergeRequests => this.mergeRequests = mergeRequests);
+        let excludes = this.config.get('excludeByLabels');
+        promise.then(mergeRequests => this.mergeRequests = mergeRequests.filter(mr => (
+            excludes.filter(l=>mr.labels.includes(l)).length==0 // keep all merge requests not including a exclude label
+            )));
 
         return promise;
     }
@@ -90,8 +93,11 @@ class report extends Base {
      */
     getIssues() {
         let promise = this.all(`projects/${this.project.id}/issues${this.params()}`);
-        promise.then(issues => this.issues = issues);
-
+        let excludes = this.config.get('excludeByLabels');
+        promise.then(issues => this.issues = issues.filter(issue => (
+            issue.moved_to_id == null && // filter moved issues in any case
+            excludes.filter(l=>issue.labels.includes(l)).length==0 // keep all issues not including a exclude label
+            )));
         return promise;
     }
 

src/output/base.js

@@ -38,6 +38,14 @@ class base {
         this.write(this.formats.headline(string));
     }
 
+    /**
+     * print a headline for warnings
+     * @param string
+     */
+    warningHeadline(string) {
+        if (this.config.get('noWarnings')) return;
+        this.headline(string);
+    }
     /**
      * print a warning
      * @param string
@@ -104,6 +112,9 @@ class base {
         let users = {};
         let projects = {};
         let times = [];
+        let timesWarnings = [];
+        let days = {};
+        let daysMoment = {};
 
         let spentFreeLabels = this.config.get('freeLabels');
         if(undefined === spentFreeLabels) {
@@ -113,11 +124,24 @@ class base {
         ['issues', 'mergeRequests'].forEach(type => {
             this.report[type].forEach(issue => {
                 issue.times.forEach(time => {
+                    let dateGrp = time.date.format(this.config.get('dateFormatGroupReport'));
                     if (!users[time.user]) users[time.user] = 0;
                     if (!projects[time.project_namespace]) projects[time.project_namespace] = 0;
+                    if (!days[dateGrp]) {
+                        days[dateGrp] = {}
+                        daysMoment[dateGrp] = time.date;
+                    };
+                    if(!days[dateGrp][time.project_namespace]) {
+                        days[dateGrp][time.project_namespace] = {};
+                    }
+                    if(!days[dateGrp][time.project_namespace][time.iid]) {
+                        days[dateGrp][time.project_namespace][time.iid] = 0;
+                    }
+
 
                     users[time.user] += time.seconds;
                     projects[time.project_namespace] += time.seconds;
+                    days[dateGrp][time.project_namespace][time.iid] += time.seconds;
 
                     spent += time.seconds;
                     //if(time.parent.labels)
@@ -132,6 +156,7 @@ class base {
                     }
                     times.push(time);
                 });
+                issue.timesWarnings.forEach(warning => timesWarnings.push(warning));
 
                 totalEstimate += parseInt(issue.stats.time_estimate);
                 totalSpent += parseInt(issue.stats.total_time_spent);
@@ -152,6 +177,8 @@ class base {
             return a.date.isBefore(b.date) ? 1 : -1;
         });
 
+        this.days = days;
+        this.daysMoment = daysMoment;
         this.users = _.mapObject(users, user => this.config.toHumanReadable(user, 'stats'));
         this.projects = _.mapObject(projects, project => this.config.toHumanReadable(project, 'stats'));
         this.stats = {
@@ -164,6 +191,7 @@ class base {
         this.spent = spent;
         this.spentFree = spentFree;
         this.totalSpent = totalSpent;
+        this.timesWarnings = timesWarnings;
     }
 
     /**