diff --git a/.gitignore b/.gitignore new file mode 100644 index 0000000..2209f42 --- /dev/null +++ b/.gitignore @@ -0,0 +1,24 @@ +#If ignorance is bliss, then somebody knock the smile off my face + +*.csproj.user +*.suo +*.cache +Thumbs.db +*.DS_Store + +*.bak +*.cache +*.log +*.swp +*.user + +node_modules + + + + + + + + + \ No newline at end of file diff --git a/.jshintrc b/.jshintrc new file mode 100644 index 0000000..cf48aac --- /dev/null +++ b/.jshintrc @@ -0,0 +1,16 @@ +{ + "browser": true + , "devel": true + , "bitwise": true + , "undef": true + , "trailing": true + , "quotmark": false + , "indent": 4 + , "unused": "vars" + , "latedef": "nofunc" + , "globals": { + "module": false, + "exports": false, + "require": false + } +} diff --git a/.travis.yml b/.travis.yml new file mode 100644 index 0000000..b9af4c5 --- /dev/null +++ b/.travis.yml @@ -0,0 +1,4 @@ +language: node_js +sudo: false +node_js: + - "4.2" diff --git a/README.md b/README.md index e8a34a8..ecc6cac 100644 --- a/README.md +++ b/README.md @@ -17,6 +17,8 @@ # under the License. --> +[![Build Status](https://travis-ci.org/apache/cordova-plugin-statusbar.svg?branch=master)](https://travis-ci.org/apache/cordova-plugin-statusbar) + # cordova-plugin-statusbar StatusBar diff --git a/package.json b/package.json index bf5a4fe..c15aed0 100644 --- a/package.json +++ b/package.json @@ -26,6 +26,10 @@ "cordova-wp8", "cordova-windows" ], + "scripts": { + "test": "npm run jshint", + "jshint": "node node_modules/jshint/bin/jshint www && node node_modules/jshint/bin/jshint src && node node_modules/jshint/bin/jshint tests" + }, "engines": [ { "name": "cordova", @@ -33,5 +37,8 @@ } ], "author": "Apache Software Foundation", - "license": "Apache 2.0" + "license": "Apache 2.0", + "devDependencies": { + "jshint": "^2.6.0" + } } diff --git a/src/browser/statusbar.js b/src/browser/statusbar.js index d5637a5..3e9d05e 100644 --- a/src/browser/statusbar.js +++ b/src/browser/statusbar.js @@ -18,8 +18,6 @@ * */ -var cordova = require('cordova'); - function notSupported() { console.log('StatusBar is not supported'); return false; diff --git a/src/windows/StatusBarProxy.js b/src/windows/StatusBarProxy.js index fe18108..1755fb7 100644 --- a/src/windows/StatusBarProxy.js +++ b/src/windows/StatusBarProxy.js @@ -18,16 +18,18 @@ * */ - var _supported = null; // set to null so we can check first time +/* global Windows */ - function isSupported() { - // if not checked before, run check - if (_supported == null) { +var _supported = null; // set to null so we can check first time + +function isSupported() { + // if not checked before, run check + if (_supported === null) { var viewMan = Windows.UI.ViewManagement; _supported = (viewMan.StatusBar && viewMan.StatusBar.getForCurrentView); } return _supported; - } +} function getViewStatusBar() { if (!isSupported()) { @@ -94,14 +96,14 @@ module.exports = { }, show: function (win, fail) { - // added support check so no error thrown, when calling this method + // added support check so no error thrown, when calling this method if (isSupported()) { getViewStatusBar().showAsync().done(win, fail); } }, hide: function (win, fail) { - // added support check so no error thrown, when calling this method + // added support check so no error thrown, when calling this method if (isSupported()) { getViewStatusBar().hideAsync().done(win, fail); } diff --git a/tests/tests.js b/tests/tests.js index b66c5cc..5a8fe39 100644 --- a/tests/tests.js +++ b/tests/tests.js @@ -19,6 +19,9 @@ * */ +/* jshint jasmine: true */ +/* global StatusBar */ + exports.defineAutoTests = function () { describe("StatusBar", function () { it("statusbar.spec.1 should exist", function() { diff --git a/www/statusbar.js b/www/statusbar.js index 7ebca30..afef7dd 100644 --- a/www/statusbar.js +++ b/www/statusbar.js @@ -19,6 +19,8 @@ * */ +/* global cordova */ + var exec = require('cordova/exec'); var namedColors = {