frafor
Forum Replies Created
-
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?
@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.
As a workaround, I am loading this function on DOMContentLoaded which seems to do the trick
function cmplz_workaround() {
? let placeholders = document.querySelectorAll('.cmplz-accept-marketing');
? placeholders.forEach(function(placeholder) {
? ? placeholder.addEventListener('click', function(e){
? ? ? e.preventDefault();
? ? ? let obj = this;
? ? ? let service = obj.getAttribute('data-service');
? ? ? let category = obj.getAttribute('data-category');
? ? ? category = category ? category : 'marketing';
? ? ? if ( complianz.clean_cookies == 1 && typeof service !== 'undefined' && service ){
? ? ? ? ? cmplz_set_service_consent(service, true);
? ? ? ? ? cmplz_enable_category('', 'general');
? ? ? ? ? cmplz_enable_category('', service);
? ? ? } else {
? ? ? ? ? //we're activating a category, so do not need service activation here.
? ? ? ? ? // cmplz_enable_category('', 'general');
? ? ? ? ? cmplz_set_consent( category, 'allow' );
? ? ? }
? ? ? cmplz_set_banner_status('dismissed');
? ? ? cmplz_fire_categories_event();
? ? ? cmplz_track_status();
? ? });
? });
}To be honest on Firefox I see that the only event attached is based on the following lines
let cmplzBlockedContent = document.querySelector('.cmplz-blocked-content-notice'); if ( cmplzBlockedContent) { cmplzBlockedContent.addEventListener('click', function(event) { event.stopPropagation(); }); }
While there is no sign of the following one
cmplz_add_event('click', '.cmplz-accept-category, .cmplz-accept-marketing', function(e){
? ? e.preventDefault();
? ? let obj = e.target;
? ? let service = obj.getAttribute('data-service');
? ? let category = obj.getAttribute('data-category');
? ? category = category ? category : 'marketing';
? ? if ( complianz.clean_cookies == 1 && typeof service !== 'undefined' && service ){
? ? ? ? cmplz_set_service_consent(service, true);
? ? ? ? cmplz_enable_category('', 'general');
? ? ? ? cmplz_enable_category('', service);
? ? } else {
? ? ? ? //we're activating a category, so do not need service activation here.
? ? ? ? // cmplz_enable_category('', 'general');
? ? ? ? cmplz_set_consent( category, 'allow' );
? ? }
? ? cmplz_set_banner_status('dismissed');
? ? cmplz_fire_categories_event();
? ? cmplz_track_status();
});There’s an option in GTM4WP for that in Advanced > Load GTM container as early as possible
Solved by loading Gtm container earlier!
Forum: Plugins
In reply to: [Slim SEO - Fast & Automated WordPress SEO Plugin] 404 Logs – /embed/t.valueI am having these request as well, wonder if it’s some king of attack
Just an update: after moving back to 4.4 it’s behaving same as 4.2, this means that each image is processed twice, but since the counter removes 1 anyway, one image is skipped.
Screenshot of bulk editor here: https://ibb.co/PmdJjJM
Screenshot of resulting media lib: https://ibb.co/TYJgVFR
Hi I am using the “classic widget” interface to show/hide widgets according to user role, unfortunately widgets are always being displayed to website admins.
Is this expected behavior? Because it seems to relate to this post & it’s confusing website admins. Im on 2.5.0, all up to date.
Other roles work fine tho
Forum: Plugins
In reply to: [WooCommerce] Woocommerce AnalyticsAdditionally, flagging “Pending” order in the Analytics settings does not change the situation.
@doublezed2 please fix this issue because otherwise analytics are worthless
Forum: Plugins
In reply to: [WooCommerce] Woocommerce AnalyticsHi there, I would like to inform you that I am facing the exact same issue: COD orders with ‘processing’ status are excluded by WooCommerce Analytics. None of the above suggestions is able to fix this issue.
- This reply was modified 9 months, 4 weeks ago by frafor.
Hi there, I am having the same exact db error on a website of my which is just hosting a landing page at the moment, so there’s nothing complex on it…
[12-Feb-2024 16:45:54 UTC] WordPress database error Commands out of sync; you can't run this command now for query UPDATE
wp_options
SEToption_value
= 'a:19:{s:10:\"127e677c8d\";i:1707756353;s:10:\"188ebe03c3\";i:1707756353;s:10:\"28d903f7ed\";i:1707756353;s:10:\"47ea943e39\";i:1707756353;s:10:\"4ad1ffa77a\";i:1707756353;s:10:\"655840fb76\";i:1707756353;s:10:\"77230c6457\";i:1707756353;s:10:\"7912f2e30f\";i:1707756353;s:10:\"8a99975c88\";i:1707756353;s:10:\"962eaf65b9\";i:1707756353;s:10:\"9da8092e03\";i:1707756353;s:10:\"ab90d271d0\";i:1707756353;s:10:\"b3f7345be7\";i:1707756353;s:10:\"cabbb5f088\";i:1707756353;s:10:\"ce8a0064d3\";i:1707756353;s:10:\"cf3927860d\";i:1707756353;s:10:\"dad3af107f\";i:1707756353;s:10:\"ecac7f51dd\";i:1707756353;s:10:\"f8bf733f55\";i:1707756353;}' WHEREoption_name
= 'apto-dbs-ready-status' made by shutdown_action_hook, do_action('shutdown'), WP_Hook->do_action, WP_Hook->apply_filters, FernleafSystems\Wordpress\Plugin\Core\Databases\Common\TableReadyCache->save, FernleafSystems\Wordpress\Services\Core\General->updateOption, update_option [14-Feb-2024 03:46:12 UTC] WordPress database error Commands out of sync; you can't run this command now for query UPDATEwp_options
SEToption_value
= 'a:19:{s:10:\"127e677c8d\";i:1707882371;s:10:\"188ebe03c3\";i:1707882371;s:10:\"28d903f7ed\";i:1707882371;s:10:\"47ea943e39\";i:1707882371;s:10:\"4ad1ffa77a\";i:1707882371;s:10:\"655840fb76\";i:1707882371;s:10:\"77230c6457\";i:1707882371;s:10:\"7912f2e30f\";i:1707882371;s:10:\"8a99975c88\";i:1707882371;s:10:\"962eaf65b9\";i:1707882371;s:10:\"9da8092e03\";i:1707882371;s:10:\"ab90d271d0\";i:1707882371;s:10:\"b3f7345be7\";i:1707882371;s:10:\"cabbb5f088\";i:1707882371;s:10:\"ce8a0064d3\";i:1707882371;s:10:\"cf3927860d\";i:1707882371;s:10:\"dad3af107f\";i:1707882371;s:10:\"ecac7f51dd\";i:1707882371;s:10:\"f8bf733f55\";i:1707882371;}' WHEREoption_name
= 'apto-dbs-ready-status' made by shutdown_action_hook, do_action('shutdown'), WP_Hook->do_action, WP_Hook->apply_filters, FernleafSystems\Wordpress\Plugin\Core\Databases\Common\TableReadyCache->save, FernleafSystems\Wordpress\Services\Core\General->updateOption, update_optionForum: Plugins
In reply to: [Change Quantity on Checkout for WooCommerce] Latest Version Breaks CheckoutJust to give a quick update on the matter to @arxdesign and @firstclasscode, Bhavik was very kind to check out my website. We managed to find a conflict between the WPC Fly Cart plugin I am using and CQOC, as them both rely on cart_fragments.min.js to update their content and this can cause some issues.
As a workaround, I just had to disable WPC Fly Cart in the checkout to get things work fine, also because a fly out cart in the checkout is useless when there’s CQOC in place.
That said, CQOC 2.1 still works more consistently: with this version I just get issues on WPC Fly Cart layout, not the whole header / footer css. Still it is better not to show it in production.
So maybe in your case could be the Salient theme having something to do with its cart too. Usually custom carts always rely on WC’s cart_fragments, so if you have one in the header, please check the theme setting if it’s possible to disable it on specific pages (in my case, WPC Fly Cart have it).
Forum: Plugins
In reply to: [Change Quantity on Checkout for WooCommerce] Latest Version Breaks CheckoutAnyway we can do it the other way around: get my e-mail address by visiting my company website at https://sito.express you can find it in the footer
Forum: Plugins
In reply to: [Change Quantity on Checkout for WooCommerce] Latest Version Breaks CheckoutI am sorry, but the dev env of any website is obviously password protected and the URL not published around for very basic security reasons. The production env is accessible, but it has 2.1 running as it’s an e-commerce, like… selling products, so it has to work fine!
- This reply was modified 1 year, 1 month ago by frafor.