array-reverse-polyfill - npm Package Compare versions

Comparing version 1.0.8 to 1.0.9

index.js

 (function() {
-  if (typeof weex !== 'undefined') { // use in weex
-    if (weex.config.env.osName !== 'iOS') return;
-    if (weex.config.env.osVersion.indexOf('12') !== 0) return;
-  } else {
-    var ua = navigator.userAgent;
-    if (!ua.match(/(iPhone|iPad|iPod)/)) return;
-    var matched = ua.match(/OS ([\d_\.]+) like Mac OS X/);
-    if (!matched || !matched[1]) return;
-    // Match all iOS 12, because we don't know the time when Apple fixed the bug.
-    if (matched[1].indexOf('12') !== 0) return;
+  function buggy() {
+    function detect() {
+      var a = [0, 1];
+      a.reverse();
+      return a[0] === 0;
+    }
+    return detect() || detect();
   }
+  if(!buggy()) return;
   Array.prototype._reverse = Array.prototype.reverse;
@@ -20,5 +18,5 @@ Array.prototype.reverse = function reverse() {
   Object.defineProperties(Array.prototype, {
-      _reverse: nonenum,
-      reverse: nonenum,
+    _reverse: nonenum,
+    reverse: nonenum,
   });
 })();

package.json

 {
   "name": "array-reverse-polyfill",
-  "version": "1.0.8",
+  "version": "1.0.9",
   "description": "Fix IOS 12 Array Reverse Bug.",
@@ -5,0 +5,0 @@ "main": "index.js",

README.md

@@ -0,1 +1,3 @@
+![](https://user-images.githubusercontent.com/159840/45736779-8a4ad180-bc1e-11e8-9022-b58935f026eb.png)
+
 # array-reverse-polyfill
@@ -11,3 +13,3 @@ Fixed Array Reverse Bug On iOS 12. (See the bug: https://stackoverflow.com/questions/52390368/array-state-will-be-cached-in-ios-12-safari-is-bug-or-feature/52392901#52392901)
 ```html
-<script src="//g.alicdn.com/mtb/fix-ios12-array/1.0.8/index.js"></script>
+<script src="//g.alicdn.com/mtb/fix-ios12-array/1.0.9/index.js"></script>
 ```
@@ -14,0 +16,0 @@

No alert changes

Improved metrics

Total package byte prevSize

5194

5.33%

Number of lines in readme file

36

5.88%

Worsened metrics

Lines of code

21

-8.7%

No dependency changes