Shake gesture detection in PhoneGap / Cordova

shake.js

/*
THIS GIST IS OUT OF DATE AND NOT MONITORED
PLEASE SEE https://github.com/leecrossley/cordova-plugin-shake-detection
*/

var shake = (function () {
	var shake = {},
		watchId = null,
		options = { frequency: 300 },
		previousAcceleration = { x: null, y: null, z: null },
		shakeCallBack = null;
	
	// Start watching the accelerometer for a shake gesture
	shake.startWatch = function (onShake) {
		if (onShake) {
			shakeCallBack = onShake;
		}
		watchId = navigator.accelerometer.watchAcceleration(getAccelerationSnapshot, handleError, options);
	};
	
	// Stop watching the accelerometer for a shake gesture
	shake.stopWatch = function () {
		if (watchId !== null) {
			navigator.accelerometer.clearWatch(watchId);
			watchId = null;
		}
	};
	
	// Gets the current acceleration snapshot from the last accelerometer watch
	function getAccelerationSnapshot() {
		navigator.accelerometer.getCurrentAcceleration(assessCurrentAcceleration, handleError);
	}
	
	// Assess the current acceleration parameters to determine a shake
	function assessCurrentAcceleration(acceleration) {
		var accelerationChange = {};
		if (previousAcceleration.x !== null) {
			accelerationChange.x = Math.abs(previousAcceleration.x, acceleration.x);
			accelerationChange.y = Math.abs(previousAcceleration.y, acceleration.y);
			accelerationChange.z = Math.abs(previousAcceleration.z, acceleration.z);
		}
		if (accelerationChange.x + accelerationChange.y + accelerationChange.z > 30) {
			// Shake detected
			if (typeof (shakeCallBack) === "function") {
				shakeCallBack();
			}
			shake.stopWatch();
			setTimeout(shake.startWatch, 1000);
			previousAcceleration = { 
				x: null, 
				y: null, 
				z: null
			}
		} else {
			previousAcceleration = {
				x: acceleration.x,
				y: acceleration.y,
				z: acceleration.z
			}
		}
	}

	// Handle errors here
	function handleError() {
	}
	
	return shake;
})();

Same issue where shaking stops after first detection. Tried the fix @vijnv suggested, no result.

The maths in this is just plain wrong. To find whether a movement is big enough, you have to know the magnitude of its vector. That's why I forked it.

This gist is out of date and not monitored. Please see the full repo: https://github.com/leecrossley/cordova-plugin-shake-detection

Note also that the repo addresses all the problems highlighted in the comments above.

Any idea how/if this is usable with Ionic? and if so... how? I can't see "shake" when using it.

@MrRhodes make sure the deviceready event has been fired first. Also this gist is out of date and not monitored. Please see the full repo: https://github.com/leecrossley/cordova-plugin-shake-detection

Math.abs(previousAcceleration.x, acceleration.x)
i think this is not right, Math.abs(previousAcceleration.x - acceleration.x) is OK?
i do not know what you want do with this line.