Hello @frafor,
Thank you for reaching out to me. We are currently looking into this issue.
If you have any further questions, please feel free to ask. I am here to assist you.
Thank you,
ZealousWeb
Thread Starter
frafor
(@frafor)
@zealopensource As for the plugin breaking js, I figured it out: you are loading “jquery-core” in class.cf7pe.front.action.php. If other plugins are loading full “jquery”, this leads to jquery core functions being loaded twice, hence subsequent jquery-dependant scripts may experience unpredictable behaviour, most likely failing. I have just switched your script dependancies to rely on “jquery” and it seems to do the trick.
The “undefined array key” error seems to have gone after deactivating/reactivating the plugin, but I’ll monitor it.
I am still getting the “headers already sent” error, but that shouldn’t prevent the plugin to do the job. I still have to test it thou, will come back later with an update.
Thread Starter
frafor
(@frafor)
I have found another issue in class.cf7pe.lib.php: after testing, the description passed to PayPal was always the website name.
This is happening because $description field is defined on line 612 as:
$description = get_post_meta( $form_ID, CF7PE_META_PREFIX . 'description', true );
Then, it’s overridden on line 621 gathering the value from $posted_data:
$description = ( ( !empty( $description ) && array_key_exists( $description, $posted_data ) ) ? $posted_data[$description] : get_bloginfo( 'name' ) );
That would be ok, but still on line 627, a new variable $description_val is defined and later it’s the one sent to PayPal. Unfortunately, $description_val will always default to get_bloginfo(‘name’), because $description has been overwritten:
$description_val = ( ( !empty( $description ) && array_key_exists( $description, $posted_data ) ) ? $posted_data[$description] : get_bloginfo( 'name' ) );
This was most likely a mistake as the second evaluation for $description shouldn’t be there.
I also noticed that $mail defined in line 620 is never used, should it be left there hanging?
Dear @frafor,
Thank you for bringing this issue to our attention. We are currently investigating it.
If you have any questions, please feel free to ask—I’m here to assist you.
Thank you,
ZealousWeb
Dear @frafor,
Thank you so much. We are currently investigating it.
If you have any questions, please feel free to ask—I’m here to assist you.
Thank you,
ZealousWeb
