Update parameter names of Rhymix.ajaxForm() for consistency with Rhymix.ajax()

This commit is contained in:
Kijin Sung 2025-06-18 02:09:10 +09:00
parent f09cce5d1a
commit 213b354891

View file

@ -586,21 +586,21 @@ Rhymix.ajax = function(action, params, callback_success, callback_error) {
* Submit a form using AJAX instead of navigating away * Submit a form using AJAX instead of navigating away
* *
* @param HTMLElement form * @param HTMLElement form
* @param function success * @param function callback_success
* @param function error * @param function callback_error
* @return void * @return void
*/ */
Rhymix.ajaxForm = function(form, success, error) { Rhymix.ajaxForm = function(form, callback_success, callback_error) {
const $form = $(form); const $form = $(form);
// Get success and error callback functions. // Get success and error callback functions.
if (typeof success === 'undefined') { if (typeof callback_success === 'undefined') {
success = $form.data('callbackSuccess'); callback_success = $form.data('callbackSuccess');
if (success && $.isFunction(success)) { if (callback_success && typeof callback_success === 'function') {
// no-op // no-op
} else if (success && window[success] && $.isFunction(window[success])) { } else if (callback_success && window[callback_success] && typeof window[callback_success] === 'function') {
success = window[success]; callback_success = window[callback_success];
} else { } else {
success = function(data) { callback_success = function(data) {
if (data.message && data.message !== 'success') { if (data.message && data.message !== 'success') {
alert(data.message); alert(data.message);
} }
@ -610,17 +610,17 @@ Rhymix.ajaxForm = function(form, success, error) {
}; };
} }
} }
if (typeof error === 'undefined') { if (typeof callback_error === 'undefined') {
error = $form.data('callbackError'); callback_error = $form.data('callbackError');
if (error && $.isFunction(error)) { if (callback_error && typeof callback_error === 'function') {
// no-op // no-op
} else if (error && window[error] && $.isFunction(window[error])) { } else if (callback_error && window[callback_error] && typeof window[callback_error] === 'function') {
error = window[error]; callback_error = window[callback_error];
} else { } else {
error = null; callback_error = null;
} }
} }
this.ajax(null, new FormData($form[0]), success, error); this.ajax(null, new FormData($form[0]), callback_success, callback_error);
}; };
/** /**