@syncfusion/ej2-grids - npm Package Compare versions

Comparing version 16.1.47 to 16.1.48

CHANGELOG.md

@@ -9,2 +9,14 @@ # Changelog
 
+- Dropdown Editing issue fixed when grid is resides in Dialog
+- Complex object properties missed while batch editing issue fixed.
+- Refreshing grid in edit state while using frozen rows and columns is fixed
+- Sorting in disable state while set dynamic sort settings with descending order issue fixed  
+
+
+## 16.1.47 (2018-06-06)
+
+### Grid
+
+#### Bug Fixes
+
 - Memory leak issue with angular routing fixed.
@@ -11,0 +23,0 @@ - Sort icon misalignment with auto wrap feature.

dist/global/index.d.ts

 /*!
 *  filename: index.d.ts
-*  version : 16.1.47
+*  version : 16.1.48
 *  Copyright Syncfusion Inc. 2001 - 2018. All rights reserved.
@@ -5,0 +5,0 @@ *  Use of this code is subject to the terms of our license.

grid/resize.js

@@ -814,3 +814,5 @@ import { Browser, ChildProperty, EventHandler, Internationalization, closest, compile, createElement, detach, extend, formatUnit, getValue, isNullOrUndefined, merge, setCulture } from '@syncfusion/ej2-base';
         if (edit) {
-            edit.querySelector('colgroup').children[index].style.width = fWidth;
+            if (edit.querySelector('colgroup').children[index]) {
+                edit.querySelector('colgroup').children[index].style.width = fWidth;
+            }
         }
@@ -817,0 +819,0 @@ };

grid/sort.js

@@ -674,5 +674,13 @@ import { Browser, classList, closest, createElement, extend, isNullOrUndefined } from '@syncfusion/ej2-base';
         }
+        this.refreshSortSettings();
         this.removeSortIcons();
         this.addSortIcons();
     };
+    Sort.prototype.refreshSortSettings = function () {
+        var _this = this;
+        this.sortedColumns = [];
+        this.sortSettings.columns.forEach(function (e) {
+            _this.sortedColumns.push(e.field);
+        });
+    };
     /**
@@ -679,0 +687,0 @@ * Clears all the sorted columns of the Grid.

package.json

 {
   "name": "@syncfusion/ej2-grids",
-  "version": "16.1.47",
+  "version": "16.1.48",
   "description": "Essential JS 2 Grid Component",
@@ -15,6 +15,6 @@ "author": "Syncfusion Inc.",
     "@syncfusion/ej2-inputs": "~16.1.45",
-    "@syncfusion/ej2-calendars": "~16.1.45",
+    "@syncfusion/ej2-calendars": "~16.1.48",
     "@syncfusion/ej2-dropdowns": "~16.1.46",
     "@syncfusion/ej2-popups": "~16.1.47",
-    "@syncfusion/ej2-buttons": "~16.1.37",
+    "@syncfusion/ej2-buttons": "~16.1.48",
     "@syncfusion/ej2-lists": "~16.1.42",
@@ -21,0 +21,0 @@ "@syncfusion/ej2-excel-export": "~16.1.37",

src/grid/actions/batch-edit.js

@@ -121,3 +121,3 @@ define(["require", "exports", "@syncfusion/ej2-base", "@syncfusion/ej2-base", "@syncfusion/ej2-base", "../base/enum", "../base/util", "../base/constant", "../renderer/row-renderer", "../renderer/cell-renderer", "../models/cell", "../services/row-model-generator"], function (require, exports, ej2_base_1, ej2_base_2, ej2_base_3, enum_1, util_1, events, row_renderer_1, cell_renderer_1, cell_1, row_model_generator_1) {
             var rows = this.parent.getRowsObject();
-            if (gObj.frozenColumns) {
+            if (gObj.frozenColumns && rows.length < this.parent.currentViewData.length * 2) {
                 rows.push.apply(rows, this.parent.getMovableRowsObject());
@@ -251,9 +251,2 @@ }
                 if (mRow.isDirty) {
-                    var i_1 = 0;
-                    Object.keys(mRow.changes).forEach(function (key) {
-                        if (i_1 < frzCols) {
-                            delete mRow.changes[key];
-                        }
-                        i_1++;
-                    });
                     ej2_base_1.extend(row.changes, mRow.changes);
@@ -516,3 +509,3 @@ }
                     var mRowData = this.parent.getRowObjectFromUID(this.parent.getMovableRows()[index].getAttribute('data-uid'));
-                    rowData = mRowData.changes ? ej2_base_1.extend(mRowData.data, mRowData.changes) : ej2_base_1.extend({}, this.getDataByIndex(index));
+                    rowData = mRowData.changes ? ej2_base_1.extend({}, mRowData.changes) : ej2_base_1.extend({}, this.getDataByIndex(index));
                 }
@@ -582,3 +575,9 @@ else {
             }
-            isComplex ? rowObj.changes[field.split('.')[0]][field.split('.')[1]] = value : rowObj.changes[field] = value;
+            if (isComplex) {
+                rowObj.changes[field.split('.')[0]] = ej2_base_1.extend({}, rowObj.data[field.split('.')[0]]);
+                rowObj.changes[field.split('.')[0]][field.split('.')[1]] = value;
+            }
+            else {
+                rowObj.changes[field] = value;
+            }
             if (rowObj.data[field] !== value) {
@@ -643,3 +642,14 @@ rowObj.isDirty = true;
             var isComplex = !ej2_base_3.isNullOrUndefined(column.field) && util_1.isComplexField(column.field);
-            editedData = ej2_base_1.extend(this.cellDetails.rowData, editedData);
+            var cloneEditedData = ej2_base_1.extend({}, editedData);
+            editedData = ej2_base_1.extend({}, editedData, this.cellDetails.rowData);
+            if (isComplex) {
+                var splits = column.field.split('.');
+                var cloneComplexData = ej2_base_1.extend({}, this.cellDetails.rowData[splits[0]]);
+                editedData[splits[0]] = ej2_base_1.extend({}, editedData[splits[0]], this.cellDetails.rowData[splits[0]]);
+                this.cellDetails.rowData[splits[0]][splits[1]] = cloneComplexData[splits[1]];
+                editedData[splits[0]][splits[1]] = cloneEditedData[splits[0]][splits[1]];
+            }
+            else {
+                editedData[column.field] = cloneEditedData[column.field];
+            }
             var args = {
@@ -646,0 +656,0 @@ columnName: column.field,

src/grid/actions/sort.d.ts

@@ -53,2 +53,3 @@ import { SortSettings } from '../base/grid';
     onPropertyChanged(e: NotifyArgs): void;
+    private refreshSortSettings();
     /**
@@ -55,0 +56,0 @@ * Clears all the sorted columns of the Grid.

src/grid/actions/sort.js

@@ -115,5 +115,13 @@ define(["require", "exports", "@syncfusion/ej2-base", "@syncfusion/ej2-base", "@syncfusion/ej2-base", "../base/util", "../base/constant", "../services/aria-service"], function (require, exports, ej2_base_1, ej2_base_2, ej2_base_3, util_1, events, aria_service_1) {
             }
+            this.refreshSortSettings();
             this.removeSortIcons();
             this.addSortIcons();
         };
+        Sort.prototype.refreshSortSettings = function () {
+            var _this = this;
+            this.sortedColumns = [];
+            this.sortSettings.columns.forEach(function (e) {
+                _this.sortedColumns.push(e.field);
+            });
+        };
         Sort.prototype.clearSorting = function () {
@@ -120,0 +128,0 @@ var cols = util_1.getActualPropFromColl(this.sortSettings.columns);

src/grid/renderer/dropdown-edit-cell.js

@@ -50,3 +50,3 @@ define(["require", "exports", "@syncfusion/ej2-base", "@syncfusion/ej2-base", "@syncfusion/ej2-dropdowns", "@syncfusion/ej2-data", "../base/util", "../base/util"], function (require, exports, ej2_base_1, ej2_base_2, ej2_dropdowns_1, ej2_data_1, util_1, util_2) {
             var dlgElement = util_2.parentsUntil(this.obj.element, 'e-dialog');
-            if (!ej2_base_1.isNullOrUndefined(dlgElement)) {
+            if (this.parent.editSettings.mode === 'Dialog' && !ej2_base_1.isNullOrUndefined(dlgElement)) {
                 var dlgObj = this.parent.element.querySelector('#' + dlgElement.id).ej2_instances[0];
@@ -53,0 +53,0 @@ args.popup.element.style.zIndex = (dlgObj.zIndex + 1).toString();

src/grid/services/width-controller.js

@@ -69,3 +69,5 @@ define(["require", "exports", "@syncfusion/ej2-base", "@syncfusion/ej2-base", "../base/constant", "../models/column"], function (require, exports, ej2_base_1, ej2_base_2, constant_1, column_1) {
             if (edit) {
-                edit.querySelector('colgroup').children[index].style.width = fWidth;
+                if (edit.querySelector('colgroup').children[index]) {
+                    edit.querySelector('colgroup').children[index].style.width = fWidth;
+                }
             }
@@ -72,0 +74,0 @@ };

New alerts

License Policy Violation

License

This package is not allowed per your license policy. Review the package's license to ensure compliance.

Found 1 instance in 1 package

Fixed alerts

License Policy Violation

License

This package is not allowed per your license policy. Review the package's license to ensure compliance.

Found 1 instance in 1 package

Improved metrics

Total package byte prevSize

35090008

increased by0.05%

Lines of code

327219

increased by0.03%

Dependency changes

• Updated@syncfusion/ej2-buttons@~16.1.48

• Updated@syncfusion/ej2-calendars@~16.1.48