@osimatic/helpers-js 1.5.24 → 1.5.25

package/form_date.js

@@ -144,6 +144,9 @@ class FormDate {
 
 	static fillYearSelect(select, nbYearsBefore=5, nbYearsAfter=0) {
 		select = toEl(select);
+		if (!select) {
+			return;
+		}
 		const currentDate = new Date();
 		for (let year=currentDate.getUTCFullYear()-nbYearsBefore; year<=(currentDate.getUTCFullYear()+nbYearsAfter); year++) {
 			select.insertAdjacentHTML('beforeend', '<option value="'+year+'">'+year+'</option>');
@@ -152,6 +155,9 @@ class FormDate {
 
 	static fillMonthSelect(select, locale) {
 		select = toEl(select);
+		if (!select) {
+			return;
+		}
 		for (let month=1; month<=12; month++) {
 			select.insertAdjacentHTML('beforeend', '<option value="'+month+'">'+DateTime.getMonthNameByMonth(month, locale).capitalize()+'</option>');
 		}
@@ -159,6 +165,9 @@ class FormDate {
 
 	static fillDayOfWeekSelect(select, locale) {
 		select = toEl(select);
+		if (!select) {
+			return;
+		}
 		for (let dayOfWeek=1; dayOfWeek<=7; dayOfWeek++) {
 			select.insertAdjacentHTML('beforeend', '<option value="'+dayOfWeek+'">'+DateTime.getDayNameByDayOfWeek(dayOfWeek, locale).capitalize()+'</option>');
 		}

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@osimatic/helpers-js",
-  "version": "1.5.24",
+  "version": "1.5.25",
   "main": "main.js",
   "scripts": {
     "test": "jest",

package/tests/form_date.test.js

@@ -182,6 +182,11 @@ describe('FormDate', () => {
 			mockSelect = { insertAdjacentHTML: jest.fn() };
 		});
 
+		test('should do nothing if element does not exist', () => {
+			expect(() => FormDate.fillYearSelect(null)).not.toThrow();
+			expect(() => FormDate.fillYearSelect(undefined)).not.toThrow();
+		});
+
 		test('should fill select with years from 5 years before to current year', () => {
 			FormDate.fillYearSelect(mockSelect);
 
@@ -259,6 +264,11 @@ describe('FormDate', () => {
 			}
 		});
 
+		test('should do nothing if element does not exist', () => {
+			expect(() => FormDate.fillMonthSelect(null)).not.toThrow();
+			expect(() => FormDate.fillMonthSelect(undefined)).not.toThrow();
+		});
+
 		test('should fill select with 12 months', () => {
 			FormDate.fillMonthSelect(mockSelect, 'fr');
 
@@ -281,6 +291,11 @@ describe('FormDate', () => {
 			}
 		});
 
+		test('should do nothing if element does not exist', () => {
+			expect(() => FormDate.fillDayOfWeekSelect(null)).not.toThrow();
+			expect(() => FormDate.fillDayOfWeekSelect(undefined)).not.toThrow();
+		});
+
 		test('should fill select with 7 days of week', () => {
 			FormDate.fillDayOfWeekSelect(mockSelect, 'fr');