@osimatic/helpers-js 1.5.22 → 1.5.24

package/form_helper.js

@@ -279,9 +279,10 @@ class FormHelper {
 	}
 
 	static hideFormErrors(form) {
+		const wasJQuery = form && form.jquery;
 		form = toEl(form);
 		form.querySelectorAll('div.form_errors').forEach(el => el.remove());
-		return form;
+		return wasJQuery ? toJquery(form) : form;
 	}
 
 	static getFormErrorText(errors) {

package/multiple_action_in_table.js

@@ -7,7 +7,7 @@ class MultipleActionInTable {
 	// Doit être appelé AVANT l'initialisation DataTable.
 	static initCols(table, cellSelector = 'select') {
 		table = toEl(table);
-		if (!table.classList.contains('table-action_multiple')) {
+		if (!table || !table.classList.contains('table-action_multiple')) {
 			return;
 		}
 		if (MultipleActionInTable.getDivBtn(table) == null) {
@@ -31,7 +31,7 @@ class MultipleActionInTable {
 		table = toEl(table);
 		const { cellSelector = 'select', imgArrow = '' } = options;
 
-		if (!table.classList.contains('table-action_multiple')) {
+		if (!table || !table.classList.contains('table-action_multiple')) {
 			return;
 		}
 

package/package.json

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

package/tests/form_helper.test.js

@@ -44,9 +44,15 @@ function addSelect(form, name, options = []) {
 	return select;
 }
 
+// Simule un objet jQuery wrappant un élément DOM natif
+function mockJQuery(el) {
+	return { jquery: '3.0', 0: el, length: 1 };
+}
+
 afterEach(() => {
 	document.body.innerHTML = '';
 	jest.clearAllMocks();
+	delete global.$;
 });
 
 describe('FormHelper', () => {
@@ -89,6 +95,18 @@ describe('FormHelper', () => {
 			expect(onSubmitCallback).toHaveBeenCalledWith(form, customBtn);
 		});
 
+		test('should return jQuery wrapper when passed a jQuery object', () => {
+			const form = setupForm();
+			addButton(form, 'validate', 'Submit');
+			const jqForm = mockJQuery(form);
+			global.$ = jest.fn(el => mockJQuery(el));
+
+			const result = FormHelper.init(jqForm, jest.fn());
+
+			expect(result).toHaveProperty('jquery');
+			expect(result[0]).toBe(form);
+		});
+
 		test('should call buttonLoader with loading on submit', () => {
 			const form = setupForm();
 			const btn = addButton(form, 'validate', 'Submit');
@@ -522,6 +540,17 @@ describe('FormHelper', () => {
 			expect(form.querySelectorAll('div.form_errors').length).toBe(0);
 			expect(result).toBe(form);
 		});
+
+		test('should return jQuery wrapper when passed a jQuery object', () => {
+			const form = setupForm();
+			const jqForm = mockJQuery(form);
+			global.$ = jest.fn(el => mockJQuery(el));
+
+			const result = FormHelper.hideFormErrors(jqForm);
+
+			expect(result).toHaveProperty('jquery');
+			expect(result[0]).toBe(form);
+		});
 	});
 
 	describe('displayFormErrorsFromText', () => {