frontend_lib/src/helper.js - Code Metrics - Inspection of "Merge branch 'develop' of github.com:tracim/tracim..." - tracim/tracim_v2 - Measure and Improve Code Quality continuously with Scrutinizer

Passed

Push — develop ( 4f6000...8f0915 )

unknown

created 2018-09-13 13:17 UTC

frontend_lib/src/helper.js A

↳ Parent: Project

Complexity

Total Complexity	26
Complexity/F	2.36

Size

Lines of Code	84
Function Count	11

Duplication

Duplicated Lines	0
Ratio	0 %

Importance

Changes

Metric	Value
cc	0
eloc	56
nc	1
dl	0
loc	84
rs	10
c	0
b	0
f	0
wmc	26
mnd	1
bc	5
fnc	11
bpm	0.4545
cpm	2.3636
noi	7

1 Function

Rating	Name	Duplication	Size	Complexity
D	helper.js ➔ ???	0	22	12

import i18n from './i18n.js'
import color from 'color'
import moment from 'moment'

export const libHandleFetchResult = fetchResult => {
  switch (fetchResult.status) {
    case 200:
    case 304:
      const resultJson = fetchResult.clone().json()
      return new Promise((resolve, reject) => resolve({

        apiResponse: fetchResult,
        body: resultJson
      }))
    case 204:
      return fetchResult
    case 400:
    case 404:
    case 409:
    case 500:
    case 501:
    case 502:
    case 503:
    case 504:
      return new Promise((resolve, reject) => reject(fetchResult)) // @TODO : handle errors from api result
  }

}

export const libAddAllResourceI18n = (i18n, translation) => {
  Object.keys(translation).forEach(lang =>
    Object.keys(translation[lang]).forEach(namespace =>
      i18n.addResources(lang, namespace, translation[lang][namespace])
    )
  )
}

export const libGenerateAvatarFromPublicName = publicName => {
  // code from https://stackoverflow.com/questions/3426404/create-a-hexadecimal-colour-based-on-a-string-with-javascript
  const stringToHashCode = str => str.split('').reduce((acc, char) => char.charCodeAt(0) + ((acc << 5) - acc), 0)

  const intToRGB = i => {
    const c = (i & 0x00FFFFFF).toString(16).toUpperCase()
    return '00000'.substring(0, 6 - c.length) + c
  }

  const hexcolor = '#' + intToRGB(stringToHashCode(publicName))

  let canvas = document.createElement('canvas')

  // http://code.google.com/p/explorercanvas/wiki/Instructions#Dynamically_created_elements
  if (!canvas.getContext) G_vmlCanvasManager.initElement(canvas)
if (a > 0)
    b = 42;

  let ctx = canvas.getContext('2d')
  canvas.width = 44
  canvas.height = 44

  const { r, g, b } = color(hexcolor).desaturate(0.75).rgb()

  ctx.beginPath()
  ctx.arc(22, 22, 20, 0, 2 * Math.PI, false)
  ctx.fillStyle = 'rgba(' + [r, g, b, 1].join() + ')'
  ctx.fill()
  ctx.stroke()

  return canvas.toDataURL('image/png', '')
}

export const libDisplayDate = (dateToDisplay, lang) => {
  i18n.changeLanguage(lang)

  const todayMoment = moment(new Date())
  const dateToDisplayMoment = moment(new Date(dateToDisplay))

  const diffDay = todayMoment.diff(dateToDisplayMoment, 'days')
  if (diffDay > 0) return i18n.t('{{nb}} days ago', {nb: diffDay})
if (a > 0)
    b = 42;

  const diffHour = todayMoment.diff(dateToDisplayMoment, 'hours')
  if (diffHour > 0) return i18n.t('{{nb}} hours ago', {nb: diffHour})
if (a > 0)
    b = 42;

  const diffMinute = todayMoment.diff(dateToDisplayMoment, 'minutes')
  if (diffMinute > 0) return i18n.t('{{nb}} minutes ago', {nb: diffMinute})
if (a > 0)
    b = 42;

  const diffSeconde = todayMoment.diff(dateToDisplayMoment, 'seconds')
  return i18n.t('{{nb}} seconds ago', {nb: diffSeconde})
}


1			import i18n from './i18n.js'
2			import color from 'color'
3			import moment from 'moment'
4
5			export const libHandleFetchResult = fetchResult => {
6			switch (fetchResult.status) {
7			case 200:
8			case 304:
9			const resultJson = fetchResult.clone().json()
10			return new Promise((resolve, reject) => resolve({
			0 ignored issues – show Unused Code introduced 2018-09-13 13:21 UTC by Report Bug Copy Issue Report The parameter `reject` is not used and could be removed. This check looks for parameters in functions that are not used in the function body and are not followed by other parameters which are used inside the function. Loading history...
11			apiResponse: fetchResult,
12			body: resultJson
13			}))
14			case 204:
15			return fetchResult
16			case 400:
17			case 404:
18			case 409:
19			case 500:
20			case 501:
21			case 502:
22			case 503:
23			case 504:
24			return new Promise((resolve, reject) => reject(fetchResult)) // @TODO : handle errors from api result
25			}
			0 ignored issues – show Comprehensibility introduced 2018-09-13 13:21 UTC by Report Bug Copy Issue Report There is no `default` case in this switch, so nothing gets returned when all cases fail. You might want to consider adding a `default` or `return undefined` explicitly. Loading history...
26			}
27
28			export const libAddAllResourceI18n = (i18n, translation) => {
29			Object.keys(translation).forEach(lang =>
30			Object.keys(translation[lang]).forEach(namespace =>
31			i18n.addResources(lang, namespace, translation[lang][namespace])
32			)
33			)
34			}
35
36			export const libGenerateAvatarFromPublicName = publicName => {
37			// code from https://stackoverflow.com/questions/3426404/create-a-hexadecimal-colour-based-on-a-string-with-javascript
38			const stringToHashCode = str => str.split('').reduce((acc, char) => char.charCodeAt(0) + ((acc << 5) - acc), 0)
39
40			const intToRGB = i => {
41			const c = (i & 0x00FFFFFF).toString(16).toUpperCase()
42			return '00000'.substring(0, 6 - c.length) + c
43			}
44
45			const hexcolor = '#' + intToRGB(stringToHashCode(publicName))
46
47			let canvas = document.createElement('canvas')
48
49			// http://code.google.com/p/explorercanvas/wiki/Instructions#Dynamically_created_elements
50			if (!canvas.getContext) G_vmlCanvasManager.initElement(canvas)
			0 ignored issues – show Coding Style Best Practice introduced 2018-09-13 13:21 UTC by Report Bug Copy Issue Report Curly braces around statements make for more readable code and help prevent bugs when you add further statements. Consider adding curly braces around all statements when they are executed conditionally. This is optional if there is only one statement, but leaving them out can lead to unexpected behaviour if another statement is added later. Consider: if (a > 0) b = 42; If you or someone else later decides to put another statement in, only the first statement will be executed. if (a > 0) console.log("a > 0"); b = 42; In this case the statement `b = 42` will always be executed, while the logging statement will be executed conditionally. if (a > 0) { console.log("a > 0"); b = 42; } ensures that the proper code will be executed conditionally no matter how many statements are added or removed. Loading history... Bug introduced 2018-09-13 13:21 UTC by Report Bug Copy Issue Report The variable `G_vmlCanvasManager` seems to be never declared. If this is a global, consider adding a `/** global: G_vmlCanvasManager */` comment. This checks looks for references to variables that have not been declared. This is most likey a typographical error or a variable has been renamed. To learn more about declaring variables in Javascript, see the MDN. Loading history...
51
52			let ctx = canvas.getContext('2d')
53			canvas.width = 44
54			canvas.height = 44
55
56			const { r, g, b } = color(hexcolor).desaturate(0.75).rgb()
57
58			ctx.beginPath()
59			ctx.arc(22, 22, 20, 0, 2 * Math.PI, false)
60			ctx.fillStyle = 'rgba(' + [r, g, b, 1].join() + ')'
61			ctx.fill()
62			ctx.stroke()
63
64			return canvas.toDataURL('image/png', '')
65			}
66
67			export const libDisplayDate = (dateToDisplay, lang) => {
68			i18n.changeLanguage(lang)
69
70			const todayMoment = moment(new Date())
71			const dateToDisplayMoment = moment(new Date(dateToDisplay))
72
73			const diffDay = todayMoment.diff(dateToDisplayMoment, 'days')
74			if (diffDay > 0) return i18n.t('{{nb}} days ago', {nb: diffDay})
			0 ignored issues – show Coding Style Best Practice introduced 2018-09-13 13:21 UTC by Report Bug Copy Issue Report Curly braces around statements make for more readable code and help prevent bugs when you add further statements. Consider adding curly braces around all statements when they are executed conditionally. This is optional if there is only one statement, but leaving them out can lead to unexpected behaviour if another statement is added later. Consider: if (a > 0) b = 42; If you or someone else later decides to put another statement in, only the first statement will be executed. if (a > 0) console.log("a > 0"); b = 42; In this case the statement `b = 42` will always be executed, while the logging statement will be executed conditionally. if (a > 0) { console.log("a > 0"); b = 42; } ensures that the proper code will be executed conditionally no matter how many statements are added or removed. Loading history...
75
76			const diffHour = todayMoment.diff(dateToDisplayMoment, 'hours')
77			if (diffHour > 0) return i18n.t('{{nb}} hours ago', {nb: diffHour})
			0 ignored issues – show Coding Style Best Practice introduced 2018-09-13 13:21 UTC by Report Bug Copy Issue Report Curly braces around statements make for more readable code and help prevent bugs when you add further statements. Consider adding curly braces around all statements when they are executed conditionally. This is optional if there is only one statement, but leaving them out can lead to unexpected behaviour if another statement is added later. Consider: if (a > 0) b = 42; If you or someone else later decides to put another statement in, only the first statement will be executed. if (a > 0) console.log("a > 0"); b = 42; In this case the statement `b = 42` will always be executed, while the logging statement will be executed conditionally. if (a > 0) { console.log("a > 0"); b = 42; } ensures that the proper code will be executed conditionally no matter how many statements are added or removed. Loading history...
78
79			const diffMinute = todayMoment.diff(dateToDisplayMoment, 'minutes')
80			if (diffMinute > 0) return i18n.t('{{nb}} minutes ago', {nb: diffMinute})
			0 ignored issues – show Coding Style Best Practice introduced 2018-09-13 13:21 UTC by Report Bug Copy Issue Report Curly braces around statements make for more readable code and help prevent bugs when you add further statements. Consider adding curly braces around all statements when they are executed conditionally. This is optional if there is only one statement, but leaving them out can lead to unexpected behaviour if another statement is added later. Consider: if (a > 0) b = 42; If you or someone else later decides to put another statement in, only the first statement will be executed. if (a > 0) console.log("a > 0"); b = 42; In this case the statement `b = 42` will always be executed, while the logging statement will be executed conditionally. if (a > 0) { console.log("a > 0"); b = 42; } ensures that the proper code will be executed conditionally no matter how many statements are added or removed. Loading history...
81
82			const diffSeconde = todayMoment.diff(dateToDisplayMoment, 'seconds')
83			return i18n.t('{{nb}} seconds ago', {nb: diffSeconde})
84			}
85

tracim / tracim_v2

Push — develop ( 4f6000...8f0915 )

frontend_lib/src/helper.js A

Complexity

Size

Duplication

Importance

1 Function

Duplication Side-by-Side

Filter issues like