script.js ➔ acceptCookies - Code Metrics - Inspection of "Base component block cookie notification" - flyntwp/flynt-starter-theme - Measure and Improve Code Quality continuously with Scrutinizer

Completed

Pull Request — master (#230)

unknown

created 2017-06-23 15:03 UTC

script.js ➔ acceptCookies A

↳ Parent: Components/BlockCookieNotification/Partials/AcceptButton/script.js

Complexity

Conditions	1
Paths	1

Size

Total Lines

Duplication

Lines	0
Ratio	0 %

Importance

Changes	1
Bugs	0	Features	0

Metric	Value
cc	1
nc	1
nop	3
dl	0
loc	9
rs	9.6666
c	1
b	0
f	0

1 Function

Rating	Name	Duplication	Size	Complexity
A	script.js ➔ ... ➔	0	7	1

import $ from 'jquery'

export function getOptions (optionContainer) {

    return {
      expireDays: 7,
      cookieName: 'cookies_accepted'
    }
  }

export function acceptCookies ($container, cookieName, expireDays) {
  return function (e) {

    const date = new Date()
    date.setTime(date.getTime() + (expireDays*24*60*60*1000))
    document.cookie = cookieName + "=true; expires=" + date.toGMTString()

    $container.remove()
  }
}

export function checkCookies ($container, cookieName) {
    const cookiesAccepted = getCookieValue(cookieName)
    
    if (!cookiesAccepted) {
      $container.addClass('cookieNotification-isVisible')
    } else {
      $container.remove()
    }
  }



function getCookieValue (cookie) {
  const value = "; " + document.cookie
  const parts = value.split('; ' + cookie + '=')
  if (parts.length == 2) {
    return parts.pop().split(';').shift()
  } else {
    return false
  }
}


1			import $ from 'jquery'
2
3			export function getOptions (optionContainer) {
			0 ignored issues – show Unused Code introduced 2017-06-23 15:00 UTC by Report Bug Copy Issue Report The parameter `optionContainer` 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...
4			return {
5			expireDays: 7,
6			cookieName: 'cookies_accepted'
7			}
8			}
9
10			export function acceptCookies ($container, cookieName, expireDays) {
11			return function (e) {
			0 ignored issues – show Unused Code introduced 2017-06-23 15:00 UTC by Report Bug Copy Issue Report The parameter `e` 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...
12			const date = new Date()
13			date.setTime(date.getTime() + (expireDays2460601000))
14			document.cookie = cookieName + "=true; expires=" + date.toGMTString()
15
16			$container.remove()
17			}
18			}
19
20			export function checkCookies ($container, cookieName) {
21			const cookiesAccepted = getCookieValue(cookieName)
22
23			if (!cookiesAccepted) {
24			$container.addClass('cookieNotification-isVisible')
25			} else {
26			$container.remove()
27			}
28			}
29
30
31
32			function getCookieValue (cookie) {
33			const value = "; " + document.cookie
34			const parts = value.split('; ' + cookie + '=')
35			if (parts.length == 2) {
36			return parts.pop().split(';').shift()
37			} else {
38			return false
39			}
40			}
41

flyntwp / flynt-starter-theme

Pull Request — master (#230)

script.js ➔ acceptCookies A

Complexity

Size

Duplication

Importance

1 Function

Duplication Side-by-Side

Filter issues like