This is an automated email from the ASF dual-hosted git repository. normanbreau pushed a commit to branch master in repository https://gitbox.apache.org/repos/asf/cordova-android.git
The following commit(s) were added to refs/heads/master by this push: new b91639db refactor: Removed obsolete version code checks (#1588) b91639db is described below commit b91639dbb5410617eff7333fdd169958fb460da2 Author: Norman Breau <nor...@nbsolutions.ca> AuthorDate: Sat Apr 8 15:32:19 2023 -0300 refactor: Removed obsolete version code checks (#1588) Now that our Min SDK is 24, testing for >= N (API 24) and >= M (API 22) is obsolete as it will always be true. Simplify the codebase by removing the conditions and keeping only the API 24 or later codepath. --- .../src/org/apache/cordova/CordovaInterfaceImpl.java | 10 +--------- framework/src/org/apache/cordova/PluginManager.java | 19 ++++--------------- 2 files changed, 5 insertions(+), 24 deletions(-) diff --git a/framework/src/org/apache/cordova/CordovaInterfaceImpl.java b/framework/src/org/apache/cordova/CordovaInterfaceImpl.java index 84e2c0a4..eccc9663 100644 --- a/framework/src/org/apache/cordova/CordovaInterfaceImpl.java +++ b/framework/src/org/apache/cordova/CordovaInterfaceImpl.java @@ -237,14 +237,6 @@ public class CordovaInterfaceImpl implements CordovaInterface { public boolean hasPermission(String permission) { - if(Build.VERSION.SDK_INT >= Build.VERSION_CODES.M) - { - int result = activity.checkSelfPermission(permission); - return PackageManager.PERMISSION_GRANTED == result; - } - else - { - return true; - } + return PackageManager.PERMISSION_GRANTED == activity.checkSelfPermission(permission); } } diff --git a/framework/src/org/apache/cordova/PluginManager.java b/framework/src/org/apache/cordova/PluginManager.java index 05029937..9531e8fa 100755 --- a/framework/src/org/apache/cordova/PluginManager.java +++ b/framework/src/org/apache/cordova/PluginManager.java @@ -339,22 +339,11 @@ public class PluginManager { public Object postMessage(String id, Object data) { LOG.d(TAG, "postMessage: " + id); synchronized (this.pluginMap) { - if (Build.VERSION.SDK_INT >= Build.VERSION_CODES.N) { - this.pluginMap.forEach((s, plugin) -> { - if (plugin != null) { - plugin.onMessage(id, data); - } - }); - } else { - for (CordovaPlugin plugin : this.pluginMap.values()) { - if (plugin != null) { - Object obj = plugin.onMessage(id, data); - if (obj != null) { - return obj; - } - } + this.pluginMap.forEach((s, plugin) -> { + if (plugin != null) { + plugin.onMessage(id, data); } - } + }); } return ctx.onMessage(id, data); } --------------------------------------------------------------------- To unsubscribe, e-mail: commits-unsubscr...@cordova.apache.org For additional commands, e-mail: commits-h...@cordova.apache.org