controllers/message.js - Code Metrics - GroganBurners/ferveo - Measure and Improve Code Quality continuously with Scrutinizer

controllers/message.js A
last analyzed 2017-01-10 23:07 UTC

↳ Parent: Project

Size

Lines of Code

Duplication

Duplicated Lines	0
Ratio	0 %

Importance

Changes	2
Bugs	0	Features	0

Metric	Value
c	2
b	0
f	0
nc	1
dl	0
loc	88
rs	10
noi	10

1 Function

Rating	Name	Duplication	Size	Complexity
A	message.js ➔ ???	0	11	1

const nodemailer = require('nodemailer')

const Router = require('express')

const config = require('../config')

const logger = require('winston')

var rp = require('request-promise')
var ok = require('./utils').ok
var fail = require('./utils').fail

module.exports = class MessageController {
  constructor () {
    this.smtpConfig = {
      host: config.mail.server,

      port: config.mail.port,
      secure: true, // use SSL
      auth: {
        user: config.mail.username,
        pass: config.mail.password
      }
    }
  }

  sendSMS (sms) {
    var data = {
      'username': config.sms.username,

      'password': config.sms.password,
      'function': 'sendSms',
      'number': sms.number,
      'message': sms.message,
      'senderid': 'GrogBurners'
    }

    return rp('https://www.my-cool-sms.com/api-socket.php', { json: true, body: data })
            .then((resp) => {
              if (resp.hasOwnProperty('errorcode')) {
                logger.error('Send SMS failed: ' + resp.description)

                throw new Error('SMS Send failed!')
              } else {
                logger.info('SMS Sent to: ' + JSON.Stringify(resp))
                logger.debug('SMS Sent: ' + JSON.Stringify(resp))
                return { message: 'Message sent' }
              }
            })
        /* .catch((err) => {
            return new Error('Error creating message: ' + err)
        }); */
  }

  sendTestEmail (body) {
    const mailOptions = {
      from: '"' + config.mail.from.name + '" <' + config.mail.from.address + '>', // sender address

      to: body.to, // list of receivers
      subject: body.subject, // Subject line
      text: body.text, // plaintext body
      html: body.html // html body
    }
    return this.send(mailOptions)
  }

  sendEmail (email) {
    const transporter = nodemailer.createTransport(this.smtpConfig)

    return transporter.sendMail(email)
            .then(() => {
              return { message: 'Message sent' }
            })
        /* .catch((err) => {
            return new Error('Error creating message: ' + err)
        }); */
  }

  routeAPI () {
    const router = new Router()

    router.post('/email', (req, res) => {
      this
                .sendEmail(req.body)
                .then(ok(res))
                .then(null, fail(res))
    })

    router.post('/sms', (req, res) => {
      this
                .sendSMS(req.body)
                .then(ok(res))
                .then(null, fail(res))
    })

    return router
  }
}


1			const nodemailer = require('nodemailer')
			0 ignored issues – show Unused Code introduced 2017-01-10 23:07 UTC by Report Bug Copy Issue Report The constant `nodemailer` seems to be never used. Consider removing it. Loading history...
2			const Router = require('express')
			0 ignored issues – show Unused Code introduced 2017-01-10 23:07 UTC by Report Bug Copy Issue Report The constant `Router` seems to be never used. Consider removing it. Loading history...
3			const config = require('../config')
			0 ignored issues – show Unused Code introduced 2017-01-10 23:07 UTC by Report Bug Copy Issue Report The constant `config` seems to be never used. Consider removing it. Loading history...
4			const logger = require('winston')
			0 ignored issues – show Unused Code introduced 2017-01-10 23:07 UTC by Report Bug Copy Issue Report The constant `logger` seems to be never used. Consider removing it. Loading history...
5			var rp = require('request-promise')
6			var ok = require('./utils').ok
7			var fail = require('./utils').fail
8
9			module.exports = class MessageController {
10			constructor () {
11			this.smtpConfig = {
12			host: config.mail.server,
			0 ignored issues – show Bug introduced 2017-01-10 23:07 UTC by Report Bug Copy Issue Report The variable `config` seems to be never declared. If this is a global, consider adding a `/** global: config */` 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...
13			port: config.mail.port,
14			secure: true, // use SSL
15			auth: {
16			user: config.mail.username,
17			pass: config.mail.password
18			}
19			}
20			}
21
22			sendSMS (sms) {
23			var data = {
24			'username': config.sms.username,
			0 ignored issues – show Bug introduced 2017-01-10 23:07 UTC by Report Bug Copy Issue Report The variable `config` seems to be never declared. If this is a global, consider adding a `/** global: config */` 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...
25			'password': config.sms.password,
26			'function': 'sendSms',
27			'number': sms.number,
28			'message': sms.message,
29			'senderid': 'GrogBurners'
30			}
31
32			return rp('https://www.my-cool-sms.com/api-socket.php', { json: true, body: data })
33			.then((resp) => {
34			if (resp.hasOwnProperty('errorcode')) {
35			logger.error('Send SMS failed: ' + resp.description)
			0 ignored issues – show Bug introduced 2017-01-10 23:07 UTC by Report Bug Copy Issue Report The variable `logger` seems to be never declared. If this is a global, consider adding a `/** global: logger */` 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...
36			throw new Error('SMS Send failed!')
37			} else {
38			logger.info('SMS Sent to: ' + JSON.Stringify(resp))
39			logger.debug('SMS Sent: ' + JSON.Stringify(resp))
40			return { message: 'Message sent' }
41			}
42			})
43			/* .catch((err) => {
44			return new Error('Error creating message: ' + err)
45			}); */
46			}
47
48			sendTestEmail (body) {
49			const mailOptions = {
50			from: '"' + config.mail.from.name + '" <' + config.mail.from.address + '>', // sender address
			0 ignored issues – show Bug introduced 2017-01-10 23:07 UTC by Report Bug Copy Issue Report The variable `config` seems to be never declared. If this is a global, consider adding a `/** global: config */` 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			to: body.to, // list of receivers
52			subject: body.subject, // Subject line
53			text: body.text, // plaintext body
54			html: body.html // html body
55			}
56			return this.send(mailOptions)
57			}
58
59			sendEmail (email) {
60			const transporter = nodemailer.createTransport(this.smtpConfig)
			0 ignored issues – show Bug introduced 2017-01-10 23:07 UTC by Report Bug Copy Issue Report The variable `nodemailer` seems to be never declared. If this is a global, consider adding a `/** global: nodemailer */` 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...
61			return transporter.sendMail(email)
62			.then(() => {
63			return { message: 'Message sent' }
64			})
65			/* .catch((err) => {
66			return new Error('Error creating message: ' + err)
67			}); */
68			}
69
70			routeAPI () {
71			const router = new Router()
			0 ignored issues – show Bug introduced 2017-01-10 23:07 UTC by Report Bug Copy Issue Report The variable `Router` seems to be never declared. If this is a global, consider adding a `/** global: Router */` 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...
72			router.post('/email', (req, res) => {
73			this
74			.sendEmail(req.body)
75			.then(ok(res))
76			.then(null, fail(res))
77			})
78
79			router.post('/sms', (req, res) => {
80			this
81			.sendSMS(req.body)
82			.then(ok(res))
83			.then(null, fail(res))
84			})
85
86			return router
87			}
88			}
89

GroganBurners / ferveo

controllers/message.js A last analyzed 2017-01-10 23:07 UTC

Size

Duplication

Importance

1 Function

Duplication Side-by-Side

Filter issues like

controllers/message.js A
last analyzed 2017-01-10 23:07 UTC