Theo Pape

In my case, the issue was this:

https://wp-kama.com/2139/the-rotated-suffix-for-images

Adding this plugin, https://www.ads-software.com/plugins/fix-image-rotation/ fixes the issue… But perhaps that would explain both cases?

Is there a way you can fix that without the need for the plugin, or this is separate from what you are doing with the image?

Seems to be the same issue as this: https://www.ads-software.com/support/topic/wp-offload-media-repeating-offload-url-no-conflict-check-in-s3/

Sorry Phil, I’m not being clear enough. Try this:

Make a form that updates user details.

Inside functions.php add the add_action( ‘profile_update’, ‘profile_update_function’, 10, 1 );

Inside this function, cause a PHP fatal error, such as array_search on a null value. This should then cause the AJAX to fail and the form to freeze with no output/explanation to the user.

add_action( ‘profile_update’, ‘profile_update_function’, 10, 1 );

Is where the code comes from that is causing the error.

I don’t really want to start sharing that. But it’s included in functions.php. So you could just add a similar line of code there.

Hi Phil,

I noticed this when there was a PHP fatal error in one of my project’s files:

PHP Fatal error: Uncaught TypeError: array_search(): Argument #2 ($haystack) must be of type array, null given in …/form-actions.php:407

wp-admin/admin-ajax.php:1

   Failed to load resource: the server responded with a status of 500 (Internal Server Error)

forms.js?ver=1.9.1:1 Uncaught TypeError: Cannot read properties of undefined (reading ‘data’)
at Object.error (forms.js?ver=1.9.1:1:4106)
at c (jquery.min.js?ver=3.7.1:2:25304)
at Object.fireWith as rejectWith
at l (jquery.min.js?ver=3.7.1:2:77807)
at XMLHttpRequest. (jquery.min.js?ver=3.7.1:2:80265)

Points to this:

    onError: function(t) {
        return function(e) {
            var a = t.$el.data("acf")
              , e = e.responseJSON.data.errors;
            a.addErrors(e),
            a.showErrors()
        }
    }

Hope that helps. Would be useful if you could provide some way to hook into this ^^

Sorry, I mean that we realised we could move to using MySQLi just before you said. It’s working, I just wonder if it’s a sub-optimal approach, as you suggest in the docs.

Ah OK, so that would make sense. We could see if there is a way to grant the permissions. Is it the whole folder that is replaced, or the individual files?

Yes we already moved to using MySQL. It seems a bit odd to overwrite the config.php though – isn’t the point of that file to store the config on save?

Thank you. Yes, this is how we have it working.

Hi Phil,

No worries, I know what it’s like.

Yep, just thought I would provide what worked for us, in case others have the same issue – not sure if you really need to change that, as the form load hook is available…

I only had an issue with the currencies field, so far, which is part of pro. Perhaps the plugin author will give you a copy to test against?

Best wishes,
Theo

• This reply was modified 1 year ago by Theo Pape.

OK, thanks Julie. Do we need to run both:

wp redis enable
wp redis update-dropin

or can we just run either of these on their own?

This will fix the issue:

    if( typeof(acf) !== 'undefined' ) {
        acf.addAction('af/form/setup', function (form) {
            $('.acf-field-acfe-currencies select').removeAttr("data-ajax").select2({
                escapeMarkup: function (html) { return html; }
            });
        });
    }

Hi Nikita,

Thank you for your response. I would probably recommend adding that option then – i.e. to have a small delay..?

In the end, we have moved to WP Rocket, which has its own method in terms of delaying until the mouse moves, etc. Gives a 90 score on mobile and 99 on desktop (https://pagespeed.web.dev/analysis/https-www-dronework-io/4xyx1gyquw)

Actually, just thinking that this is not related to Advanced Forms Pro, as only being used on the frontend.

Doesn’t really address the issue…