MAGETWO-63321: Issue when deleting item from Product options grid containing more than 20 options

Author: Sergii Kovalenko

app/code/Magento/Ui/view/base/web/js/dynamic-rows/dynamic-rows.js

@@ -270,6 +270,7 @@ define([
          */
         deleteHandler: function (index, id) {
             var defaultState;
+
             this.setDefaultState();
             defaultState = this.defaultPagesState[this.currentPage()];
             this.processingDeleteRecord(index, id);
@@ -725,6 +726,7 @@ define([
             }
 
             this.initChildren();
+
             return true;
         },
 
@@ -876,7 +878,13 @@ define([
             this._sort();
         },
 
-        _reducePages: function() {
+        /**
+         * Reduce the number of pages
+         *
+         * @private
+         * @return void
+         */
+        _reducePages: function () {
             if (this.pages() < ~~this.currentPage()) {
                 this.currentPage(this.pages());
             }

dev/tests/js/jasmine/tests/app/code/Magento/Ui/base/js/dynamic-rows/dynamic.rows.test.js

@@ -12,18 +12,28 @@ define([
     var model,
         ElementMock = function (index) {
             return {
-                destroy: function () {},
+                destroy: new Function (),
                 index: index
             };
         };
 
-    beforeEach(function(done) {
+    /**
+     * Run before each test method
+     *
+     * @return void
+     */
+    beforeEach(function (done) {
         model = new DynamicRows({});
         done();
     });
 
+    /**
+     * Testing changePage and delete record methods
+     *
+     * @return void
+     */
     describe('Magento_Ui/js/dynamic-rows/dynamic-rows', function () {
-        it('changePage without Records', function() {
+        it('changePage without Records', function () {
             model.recordData = function () {
                 return {
                     length: 0
@@ -33,21 +43,27 @@ define([
             expect(model.changePage(1)).toBeFalsy();
         });
 
-        it('changePage with Fake Page', function() {
+        it('changePage with Fake Page', function () {
             model.pages = function () {
                 return 3;
             };
 
             expect(model.changePage(4)).toBeFalsy();
         });
 
-        it('changePage', function() {
+        it('changePage', function () {
             model.startIndex = 0;
             model.pageSize = 3;
             model.relatedData = [
-                {"a": "b"},
-                {"b": "c"},
-                {"v": "g"}
+                {
+                    'a': 'b'
+                },
+                {
+                    'b': 'c'
+                },
+                {
+                    'v': 'g'
+                }
             ];
 
             model.pages = function () {
@@ -62,7 +78,8 @@ define([
             var elems,
                 recordInstanceMock = new ElementMock(1),
                 elem2 = new ElementMock(2);
-            spyOn(recordInstanceMock, "destroy");
+
+            spyOn(recordInstanceMock, 'destroy');
             model.recordData({1: {}});
             elems = [
                 recordInstanceMock,