googleplaces - npm Package Compare versions

Comparing version 0.0.1 to 0.2.0

index.js

@@ -1,11 +0,22 @@
-var PlaceSearch = require("./lib/PlaceSearch.js");
-var TextSearch = require("./lib/TextSearch.js");
-var PlaceDetailsRequest = require("./lib/PlaceDetailsRequest.js");
+(function () {
+    "use strict";
 
-module.exports = function (apiKey, outputFormat) {
-  return {
-    placeSearch:new PlaceSearch(apiKey, outputFormat),
-    textSearch:new TextSearch(apiKey, outputFormat),
-    placeDetailsRequest:new PlaceDetailsRequest(apiKey, outputFormat)
-  };
-};
+    var PlaceSearch = require("./lib/PlaceSearch.js");
+    var TextSearch = require("./lib/TextSearch.js");
+    var PlaceDetailsRequest = require("./lib/PlaceDetailsRequest.js");
+    var AddEvent = require("./lib/AddEvent.js");
+    var DeleteEvent = require("./lib/DeleteEvent.js");
+    var EventDetails = require("./lib/EventDetails.js");
+
+    module.exports = function (apiKey, outputFormat) {
+        return {
+            placeSearch: new PlaceSearch(apiKey, outputFormat),
+            textSearch: new TextSearch(apiKey, outputFormat),
+            placeDetailsRequest: new PlaceDetailsRequest(apiKey, outputFormat),
+            addEvent: new AddEvent(apiKey, outputFormat),
+            deleteEvent: new DeleteEvent(apiKey, outputFormat),
+            eventDetails: new EventDetails(apiKey, outputFormat)
+        };
+    };
+
+})();

lib/HttpResponseProcessor.js

@@ -1,19 +0,25 @@
-/**
- * Merge the data chunks of response and pass to callback
- *
- * @param callback
- * @return {Function}
- */
-module.exports = function (parseJson, callback) {
-  return function (response) {
-    var responseData = "";
-    response.setEncoding('utf8');
-    response.on("data", function (chunk) {
-      responseData += chunk;
-    });
-    response.on("end", function () {
-      if (parseJson) responseData = JSON.parse(responseData);
-      callback(responseData);
-    });
-  };
-};
+(function () {
+    "use strict";
+
+    /**
+     * Merge the data chunks of response and pass to callback
+     *
+     * @param parseJson
+     * @param callback
+     * @return {Function}
+     */
+    module.exports = function (parseJson, callback) {
+        return function (response) {
+            var responseData = "";
+            response.setEncoding("utf8");
+            response.on("data", function (chunk) {
+                responseData += chunk;
+            });
+            response.on("end", function () {
+                if (parseJson) responseData = JSON.parse(responseData);
+                callback(null, responseData);
+            });
+        };
+    };
+
+})();

lib/PlaceDetailsRequest.js

@@ -1,24 +0,29 @@
-var querystring = require("querystring");
-var https = require("https");
+(function () {
+    "use strict";
 
-var HttpResponseProcessor = require("./HttpResponseProcessor.js");
-var validate = require("./validate.js");
+    var querystring = require("querystring");
+    var https = require("https");
 
-module.exports = function (apiKey, outputFormat) {
-  return function (parameters, callback) {
-    validate.apiKey(apiKey);
-    validate.outputFormat(outputFormat);
-    parameters.key = apiKey;
-    parameters.sensor = parameters.sensor || false;
-    if (!parameters.reference) throw new Error("Place reference is required")
-    var options = {
-      hostname:"maps.googleapis.com",
-      path:"/maps/api/place/details/" + outputFormat + "?" + querystring.stringify(parameters)
+    var HttpResponseProcessor = require("./HttpResponseProcessor.js");
+    var validate = require("./validate.js");
+
+    module.exports = function (apiKey, outputFormat) {
+        return function (parameters, callback) {
+            validate.apiKey(apiKey);
+            validate.outputFormat(outputFormat);
+            parameters.key = apiKey;
+            parameters.sensor = parameters.sensor || false;
+            if (!parameters.reference && !parameters.placeid) throw new Error("Place reference or placeid is required");
+            var options = {
+                hostname: "maps.googleapis.com",
+                path: "/maps/api/place/details/" + outputFormat + "?" + querystring.stringify(parameters)
+            };
+            var request = https.request(options, new HttpResponseProcessor(outputFormat === "json", callback));
+            request.on("error", function (error) {
+                throw new Error(error);
+            });
+            request.end();
+        };
     };
-    var request = https.request(options, new HttpResponseProcessor(outputFormat === "json", callback));
-    request.on("error", function (error) {
-      throw new Error(error);
-    });
-    request.end();
-  };
-};
+
+})();

lib/PlaceSearch.js

@@ -1,26 +0,31 @@
-var querystring = require("querystring");
-var https = require("https");
+(function () {
+    "use strict";
 
-var HttpResponseProcessor = require("./HttpResponseProcessor.js");
-var validate = require("./validate.js");
+    var querystring = require("querystring");
+    var https = require("https");
 
-module.exports = function (apiKey, outputFormat) {
-  return function (parameters, callback) {
-    validate.apiKey(apiKey);
-    validate.outputFormat(outputFormat);
-    parameters.key = apiKey;
-    parameters.location = parameters.location || "-33.8670522,151.1957362";
-    if (typeof parameters.location === "object") parameters.location = parameters.location.toString();
-    parameters.radius = parameters.radius || 500;
-    parameters.sensor = parameters.sensor || false;
-    var options = {
-      hostname:"maps.googleapis.com",
-      path:"/maps/api/place/search/" + outputFormat + "?" + querystring.stringify(parameters)
+    var HttpResponseProcessor = require("./HttpResponseProcessor.js");
+    var validate = require("./validate.js");
+
+    module.exports = function (apiKey, outputFormat) {
+        return function (parameters, callback) {
+            validate.apiKey(apiKey);
+            validate.outputFormat(outputFormat);
+            parameters.key = apiKey;
+            parameters.location = parameters.location || "-33.8670522,151.1957362";
+            if (typeof parameters.location === "object") parameters.location = parameters.location.toString();
+            parameters.radius = parameters.radius || 500;
+            parameters.sensor = parameters.sensor || false;
+            var options = {
+                hostname: "maps.googleapis.com",
+                path: "/maps/api/place/search/" + outputFormat + "?" + querystring.stringify(parameters)
+            };
+            var request = https.request(options, new HttpResponseProcessor(outputFormat === "json", callback));
+            request.on("error", function (error) {
+                throw new Error(error);
+            });
+            request.end();
+        };
     };
-    var request = https.request(options, new HttpResponseProcessor(outputFormat === "json", callback));
-    request.on("error", function (error) {
-      throw new Error(error);
-    });
-    request.end();
-  };
-};
+
+})();

lib/TextSearch.js

@@ -1,25 +0,30 @@
-var querystring = require("querystring");
-var https = require("https");
+(function () {
+    "use strict";
 
-var HttpResponseProcessor = require("./HttpResponseProcessor.js");
-var validate = require("./validate.js");
+    var querystring = require("querystring");
+    var https = require("https");
 
-module.exports = function (apiKey, outputFormat) {
-  return function (parameters, callback) {
-    validate.apiKey(apiKey);
-    validate.outputFormat(outputFormat);
-    parameters.key = apiKey;
-    parameters.query = parameters.query || "restaurant";
-    parameters.sensor = parameters.sensor || false;
-    if (typeof parameters.location === "object") parameters.location = parameters.location.toString();
-    var options = {
-      hostname:"maps.googleapis.com",
-      path:"/maps/api/place/textsearch/" + outputFormat + "?" + querystring.stringify(parameters)
+    var HttpResponseProcessor = require("./HttpResponseProcessor.js");
+    var validate = require("./validate.js");
+
+    module.exports = function (apiKey, outputFormat) {
+        return function (parameters, callback) {
+            validate.apiKey(apiKey);
+            validate.outputFormat(outputFormat);
+            parameters.key = apiKey;
+            parameters.query = parameters.query || "restaurant";
+            parameters.sensor = parameters.sensor || false;
+            if (typeof parameters.location === "object") parameters.location = parameters.location.toString();
+            var options = {
+                hostname: "maps.googleapis.com",
+                path: "/maps/api/place/textsearch/" + outputFormat + "?" + querystring.stringify(parameters)
+            };
+            var request = https.request(options, new HttpResponseProcessor(outputFormat === "json", callback));
+            request.on("error", function (error) {
+                throw new Error(error);
+            });
+            request.end();
+        };
     };
-    var request = https.request(options, new HttpResponseProcessor(outputFormat === "json", callback));
-    request.on("error", function (error) {
-      throw new Error(error);
-    });
-    request.end();
-  };
-};
+
+})();

lib/validate.js

@@ -1,7 +0,13 @@
-exports.apiKey = function (apiKey) {
-  if (!apiKey) throw new Error("apiKey must not be null");
-};
-exports.outputFormat = function (outputFormat) {
-  var validFormats = ["json", "xml"];
-  if (validFormats.indexOf(outputFormat) === -1)  throw new Error("outputFormat must be 'json' or 'xml'");
-};
+(function () {
+    "use strict";
+
+    exports.apiKey = function (apiKey) {
+        if (!apiKey) throw new Error("apiKey must not be null");
+    };
+
+    exports.outputFormat = function (outputFormat) {
+        var validFormats = ["json", "xml"];
+        if (validFormats.indexOf(outputFormat) === -1)  throw new Error("outputFormat must be 'json' or 'xml'");
+    };
+
+})();

package.json

 {
   "name":"googleplaces",
-  "version":"0.0.1",
+  "version":"0.2.0",
   "description":"Node.js library for the Google Places API",
@@ -12,3 +12,3 @@ "keywords":["google", "places", "api", "google places", "google places api"],
   "author":"Srirangan <[e-mail]> (http://srirangan.net)",
-  "contributors":["", ""],
+  "contributors":["[e-mail] (http://andrew.dalgleish.info)"],
   "main":"./index.js",
@@ -20,4 +20,4 @@ "repository":{
   "scripts": {
-    "test": "sh ./test/test-runner.sh"
+    "test": "sh ./test.sh"
   }
-}
+}

test/config.js

@@ -1,2 +0,7 @@
-exports.apiKey = "AIzaSyDuDuJMT7TINlRl0R3G6SsP8EZOyKubFoY";
-exports.outputFormat = "json";
+(function () {
+    "use strict";
+
+    exports.apiKey = process.env.GOOGLE_API_KEY || "your google api key (server) comes here";
+    exports.outputFormat = "json";
+
+})();

test/GooglePlacesTest.js

@@ -1,41 +0,46 @@
-var assert = new require("assert");
+(function () {
+    "use strict";
 
-var config = require("./config.js");
+    var assert = require("assert");
 
-var GooglePlaces = require("../index.js");
-var googlePlaces = new GooglePlaces(config.apiKey, config.outputFormat);
-var parameters;
+    var config = require("./config.js");
 
-/**
- * Place search - https://developers.google.com/places/documentation/#PlaceSearchRequests
- */
-parameters = {
-  location:[-33.8670522, 151.1957362],
-  types:"doctor"
-};
-googlePlaces.placeSearch(parameters, function (response) {
-  assert.notEqual(response.results.length, 0, "Place search must not return 0 results");
-});
+    var GooglePlaces = require("../index.js");
+    var googlePlaces = new GooglePlaces(config.apiKey, config.outputFormat);
+    var parameters;
 
-/**
- * Text search - https://developers.google.com/places/documentation/#TextSearchRequests
- */
-parameters = {
-  query:"restaurants in dublin"
-};
-googlePlaces.textSearch(parameters, function (response) {
-  assert.notEqual(response.results.length, 0, "Text search must not return 0 results");
-});
+    /**
+     * Place search - https://developers.google.com/places/documentation/#PlaceSearchRequests
+     */
+    parameters = {
+        location: [-33.8670522, 151.1957362],
+        types: "doctor"
+    };
+    googlePlaces.placeSearch(parameters, function (error, response) {
+        assert.notEqual(response.results.length, 0, "Place search must not return 0 results");
+    });
 
-/**
- * Place details requests - https://developers.google.com/places/documentation/#PlaceDetails
- */
-parameters = {
-  location:[-33.8670522, 151.1957362],
-  types:"doctor"
-};
-googlePlaces.placeSearch(parameters, function (response) {
-  googlePlaces.placeDetailsRequest({reference:response.results[0].reference}, function (response) {
-    assert.equal(response.status, "OK", "Place details request response status is OK");
-  });
-});
+    /**
+     * Text search - https://developers.google.com/places/documentation/#TextSearchRequests
+     */
+    parameters = {
+        query: "restaurants in dublin"
+    };
+    googlePlaces.textSearch(parameters, function (error, response) {
+        assert.notEqual(response.results.length, 0, "Text search must not return 0 results");
+    });
+
+    /**
+     * Place details requests - https://developers.google.com/places/documentation/#PlaceDetails
+     */
+    parameters = {
+        location: [-33.8670522, 151.1957362],
+        types: "doctor"
+    };
+    googlePlaces.placeSearch(parameters, function (error, response) {
+        googlePlaces.placeDetailsRequest({reference: response.results[0].reference}, function (error, response) {
+            assert.equal(response.status, "OK", "Place details request response status is OK");
+        });
+    });
+
+})();

test/PlaceDetailsRequestTest.js

@@ -1,19 +0,24 @@
-var assert = new require("assert");
+(function () {
+    "use strict";
 
-var PlaceSearch = require("../lib/PlaceSearch.js");
-var PlaceDetailsRequest = require("../lib/PlaceDetailsRequest.js");
-var config = require("./config.js");
+    var assert = require("assert");
 
-var placeSearch = new PlaceSearch(config.apiKey, config.outputFormat);
-var placeDetailsRequest = new PlaceDetailsRequest(config.apiKey, config.outputFormat);
+    var PlaceSearch = require("../lib/PlaceSearch.js");
+    var PlaceDetailsRequest = require("../lib/PlaceDetailsRequest.js");
+    var config = require("./config.js");
 
-var parameters = {
-  location:[-33.8670522, 151.1957362],
-  types:"doctor"
-};
+    var placeSearch = new PlaceSearch(config.apiKey, config.outputFormat);
+    var placeDetailsRequest = new PlaceDetailsRequest(config.apiKey, config.outputFormat);
 
-placeSearch(parameters, function (response) {
-  placeDetailsRequest({reference:response.results[0].reference}, function (response) {
-    assert.equal(response.status, "OK", "Place details request response status is OK");
-  });
-});
+    var parameters = {
+        location: [-33.8670522, 151.1957362],
+        types: "doctor"
+    };
+
+    placeSearch(parameters, function (error, response) {
+        placeDetailsRequest({reference: response.results[0].reference}, function (error, response) {
+            assert.equal(response.status, "OK", "Place details request response status is OK");
+        });
+    });
+
+})();

test/PlaceSearchTest.js

@@ -1,16 +0,20 @@
-var assert = new require("assert");
+(function () {
+    "use strict";
 
-var PlaceSearch = require("../lib/PlaceSearch.js");
-var config = require("./config.js");
+    var assert = require("assert");
 
-var placeSearch = new PlaceSearch(config.apiKey, config.outputFormat);
+    var PlaceSearch = require("../lib/PlaceSearch.js");
+    var config = require("./config.js");
 
-var parameters = {
-  location:[-33.8670522, 151.1957362],
-  types:"doctor"
-};
+    var placeSearch = new PlaceSearch(config.apiKey, config.outputFormat);
 
-placeSearch(parameters, function (response) {
-//  console.log(response);
-  assert.notEqual(response.results.length, 0, "Place search must not return 0 results");
-});
+    var parameters = {
+        location: [40.7127, -74.0059],
+        types: "doctor"
+    };
+
+    placeSearch(parameters, function (error, response) {
+        assert.notEqual(response.results.length, 0, "Place search must not return 0 results");
+    });
+
+})();

test/TextSearchTest.js

@@ -1,15 +0,20 @@
-var assert = new require("assert");
+(function () {
+    "use strict";
 
-var TextSearch = require("../lib/TextSearch.js");
-var config = require("./config.js");
+    var assert = require("assert");
 
-var textSearch = new TextSearch(config.apiKey, config.outputFormat);
+    var TextSearch = require("../lib/TextSearch.js");
+    var config = require("./config.js");
 
-var parameters = {
-  query:"restaurants in dublin"
-};
+    var textSearch = new TextSearch(config.apiKey, config.outputFormat);
 
-textSearch(parameters, function (response) {
-//  console.log(response);
-  assert.notEqual(response.results.length, 0, "Text search must not return 0 results");
-});
+    var parameters = {
+        query: "restaurants in dublin"
+    };
+
+    textSearch(parameters, function (error, response) {
+        assert.notEqual(response.results.length, 0, "Text search must not return 0 results");
+    });
+
+})();
+

New alerts

Major refactor

Supply chain risk

Package has recently undergone a major refactor. It may be unstable or indicate significant internal changes. Use caution when updating to versions that include significant changes.

Found 1 instance in 1 package

New author

Supply chain risk

A new npm collaborator published a version of the package for the first time. New collaborators are usually benign additions to a project, but do indicate a change to the security surface area of a package.

Found 1 instance in 1 package

Fixed alerts

Non-existent author

Supply chain risk

The package was published by an npm account that no longer exists.

Found 1 instance in 1 package

Dynamic require

Supply chain risk

Dynamic require can indicate the package is performing dangerous or unsafe dynamic code execution.

Found 1 instance in 1 package

Improved metrics

Total package byte prevSize

20239

increased by86.4%

Number of package files

21

increased by40%

Lines of code

382

increased by120.81%

Number of high supply chain risk alerts

0

decreased by-100%

Number of low supply chain risk alerts

0

decreased by-100%

Worsened metrics

Number of medium supply chain risk alerts

8

increased by166.67%

No dependency changes