From 66be5213333702a1182e6fef73aa15cb45eb04ee Mon Sep 17 00:00:00 2001 From: Rigel Kent Date: Sun, 12 Apr 2020 11:10:44 +0200 Subject: [PATCH] Revert 41a94d0, keep the fix for years --- .../src/app/shared/angular/from-now.pipe.ts | 93 +++++-------------- 1 file changed, 23 insertions(+), 70 deletions(-) diff --git a/client/src/app/shared/angular/from-now.pipe.ts b/client/src/app/shared/angular/from-now.pipe.ts index 61a3c498d..07f114ab6 100644 --- a/client/src/app/shared/angular/from-now.pipe.ts +++ b/client/src/app/shared/angular/from-now.pipe.ts @@ -1,6 +1,5 @@ import { Pipe, PipeTransform } from '@angular/core' import { I18n } from '@ngx-translate/i18n-polyfill' -import { findIndex } from 'lodash-es' // Thanks: https://stackoverflow.com/questions/3177836/how-to-format-time-since-xxx-e-g-4-minutes-ago-similar-to-stack-exchange-site @Pipe({ name: 'myFromNow' }) @@ -8,79 +7,33 @@ export class FromNowPipe implements PipeTransform { constructor (private i18n: I18n) { } - transform (arg: number | Date | string, short = true) { + transform (arg: number | Date | string) { const argDate = new Date(arg) const seconds = Math.floor((Date.now() - argDate.getTime()) / 1000) - let intervals = [ - { - unit: 31536000, // 1 year - singular: (i: number) => this.i18n('{{i}} year', { i }), - plural: (i: number) => this.i18n('{{i}} years', { i }) - }, - { - unit: 2592000, // 1 month - max: 11, - singular: (i: number) => this.i18n('{{i}} month', { i }), - plural: (i: number) => this.i18n('{{i}} months', { i }) - }, - { - unit: 604800, // 1 week - max: 3, - singular: (i: number) => this.i18n('{{i}} week', { i }), - plural: (i: number) => this.i18n('{{i}} weeks', { i }) - }, - { - unit: 86400, // 1 day - max: 6, - singular: (i: number) => this.i18n('{{i}} day', { i }), - plural: (i: number) => this.i18n('{{i}} days', { i }) - }, - { - unit: 3600, // 1 hour - max: 23, - singular: (i: number) => this.i18n('{{i}} hour', { i }), - plural: (i: number) => this.i18n('{{i}} hours', { i }) - }, - { - unit: 60, // 1 min - max: 59, - singular: (i: number) => this.i18n('{{i}} min', { i }), - plural: (i: number) => this.i18n('{{i}} min', { i }) - } - ] - // compute the number of units each unit of time has, store it in "interval" - .map(i => ({ - ...i, - interval: Math.floor(seconds / i.unit) - })) - // compute the number of units each unit of time has, from the remainder of the previous bigger unit, store it in "interval" - .map((i, index, array) => ({ - ...i, - interval: index === 0 - ? i.interval - : Math.floor((seconds - array[index - 1].interval * array[index - 1].unit) / i.unit) - })) - // compute the final string from the "interval", cap it to the max value for the time unit - .map(i => ({ - ...i, - value: (i.interval > 1 ? i.plural : i.singular)(Math.min(i.max, i.interval)) - })) + + let interval = Math.floor(seconds / 31536000) + if (interval > 1) return this.i18n('{{interval}} years ago', { interval }) + if (interval === 1) return this.i18n('{{interval}} year ago', { interval }) - // only keep the first two intervals with enough seconds to be considered - const big_interval_index = findIndex(intervals, i => i.interval >= 1) - intervals = intervals - .slice(big_interval_index, big_interval_index + 2) - .filter(i => i.interval >= 1) + interval = Math.floor(seconds / 2592000) + if (interval > 1) return this.i18n('{{interval}} months ago', { interval }) + if (interval === 1) return this.i18n('{{interval}} month ago', { interval }) - if (intervals.length === 0) { - return this.i18n('just now') - } + interval = Math.floor(seconds / 604800) + if (interval > 1) return this.i18n('{{interval}} weeks ago', { interval }) + if (interval === 1) return this.i18n('{{interval}} week ago', { interval }) - return intervals.length === 1 || short - ? this.i18n('{{interval}} ago', { interval: intervals[0].value }) - : this.i18n('{{big_interval}} {{small_interval}} ago', { - big_interval: intervals[0].value, - small_interval: intervals[1].value - }) + interval = Math.floor(seconds / 86400) + if (interval > 1) return this.i18n('{{interval}} days ago', { interval }) + if (interval === 1) return this.i18n('{{interval}} day ago', { interval }) + + interval = Math.floor(seconds / 3600) + if (interval > 1) return this.i18n('{{interval}} hours ago', { interval }) + if (interval === 1) return this.i18n('{{interval}} hour ago', { interval }) + + interval = Math.floor(seconds / 60) + if (interval >= 1) return this.i18n('{{interval}} min ago', { interval }) + + return this.i18n('just now') } } -- 2.25.1