Changes as per Better Code Hub

Author: Venkatesh Katabathula

src/app.js

@@ -8,31 +8,25 @@ import logger from './common/logger';
 import routeConfig from './routes/router-config';
 
 let app = express();
-// Setting up the default headers which we receive when a request comes from a browser
+
+// Using body-parser for evaluating json requests
 app.use(bodyParser.json());
 app.use(bodyParser.urlencoded({
   extended: true
 }));
+
+// Setting up the default headers which we receive when a request comes from a browser
 app.use(function (req, res, next) {
-  logger.debug("Received Request"+req.originalUrl);
+  logger.debug('Received Request' + req.originalUrl);
   res.header('Access-Control-Allow-Origin', '*');
   res.header('Access-Control-Allow-Headers', 'Origin, X-Requested-With, Content-Type, Accept');
   next();
 });
 // Enabling cors for above headers
 app.use(cors());
-// Using body-parser for evaluating json requests
-
 
-app.use('/', (request, response,next) => {
-  // let getData = dbUtil.getData('buu:bakery:product:id:0512');
-  // getData.then((success) => {
-  //     response.end(JSON.stringify(success.value));
-  //   })
-  //   .catch(error => {
-  //     logger.debug('Caught Error :: ' + error);
-  //   });
-  logger.debug("Received Request on :: "+request.originalUrl);
+app.use('/', (request, response, next) => {
+  logger.debug('Received Request on :: ' + request.originalUrl);
   next();
 });
 

src/routes/product-route-handler.js

@@ -13,22 +13,18 @@ router.get('/', (request, response) => {
   logger.info('query fired :: ' + queryString);
   dbUtil.queryDb(N1qlQuery.fromString(queryString))
     .then(dbResponse => {
-      // logger.info('dbResponse ::: ' + JSON.stringify(dbResponse));
       response.json({
         data: dbResponse,
         success: true
       });
-      // logger.info('Response:: ' + JSON.stringify(response));
     })
     .catch(error => {
       logger.debug('Error :: ' + error);
       response.json({
         success: false,
         message: error
       });
-
     });
-  // logger.debug(response);
 });
 
 router.post('/products', (request, response) => {

src/routes/router-config.js

@@ -5,16 +5,4 @@ import express from 'express';
 let router = express.Router();
 
 router.use('/', productRouteHandler);
-export default router;
-
-/*
- export default (request, response) => {
- /!*app.get('/', (request, response) => {
- logger.info('received request ::: ' + request.originalUrl);
- response = {success: true};
- logger.debug(response);
-
- });*!/
- router.get('/', productRouteHandler);
- }
- */
+export default router;