[Résolu] CRED 1.9.4 brake the functionality of cred_save_data

[Auteur]

Publications

[davidZ-4]

I am trying to: update cred to 1.9.4

Link to a page where the issue can be seen: lien caché

I expected to see: the calculation and chart display correctly as before CRED update to version1.9.4

nnstead, I got: the calculation that worked perfectly before updating cred, (ung the cred_save_data api) ow cause the calculation to malfunction.

I have tested this in the following manner.
1. created a stagin enviroment - a copy of production
2. tested all callculation working ok on the frot end i.e. all custom functions are claculatiing and providing correct results
3. update #1 - ews plagin to 2.5.2
4. again. tested all callculation working ok on the frot end i.e. all functionalty is intact - rsults are working properly
5. update #2 - ed plagin to 1.9.4
6. again. tested all callculation working ok on the frot end - this time the claculation got broken and the rsults are not working correctly

Please advice,
thanks,

David

[davidZ-4]

I just reverted to cred 1.9.3.1 and the issue was gone.
So I conclude the issue got to do with cred 1.9.4 that was changed...

hope this helps,
thanks,

David

[Luo Yang]

Dear David,

What is the "calculation" you mentioned above?
If it is some custom PHP codes, please copy/paste it here, and describe detail steps to duplicate same problem, I need to test and debug it in my localhost, thanks

[davidZ-4]

Hi Luo,
the calculation is a custom code i wrote. it takes custom fields data and run some calculation on them then it populate the results into custom fields in the custom post. it uses the cred_save_data to fire the calculation code.

when i save the custom post using the WordPress beck end the calculation run correctly. the issue is only when i save the post in the front end using cred.

please enable private response and i will provide you with access to the staging environment.

Thanks,

David

[Luo Yang]

OK, please backup your website , provide the credentials and FTP access of your website, and point out the problem page URL and CRED form URL, and where I can edit your custom PHP codes. thanks

[Luo Yang]

I can login your website,
1) but the credentials you provide above isn't an admin account, so I can not see how do you setup the CRED form, please provide a valid admin account,
2) and point out where I can edit your custom PHP codes, how do you setup the "calculation" with custom PHP codes?

[davidZ-4]

Hi Lue,

Sorry, i have changed your account to admin.
as for the custom code the location is under the plugins menu item, "Plugins" > "Edit functions" the code start at line 640 under "// calculate education need".
the calculation is triggered using the code in line 798

add_action('save_post', 'calculate_education', 99);

add_action('cred_save_data', 'education_calc',10,2);
function education_calc($post_id, $form_data) {
	// if a specific form
    if ( $form_data['id'] == 1716 || $form_data['id'] == 1719)  {
		calculate_education( $post_id );

	}
}

thanks,

David

[Luo Yang]

Thanks for the details, I can login your website, since it is a custom codes problem, there isn't any or guaranty or ETA to fix it, I am trying to locate the problem for you, will update this thread if there is anything found

[Luo Yang]

I can see the problem in your website, since it is a custom PHP codes problem, I will need to debug your PHP codes manually, but the FTP access you provided above is not valid, it is not correspond to the website credentials, so please provide a copy of your website:
https://toolset.com/faq/provide-supporters-copy-site/
You can put it into your own google drive disk, and share the link in the "private detail box", I need to duplicate same problem and debug it in my localhost, thanks

[davidZ-4]

Hi Lue,

Thanks for the update.
I have seen the FTP issue many time with toolset support. unfortunately its mandatory to put the folder name in the FTP address in the private message and that's the cause of the ftp access issue.
Please omit the folder name from the ftp link and try again using only lien caché as the ftp address i.e. no folder name.
I checked it with FileZilla and accessed the FTP with the credentials provided above with no issue.
if this will not work I will create a duplicator package and will post it on a secure share location.

BTW the "legal disclaimer." link in the private message doesn't work in Firefox nor Edge browsers.

let me know
Thanks,

David

[Luo Yang]

I just tried the FTP access again, it does not work, it seems that it is another website, please provide the duplicator package, thanks.

[Luo Yang]

Thanks for the details, I am downloading the files, will feedback if there is anything found

[Luo Yang]

Here are what I found, the problem is in your custom PHP codes, plugin file functions.php, line 639!797:

// calculate education need
function calculate_education($post_ID)
{
...
}

It triggers the PHP warning message, for example:

Warning: number_format() expects parameter 1 to be float, string given in \wp-content\plugins\functions.php on line 1451

Warning: number_format() expects parameter 1 to be float, string given in \equi\wp-content\plugins\functions.php on line 1451

It is an error of your custom PHP codes, you will need to change line 650, from:

$parent_assump_post_id = wpcf_pr_post_get_belongs(get_the_ID(), 'assumption');

To:

$parent_assump_post_id = wpcf_pr_post_get_belongs($post_ID, 'assumption');

and test again

[davidZ-4]

Thanks Lue,
wow!!! this is a great catch!

I'm not sure why it worked just fine before the CRED update. did something change in the way that i can get parent post id using the wpcf_pr_post_get_belongs code?

So i understand that i need to use the following code to get parent post id moving forward.

$parent_post_id = wpcf_pr_post_get_belongs($post_ID, 'parent slug');

I still wonder why i had it as

$parent_post_id = wpcf_pr_post_get_belongs(get_the_ID(), 'parent slug');

anyway i changed it in my code and the issue is resolved.
thanks so much for helping with this.

David

[Luo Yang]

You are welcome