Skip to content

Commit 641ba19

Browse files
author
pipeline
committed
v23.2.6 is released
1 parent 1cf5940 commit 641ba19

File tree

126 files changed

+1922
-529
lines changed

Some content is hidden

Large Commits have some content hidden by default. Use the searchbox below for content that may be hidden.

126 files changed

+1922
-529
lines changed

controls/barcodegenerator/CHANGELOG.md

+1-1
Original file line numberDiff line numberDiff line change
@@ -2,7 +2,7 @@
22

33
## [Unreleased]
44

5-
## 23.2.5 (2023-11-23)
5+
## 23.2.6 (2023-11-28)
66

77
### Barcode
88

controls/base/CHANGELOG.md

+8
Original file line numberDiff line numberDiff line change
@@ -2,6 +2,14 @@
22

33
## [Unreleased]
44

5+
## 23.2.6 (2023-11-28)
6+
7+
### Common
8+
9+
#### Bug Fixes
10+
11+
- `#F47826` - Resolved the issue with parsing multiple white space in the template string.
12+
513
## 23.2.4 (2023-11-20)
614

715
### Common

controls/base/package.json

+1-1
Original file line numberDiff line numberDiff line change
@@ -46,7 +46,7 @@
4646
},
4747
"devDependencies": {
4848
"@types/chai": "^3.4.28",
49-
"@types/jasmine": "^2.2.29",
49+
"@types/jasmine": "2.8.22",
5050
"@types/jasmine-ajax": "^3.1.27",
5151
"@types/requirejs": "^2.1.26",
5252
"es6-promise": "^3.2.1",

controls/base/releasenotes/README.md

-183
This file was deleted.

controls/base/spec/template.spec.ts

+12-3
Original file line numberDiff line numberDiff line change
@@ -36,6 +36,7 @@ let dsJSONArray: any = [{ name: 'one', info: { id: '01' } }, { name: 'two', info
3636
let dsSubArray: any = [{ name: 'one', items: ['AR Item1', 'AR Item2'] }, { name: 'two', items: ['AR Item1', 'AR Item2'] }];
3737
let dsJSONSubArray: any = [{ name: 'one', info: { id: '01', items: ['AR Item1', 'AR Item2'] } }, { name: 'two', info: { id: '02', items: ['AR Item1', 'AR Item2'] } }];
3838
let JSONArray: any = [{ name:'one two', info: { id:'01' } }, { name:'two three', info: { id:'02' } }];
39+
let spaceDS: any = [{ name:'one two', info: { id:'01' } }, { name:'two three ', info: { id:'02' } }];
3940

4041
let tempObj: any;
4142

@@ -148,13 +149,21 @@ describe('Template', () => {
148149
expect(outDOM(template.compile(templateStr), dsJSONArray)).toEqual(result);
149150
});
150151

151-
it('JSON Array Input With multiple sapce between inner text', () => {
152+
it('JSON Array Input With multiple space between inner text', () => {
152153
let templateStr: string = '<div class=" class1 class2 class3">${name}</div>';
153154
let result: Element[] = [];
154155
result.push(createElement('div', { innerHTML: 'one two', className:' class1 class2 class3' }));
155156
result.push(createElement('div', { innerHTML: 'two three', className:' class1 class2 class3' }));
156157
expect(outDOM(template.compile(templateStr), JSONArray)).toEqual(result);
157158
});
159+
160+
it('Multiple space between inner text template string', () => {
161+
let templateStr: string = "<div>${name}</div>";
162+
let result: Element[] = [];
163+
result.push(createElement('div', { innerHTML: 'one two' }));
164+
result.push(createElement('div', { innerHTML: 'two three ' }));
165+
expect(outDOM(template.compile(templateStr), spaceDS)).toEqual(result);
166+
});
158167

159168
it('JSON array input with href value with apostrophe', () => {
160169
let templateStr: string = `<div><a href='https://en.wikipedia.org/wiki/France'>France</a><a href='/Projects/Details?id=VINET'>VINET</a></div>`;
@@ -194,8 +203,8 @@ describe('Template', () => {
194203
</div>`;
195204
/* tslint:enable */
196205
let result: Element[] = [];
197-
result.push(createElement('div', { innerHTML: '<span>one</span>01' }));
198-
result.push(createElement('div', { innerHTML: '<span>two</span>02' }));
206+
result.push(createElement('div', { innerHTML: ' <span>one</span>01 ' }));
207+
result.push(createElement('div', { innerHTML: ' <span>two</span>02 ' }));
199208
expect(outDOM(template.compile(templateStr), dsJSONArray)).toEqual(result);
200209
template.expression(new RegExp('\\${([^}]*)}', 'g'));
201210
});

controls/base/src/template.ts

+2-2
Original file line numberDiff line numberDiff line change
@@ -2,7 +2,7 @@
22
* Template Engine
33
*/
44

5-
const LINES: RegExp = new RegExp('\\n|\\r|\\s\\s+', 'g');
5+
const LINES: RegExp = new RegExp('[\\n\\r\\s]+', 'g');
66
const QUOTES: RegExp = new RegExp(/'|"/g);
77
const IF_STMT: RegExp = new RegExp('if ?\\(');
88
const ELSEIF_STMT: RegExp = new RegExp('else if ?\\(');
@@ -126,7 +126,7 @@ function evalExp(str: string, nameSpace: string, helper?: Object, ignorePrefix?:
126126
});
127127
}
128128

129-
return str.replace(LINES, '').replace(DBL_QUOTED_STR, '\'$1\'').replace(
129+
return str.replace(LINES, ' ').replace(DBL_QUOTED_STR, '\'$1\'').replace(
130130

131131
exp,
132132
// eslint-disable-next-line

controls/calendars/CHANGELOG.md

+8
Original file line numberDiff line numberDiff line change
@@ -2,6 +2,14 @@
22

33
## [Unreleased]
44

5+
## 23.2.6 (2023-11-28)
6+
7+
### Calendar
8+
9+
#### Bug Fixes
10+
11+
- `#I521911` - Fixed an issue where an exception was occurring when changing the culture dynamically.
12+
513
## 23.1.41 (2023-10-17)
614

715
### DateRangePicker

controls/calendars/src/calendar/calendar.ts

+7-1
Original file line numberDiff line numberDiff line change
@@ -1592,8 +1592,14 @@ export class CalendarBase extends Component<HTMLElement> implements INotifyPrope
15921592
this.createContentHeader();
15931593
this.createContentBody();
15941594
}
1595+
if (this.getModuleName() === 'calendar') {
1596+
const l10nLocale: object = { today: 'Today' };
1597+
this.l10 = new L10n(this.getModuleName(), l10nLocale, this.locale);
1598+
}
15951599
this.l10.setLocale(this.locale);
1596-
this.updateFooter();
1600+
if (this.showTodayButton) {
1601+
this.updateFooter();
1602+
}
15971603
break;
15981604
case 'showTodayButton':
15991605
if (newProp.showTodayButton) {

controls/charts/CHANGELOG.md

+10
Original file line numberDiff line numberDiff line change
@@ -2,6 +2,16 @@
22

33
## [Unreleased]
44

5+
## 23.2.6 (2023-11-28)
6+
7+
### Chart
8+
9+
#### Bug Fixes
10+
11+
- `#I520071` - Now, `cluster` selection is working properly in the scatter series.
12+
- `#I522808` - Fixed console error that was thrown when using the name property in the axis for a polar chart.
13+
- `#I523059` - Now, the period selector's selected index is highlighted properly whenever we resize the screen.
14+
515
## 23.2.5 (2023-11-23)
616

717
### AccumulationChart

controls/charts/package.json

+1-1
Original file line numberDiff line numberDiff line change
@@ -1,6 +1,6 @@
11
{
22
"name": "@syncfusion/ej2-charts",
3-
"version": "23.2.4",
3+
"version": "23.2.5",
44
"description": "Feature-rich chart control with built-in support for over 25 chart types, technical indictors, trendline, zooming, tooltip, selection, crosshair and trackball.",
55
"author": "Syncfusion Inc.",
66
"license": "SEE LICENSE IN license",

controls/charts/src/chart/chart.ts

+2-3
Original file line numberDiff line numberDiff line change
@@ -2623,9 +2623,8 @@ export class Chart extends Component<HTMLElement> implements INotifyPropertyChan
26232623
if (this.chartAreaType !== 'PolarRadar') {
26242624
continue;
26252625
}
2626-
if (this.chartAreaType === 'PolarRadar' && ((series.xAxisName === null && series.yAxisName !== null) ||
2627-
(series.xAxisName !== null && series.yAxisName === null) ||
2628-
(series.xAxisName !== null && series.yAxisName !== null))) {
2626+
if (this.chartAreaType === 'PolarRadar' && ((series.xAxisName !== null && (this.primaryXAxis.name !== series.xAxisName)) ||
2627+
(series.yAxisName !== null && (this.primaryYAxis.name !== series.yAxisName)))) {
26292628
continue;
26302629
}
26312630
break;

controls/charts/src/chart/user-interaction/selection.ts

+3-3
Original file line numberDiff line numberDiff line change
@@ -206,7 +206,7 @@ export class Selection extends BaseSelection {
206206
this.findTrackballElements(selectedElements, seriesStyle);
207207
const clusterIndex: number = series.marker.visible && series.isRectSeries ? 2 : 1;
208208
if (!chart.isMultiSelect && selectedElements.length > 0 &&
209-
selectedElements[0].id !== clusters[clusters.length - clusterIndex].id) {
209+
selectedElements[0].id !== (clusters[clusters.length - clusterIndex] ? clusters[clusters.length - clusterIndex].id : '')) {
210210
this.removeSelection(chart, index.series, selectedElements, seriesStyle, true);
211211
}
212212
}
@@ -279,7 +279,7 @@ export class Selection extends BaseSelection {
279279
isElement = (parentNodeId.indexOf('SeriesGroup') > 0 || parentNodeId.indexOf('SymbolGroup') > 0) ? true : false;
280280
}
281281
for (let i: number = 0; i < this.previousSelectedEle.length; i++) {
282-
if (this.previousSelectedEle[i as number].hasAttribute('class')) {
282+
if (this.previousSelectedEle[i as number] && this.previousSelectedEle[i as number].hasAttribute('class')) {
283283
if (this.previousSelectedEle[i as number].getAttribute('class').indexOf('highlight') > -1 &&
284284
(isElement || eventType === 'click')) {
285285
this.previousSelectedEle[i as number].removeAttribute('class');
@@ -411,7 +411,7 @@ export class Selection extends BaseSelection {
411411
pointIndex = chart.isMultiSelect ? this.selectedDataIndexes[i as number].point : index.point;
412412
seriesIndex = series.index;
413413
points = (<Series>series).points;
414-
if (!isNaN(pointIndex)) {
414+
if (!isNaN(pointIndex) && (pointIndex < points.length)) {
415415
yValue = (series.type !== 'RangeArea' || 'SplineRangeArea' || 'RangeStepArea') ? points[pointIndex as number].yValue :
416416
points[pointIndex as number].regions[0].y;
417417
selectedPointX = points[pointIndex as number].xValue;

controls/charts/src/common/period-selector/period-selector.ts

+2-1
Original file line numberDiff line numberDiff line change
@@ -184,7 +184,8 @@ export class PeriodSelector {
184184
});
185185
}
186186
if (!selectedPeriod && this.rootControl.getModuleName() !== 'stockChart') {
187-
this.selectedIndex = this.findSelectedIndex(this.control.startValue, this.control.endValue, buttons);
187+
const selectedIndex: number = this.findSelectedIndex(this.control.startValue, this.control.endValue, buttons);
188+
this.selectedIndex = selectedIndex ? selectedIndex : this.selectedIndex;
188189
}
189190
this.setSelectedStyle(this.selectedIndex);
190191
}

0 commit comments

Comments
 (0)