anilkumarbp / Glipped

app.js

require('dotenv').config();
var express = require('express');
var path = require('path');
var cons = require('consolidate');

var port = 8080;
var app = module.exports = express();
var swig = require('swig');

// handlebars initiation
var exphbs = require('express-handlebars');
var hbs = exphbs.create({});

// Configure View and Handlebars
app.use(express.static(path.join(__dirname)));
app.engine('handlebars', hbs.engine);
app.set('views', path.join(__dirname, 'views'));
app.set('view engine', 'handlebars');



// Create body parsers for application/json and application/x-www-form-urlencoded
var bodyParser = require('body-parser')
app.use(bodyParser.json())
var urlencodedParser = bodyParser.urlencoded({ extended: false })

if(!module.parent){ app.listen(process.env.PORT || port); }

console.log("Application started. Listening on port:" + port);

console.log looks like debug code. Are you sure you do not want to remove it?

var ringcentral = require('ringcentral');

var rcsdk = new ringcentral({

By convention, constructors like ringcentral should be capitalized.

    server: process.env.GLIP_APP_SERVER_URL,
    appKey: process.env.GLIP_APP_KEY,
    appSecret: process.env.GLIP_APP_SECRET
});

var groupMembers;
var userList = [];

    // Route for the home page
    app.get('/inviteDemo', function (req, res) {
        var badge = swig.renderFile(path.join(__dirname, '/views/badge.svg'));
        res.writeHead(200, {"Content-Type": "image/svg+xml"})
        res.write(badge);
        res.send();
    });
    // Route for the inviteDemo
    app.get('/', function (req, res) {

        console.log('Insdei');

console.log looks like debug code. Are you sure you do not want to remove it?

        rcsdk.platform()
            .login({
                username: process.env.GLIP_USERNAME,
                extension: process.env.GLIP_EXTENSION || null,
                password: process.env.GLIP_PASSWORD
            })
            .then(function(response) {

The parameter response is not used and could be removed.

                console.log('Logged in to platform');

console.log looks like debug code. Are you sure you do not want to remove it?

                return rcsdk.platform()
                    .get('/glip/groups/' + process.env.GLIP_GROUP_ID)
                    .then(function(apiResponse) {
                        var response = apiResponse.json();
                        if(response.members){
                            groupMembers = response.members.length;
                            console.log(groupMembers);

console.log looks like debug code. Are you sure you do not want to remove it?

                        }
                        res.render('index', {RC_Logo: process.env.GLIP_LOGO, RC_Community: process.env.GLIP_GROUP_NAME, RC_Total_members: groupMembers, RC_groupId: process.env.GLIP_GROUP_ID});

The variable groupMembers does not seem to be initialized in case response.members on line 66 is false. Are you sure this can never be the case?

                    })
                    .catch(function(e) {
                        console.log('INVITE USER DID NOT WORK');

console.log looks like debug code. Are you sure you do not want to remove it?

                        console.log(e);
                    });
            })
            .catch(function(e) {
                console.log('ERR_CALLBACK ' + e.message  || 'Server cannot authorize user');

console.log looks like debug code. Are you sure you do not want to remove it?

                res.send('Error');
            });
    });

 // Route for the invite user
    app.post('/inviteUser', urlencodedParser, function (req, res, next) {

The parameter next is not used and could be removed.

        var userInvite = [ req.body.userInvite || '' ];
        userList.push(userInvite);
        rcsdk.platform().loggedIn()
            .then(function(status) {

The parameter status is not used and could be removed.

                return rcsdk.platform()
                	.post('/glip/groups/' + process.env.GLIP_GROUP_ID + '/bulk-assign',{
                			"addedPersonEmails": userInvite
                		})
                	.then(function(response) {
                		console.log('The response is :', response.json());

console.log looks like debug code. Are you sure you do not want to remove it?

                        console.log('The total number of users invited to the grpup is :'+ userList.length);
                        console.log('The total number of users registered in the grpup is :'+response.json().members.length);
                        console.log("The type of data is :" + typeof userList.length);
                        res.send(userList.length.toString());
                	})
                    .catch(function(e) {
                        console.log('INVITE USER DID NOT WORK');

console.log looks like debug code. Are you sure you do not want to remove it?

                        console.log(e);
                    });
            })
            .catch(function(e) {

The parameter e is not used and could be removed.

                res.send("E_NOT_LOGGED_IN");
            });
    });


module.exports = {
    app : app
};