kennym / cf7-to-api

LOOKING FOR MAINTAINER. Wordpress plugin to transmit contact form 7 entries to remote API using POST or GET
https://wordpress.org/plugins/cf7-to-api-basic-auth/
52 stars 30 forks source link

Form is not saved #48

Closed VincentSC closed 2 years ago

VincentSC commented 2 years ago

All data under tab “API Integration” is not being saved. I tried different browsers, but all I got was “Contact form saved” and an empty form.

What could be the cause? “max_input_vars” is 1000.

mooksz commented 2 years ago

Same

VincentSC commented 2 years ago

Putting it to 10000 did not work. https://wordpress.org/support/topic/form-is-not-saved/

BurakBoz commented 2 years ago

Same issue here. Nothing is saved any suggestions?

eashman commented 2 years ago

Same for me - it can be fixed temporarily by downgrading to Contact Form 7 v5.5.2 -- it appears some change in 5.5.3 is what broke this.

https://wordpress.org/plugins/contact-form-7/advanced/ and scroll to the bottom to get to older versions of the form plugin.

VincentSC commented 2 years ago

I can confirm this works. As suddenly test-data showed up, I assume the data is actually saved but not shown in the form.

AllCdnBoy commented 2 years ago

WP has a plugin called "Rollback" which made the process of rolling back nice and easy. I think if you haven't tried to re-save the API details, they will come back when you rollback

VincentSC commented 2 years ago

No need for that plugin. By manually uploading the old version, it also rolls back without problems.

I can say that the plugin stops working after an update to 5.5.3. So it needs to be held at 5.5.2 until this is fixed.

AllCdnBoy commented 2 years ago

Doesn't appear to always be the case. The was the first thing I tried and it said "This plugin already exists"

VincentSC commented 2 years ago

Click "Replace current with uploaded"

image

AllCdnBoy commented 2 years ago

I don't have those options. It goes from this: image

to this:

image

VincentSC commented 2 years ago

Weird. No sure why there is a difference, assuming you also have the latest Wordpress version.

About this bug. When I've done my taxes, I'll spend some time to debug the problem. I found another plugin who had the same problem, so it can be done by example. If you cannot wait for my taxes, please be my guest. :)

https://wordpress.org/support/topic/the-plugin-isnt-working-with-new-version-of-cf7/ https://github.com/mariovalney/cf7-to-zapier

Main problem is to get a PR accepted, as maintenance is not really done here

VincentSC commented 2 years ago

Fixed! (yes, finished taxes)

Add one line to includes/class-cf7-api-admin.php on line 82, after add_filter( "wpcf7_contact_form_properties" ,array( $this , "add_sf_properties" ) , 10 , 2 );:

    add_filter( 'wpcf7_pre_construct_contact_form_properties', array( $this, 'add_sf_properties' ), 10, 2 );

That's all.

I'm not at my dev-machine, so cannot easily create a PR. So if you want it fixed now, you need to manually edit that file. I've already contacted the author some days ago, but did not get a response yet.

(all credit goes to @mariovalney, as he figured things out)

mariovalney commented 2 years ago

Thanks for the props @VincentSC :)

Actually the answer is in CF7 release post. The problem is CF7 seems to do not follow semantic versioning and introduced a breaking change.

I've made the PR #49 to fixes it.