Perform recommended robust check for custom actions being functions

Author: limzykenneth

src/core/environment.js

@@ -784,7 +784,7 @@ function environment(p5, fn, lifecycles){
     this.windowWidth = getWindowWidth();
     this.windowHeight = getWindowHeight();
     let executeDefault;
-    if (this._customActions.windowResized) {
+    if (typeof this._customActions.windowResized === 'function') {
       executeDefault = this._customActions.windowResized(e);
       if (executeDefault !== undefined && !executeDefault) {
         e.preventDefault();

src/events/acceleration.js

@@ -657,7 +657,7 @@ function acceleration(p5, fn, lifecycles){
       this.deviceOrientation = 'undefined';
     }
 
-    if (this._customActions.deviceMoved) {
+    if (typeof this._customActions.deviceMoved === 'function') {
       if (
         Math.abs(this.accelerationX - this.pAccelerationX) > move_threshold ||
         Math.abs(this.accelerationY - this.pAccelerationY) > move_threshold ||
@@ -667,7 +667,7 @@ function acceleration(p5, fn, lifecycles){
       }
     }
 
-    if (this._customActions.deviceTurned) {
+    if (typeof this._customActions.deviceTurned === 'function') {
       // The angles given by rotationX etc is from range [-180 to 180].
       // The following will convert them to [0 to 360] for ease of calculation
       // of cases when the angles wrapped around.
@@ -742,7 +742,7 @@ function acceleration(p5, fn, lifecycles){
       this.pRotateDirectionZ = rotateDirectionZ;
       this.turnAxis = undefined;
     }
-    if (this._customActions.deviceShaken) {
+    if (typeof this._customActions.deviceShaken === 'function') {
       let accelerationChangeX;
       let accelerationChangeY;
       // Add accelerationChangeZ if acceleration change on Z is needed

src/events/keyboard.js

@@ -670,7 +670,7 @@ function keyboard(p5, fn, lifecycles){
     this._downKeyCodes[e.code] = true;
     this._downKeys[e.key] = true;
 
-    if (this._customActions.keyPressed && !e.charCode) {
+    if (typeof this._customActions.keyPressed === 'function' && !e.charCode) {
       const executeDefault = this._customActions.keyPressed(e);
       if (executeDefault === false) {
         e.preventDefault();
@@ -835,7 +835,7 @@ function keyboard(p5, fn, lifecycles){
    * </div>
    */
   fn._onkeyup = function(e) {
-    if (this._customActions.keyReleased) {
+    if (typeof this._customActions.keyReleased === 'function') {
       const executeDefault = this._customActions.keyReleased(e);
       if (executeDefault === false) {
         e.preventDefault();
@@ -998,7 +998,7 @@ function keyboard(p5, fn, lifecycles){
     this._lastKeyCodeTyped = e.which; // track last keyCode
     this.key = e.key || String.fromCharCode(e.which) || e.which;
 
-    if (this._customActions.keyTyped) {
+    if (typeof this._customActions.keyTyped === 'function') {
       const executeDefault = this._customActions.keyTyped(e);
       if (executeDefault === false) {
         e.preventDefault();

src/events/pointer.js

@@ -6,8 +6,6 @@
  * @requires constants
  */
 
-import * as constants from '../core/constants';
-
 function pointer(p5, fn, lifecycles){
   lifecycles.presetup = function(){
     const events = [
@@ -1336,7 +1334,7 @@ function pointer(p5, fn, lifecycles){
     this._setMouseButton(e);
     this._updatePointerCoords(e);
 
-    if (this._customActions.mousePressed) {
+    if (typeof this._customActions.mousePressed === 'function') {
       executeDefault = this._customActions.mousePressed(e);
       if (executeDefault === false) {
         e.preventDefault();
@@ -1651,7 +1649,7 @@ function pointer(p5, fn, lifecycles){
    * </div>
    */
   fn._onclick = function(e) {
-    if (this._customActions.mouseClicked) {
+    if (typeof this._customActions.mouseClicked === 'function') {
       const executeDefault = this._customActions.mouseClicked(e);
       if (executeDefault === false) {
         e.preventDefault();
@@ -1781,7 +1779,7 @@ function pointer(p5, fn, lifecycles){
    */
 
   fn._ondblclick = function(e) {
-    if (this._customActions.doubleClicked) {
+    if (typeof this._customActions.doubleClicked === 'function') {
       const executeDefault = this._customActions.doubleClicked(e);
       if (executeDefault === false) {
         e.preventDefault();
@@ -1929,7 +1927,7 @@ function pointer(p5, fn, lifecycles){
    */
   fn._onwheel = function(e) {
     this._mouseWheelDeltaY = e.deltaY;
-    if (this._customActions.mouseWheel) {
+    if (typeof this._customActions.mouseWheel === 'function') {
       e.delta = e.deltaY;
       const executeDefault = this._customActions.mouseWheel(e);
       if (executeDefault === false) {