Merge branch 'master' into dependabot/npm_and_yarn/docs/babel/travers…

…e-7.23.2

.all-contributorsrc

@@ -598,8 +598,19 @@
       "contributions": [
         "doc"
       ]
+    },
+    {
+      "login": "ParthBaraiya",
+      "name": "Parth Baraiya",
+      "avatar_url": "https://avatars.githubusercontent.com/u/36261739?v=4",
+      "profile": "https://github.com/ParthBaraiya",
+      "contributions": [
+        "code",
+        "bug"
+      ]
     }
   ],
   "contributorsPerLine": 7,
-  "skipCi": true
+  "skipCi": true,
+  "commitType": "docs"
 }

README.md

@@ -405,7 +405,7 @@ the growing community of `MobX.dart`.
 ## Contributors ✨
 
 <!-- ALL-CONTRIBUTORS-BADGE:START - Do not remove or modify this section -->
-[![All Contributors](https://img.shields.io/badge/all_contributors-62-orange.svg?style=flat-square)](#contributors-)
+[![All Contributors](https://img.shields.io/badge/all_contributors-63-orange.svg?style=flat-square)](#contributors-)
 <!-- ALL-CONTRIBUTORS-BADGE:END -->
 
 Thanks goes to these wonderful people ([emoji key](https://allcontributors.org/docs/en/emoji-key)):
@@ -494,6 +494,7 @@ Thanks goes to these wonderful people ([emoji key](https://allcontributors.org/d
       <td align="center" valign="top" width="14.28%"><a href="https://molchanovsky.com"><img src="https://avatars.githubusercontent.com/u/12999702?v=4?s=64" width="64px;" alt="Sergey Molchanovsky"/><br /><sub><b>Sergey Molchanovsky</b></sub></a><br /><a href="https://github.com/mobxjs/mobx.dart/commits?author=subzero911" title="Code">💻</a></td>
       <td align="center" valign="top" width="14.28%"><a href="https://github.com/ko16a46"><img src="https://avatars.githubusercontent.com/u/100613422?v=4?s=64" width="64px;" alt="ko16a46"/><br /><sub><b>ko16a46</b></sub></a><br /><a href="https://github.com/mobxjs/mobx.dart/commits?author=ko16a46" title="Code">💻</a></td>
       <td align="center" valign="top" width="14.28%"><a href="https://github.com/yatharth25"><img src="https://avatars.githubusercontent.com/u/54071856?v=4?s=64" width="64px;" alt="Yatharth Chauhan"/><br /><sub><b>Yatharth Chauhan</b></sub></a><br /><a href="https://github.com/mobxjs/mobx.dart/commits?author=yatharth25" title="Documentation">📖</a></td>
+      <td align="center" valign="top" width="14.28%"><a href="https://github.com/ParthBaraiya"><img src="https://avatars.githubusercontent.com/u/36261739?v=4?s=64" width="64px;" alt="Parth Baraiya"/><br /><sub><b>Parth Baraiya</b></sub></a><br /><a href="https://github.com/mobxjs/mobx.dart/commits?author=ParthBaraiya" title="Code">💻</a> <a href="https://github.com/mobxjs/mobx.dart/issues?q=author%3AParthBaraiya" title="Bug reports">🐛</a></td>
     </tr>
   </tbody>
 </table>

mobx/CHANGELOG.md

@@ -1,3 +1,7 @@
+## 2.2.1
+
+- Reduces unnecessary iterations while using `Iterables` with `addAll`, `insertAll`, `replaceRange`, `setAll` and `setRange` methods. [#942](https://github.com/mobxjs/mobx.dart/pull/942).
+
 ## 2.2.0
 
 - Allows a reaction to be fired even if the value hasn't changed by [@amondnet](https://github.com/amondnet) in [#907](https://github.com/mobxjs/mobx.dart/pull/907)

mobx/lib/src/api/observable_collections/observable_list.dart

@@ -115,10 +115,13 @@ class ObservableList<T>
   @override
   void addAll(Iterable<T> iterable) {
     _context.conditionallyRunInAction(() {
-      if (iterable.isNotEmpty) {
+      final list = iterable.toList(growable: false);
+
+      if (list.isNotEmpty) {
         final index = _list.length;
-        _list.addAll(iterable);
-        _notifyRangeUpdate(index, iterable.toList(growable: false), null);
+
+        _list.addAll(list);
+        _notifyRangeUpdate(index, list, null);
       }
     }, _atom);
   }
@@ -222,9 +225,11 @@ class ObservableList<T>
   @override
   void insertAll(int index, Iterable<T> iterable) {
     _context.conditionallyRunInAction(() {
-      if (iterable.isNotEmpty) {
-        _list.insertAll(index, iterable);
-        _notifyRangeUpdate(index, iterable.toList(growable: false), null);
+      final list = iterable.toList(growable: false);
+
+      if (list.isNotEmpty) {
+        _list.insertAll(index, list);
+        _notifyRangeUpdate(index, list, null);
       }
     }, _atom);
   }
@@ -303,11 +308,13 @@ class ObservableList<T>
   @override
   void replaceRange(int start, int end, Iterable<T> newContents) {
     _context.conditionallyRunInAction(() {
-      if (end > start || newContents.isNotEmpty) {
+      final list = newContents.toList(growable: false);
+
+      if (end > start || list.isNotEmpty) {
         final oldContents = _list.sublist(start, end);
-        _list.replaceRange(start, end, newContents);
-        _notifyRangeUpdate(
-            start, newContents.toList(growable: false), oldContents);
+
+        _list.replaceRange(start, end, list);
+        _notifyRangeUpdate(start, list, oldContents);
       }
     }, _atom);
   }
@@ -333,10 +340,11 @@ class ObservableList<T>
   @override
   void setAll(int index, Iterable<T> iterable) {
     _context.conditionallyRunInAction(() {
-      if (iterable.isNotEmpty) {
-        final oldValues = _list.sublist(index, index + iterable.length);
-        final newValues = iterable.toList(growable: false);
-        _list.setAll(index, iterable);
+      final newValues = iterable.toList(growable: false);
+
+      if (newValues.isNotEmpty) {
+        final oldValues = _list.sublist(index, index + newValues.length);
+        _list.setAll(index, newValues);
         _notifyRangeUpdate(index, newValues, oldValues);
       }
     }, _atom);
@@ -347,9 +355,12 @@ class ObservableList<T>
     _context.conditionallyRunInAction(() {
       if (end > start) {
         final oldValues = _list.sublist(start, end);
+
         final newValues =
             iterable.skip(skipCount).take(end - start).toList(growable: false);
-        _list.setRange(start, end, iterable, skipCount);
+
+        _list.setRange(start, end, newValues);
+
         _notifyRangeUpdate(start, newValues, oldValues);
       }
     }, _atom);

mobx/lib/version.dart

@@ -1,4 +1,4 @@
 // Generated via set_version.dart. !!!DO NOT MODIFY BY HAND!!!
 
 /// The current version as per `pubspec.yaml`.
-const version = '2.2.0';
+const version = '2.2.1';

mobx/pubspec.yaml

@@ -1,5 +1,5 @@
 name: mobx
-version: 2.2.0
+version: 2.2.1
 description: "MobX is a library for reactively managing the state of your applications. Use the power of observables, actions, and reactions to supercharge your Dart and Flutter apps."
 
 homepage: https://github.com/mobxjs/mobx.dart

mobx/test/observable_list_test.dart

@@ -811,6 +811,88 @@ void main() {
       'reversed': (_) => _.reversed
     }.forEach(_templateIterableReadTest);
   });
+
+  group('checks for iterable use with ObservableList', () {
+    test('ObservableList addAll runs Iterable only once', () {
+      final list = ObservableList<String>();
+      var op = '';
+
+      list.addAll([1, 2].map((e) {
+        op += '$e';
+
+        return '$e';
+      }));
+
+      expect(op, '12');
+      expect(list.join(''), '12');
+    });
+
+    test('ObservableList insertAll runs Iterable only once', () {
+      final list = ObservableList<String>.of(['1']);
+      var op = '';
+
+      list.insertAll(
+          1,
+          [2, 3].map((e) {
+            op += '$e';
+
+            return '$e';
+          }));
+
+      expect(op, '23');
+      expect(list.join(''), '123');
+    });
+
+    test('ObservableList replaceRange runs Iterable only once', () {
+      final list = ObservableList<String>.of(['1', '2', '3']);
+      var op = '';
+
+      list.replaceRange(
+          0,
+          2,
+          [4, 5].map((e) {
+            op += '$e';
+
+            return '$e';
+          }));
+
+      expect(op, '45');
+      expect(list.join(''), '453');
+    });
+
+    test('ObservableList setAll runs Iterable only once', () {
+      final list = ObservableList<String>.of(['1', '2', '3']);
+      var op = '';
+
+      list.setAll(
+          0,
+          [4, 5, 6].map((e) {
+            op += '$e';
+
+            return '$e';
+          }));
+
+      expect(op, '456');
+      expect(list.join(''), '456');
+    });
+
+    test('ObservableList setRange runs Iterable only once', () {
+      final list =
+          ObservableList<String>.of(['1', '2', '3', '4', '5', '6', '7']);
+      var op = '';
+
+      list.setRange(
+          1,
+          4,
+          [8, 9, 10].map((e) {
+            op += '$e';
+
+            return '$e';
+          }));
+
+      expect(op, '8910');
+    });
+  });
 }
 
 dynamic _ignoreException(Function fn) {