George P.
Forum Replies Created
-
Thank you for the quick reply!
My client is currently not running a campaign, so it’s not required for now.
We will keep it disabled until the update.
Keep up the good work!
I am also facing this issue with Storefront theme 4.6.0, I tried it on a test installation with just WooCommerce 9.5.1 and Advanced Dynamic Pricing for WooCommerce 4.9.1, no rules set at all in the plugin, just installed it breaks the typical functionality of woocommerce’s strikethrough starting price.
It only happens when starting price is more than 3 digits and discounted price is 3 digits or less, so it might have something to do with the thousands separator formatting.
I tested it with both simple product price and variation prices, with the same issue. I’ve also tried alterning comma for dot in the thousands separator and decimals, and symbol right or left with no difference.
Again I don’t use any rule through the plugin, I just set a starting and a discounted price at the product level and the bug occurs.
- This reply was modified 1 year, 4 months ago by George P..
Καλησπέρα, το πρόβλημα φαίνεται πως παραμένει καθώς το πρόσθετο είναι ακόμη στην έκδοση 3.0.0, όπως μόλις ενημερώθηκα από πελάτη μου.
Το συγκεκριμένο θέμα είναι bug καθώς δεν έχει να κάνει με το αν είναι όντως υποχρεωτικό ή όχι το πεδίο από πλευράς τράπεζας, αλλά επειδή παρεμποδίζει τη λειτουργία άλλων τρόπων πληρωμής όπως η κατάθεση ή αντικαταβολή.
Μέχρι να δημοσιευθεί η επόμενη έκδοση, ένα workaround που εφάρμοσα στον πελάτη μου είναι το εξής, στο παρακάτω αρχείο του plugin:
woo-payment-gateway-for-piraeus-bank/classes/WC_Piraeusbank_Gateway.phpΑλλάζουμε την τιμή required από true σε false στην παρακάτω μέθοδο (γραμμή 128):
public function custom_override_checkout_fields( $billing_fields ) {
$billing_fields['cardholder_name'] = [
...
'required' => false,
...
];
return $billing_fields;
}Η προηγούμενη λύση που προτάθηκε με σχόλιο στη γραμμή ‘cardholder_name’ => ‘Cardholder Name’, δεν προτείνεται καθώς το πεδίο εξακολουθεί να γίνεται rendered ως required με αποτέλεσμα να αποτυγχάνει ο έλεγχος σε HTML επίπεδο και να εμφανίζεται το ίδιο μήνυμα όταν έχει επιλεχθεί για παράδειγμα ως τρόπος πληρωμής η κατάθεση, οπότε υπάρχει ως κρυφό το υποχρεωτικό πεδίο cardholder_name και εξακολουθεί να εμποδίζει την υποβολή της παραγγελίας.
Η παρακάτω μέθοδος εκτελείται μόνο όταν ο πελάτης επιλέξει ως τρόπο πληρωμής την κάρτα οπότε εκεί θα πρότεινα να το αφήσετε ως έχει καθώς σε αυτή την περίπτωση θέλουμε όντως να ελέγχεται η τιμή του πεδίου για λόγους ασφαλείας (γραμμή 890):
public function validate_fields() {
$requiredFields = [
...
'cardholder_name' => 'Cardholder Name',
];
...
}Και για να μην προκαλεί σύγχυση στον πελάτη η ένδειξη (Προαιρετικό) στο πεδίο “Όνομα Κατόχου Κάρτας” όταν επιλέξει ως τρόπο πληρωμής την κάρτα, μπορείτε να προσθέσετε το παρακάτω CSS, ώστε να κρύψετε την παραπάνω ένδειξη και να εμφανίσετε έναν σχετικό κόκκινο αστερίσκο υποχρεωτικής συμπλήρωσης δίπλα στην περιγραφή του πεδίου:
#cardholder_name_field label .optional {
display: none;
}
#cardholder_name_field label:after {
content: "*";
color: #e2401c;
}Ίσως φαίνεται υπερβολή η λύση καθώς είναι μόνο για το διάστημα μέχρι να βγει η επόμενη έκδοση, αλλά ακόμη πιο υπερβολή είναι όταν επί 3 βδομάδες τώρα χάνονται παραγγελίες στα e-shop εξαιτίας του παραπάνω bug.
I confirm it works fine with the latest version, with both plugins enabled.
Many thanks for the support!Hello, I can also confirm this. The automated update that occurred on 2020-11-25 caused the following error to come up in my installation, breaking any email functionality:
[02-Dec-2020 06:00:11 UTC] PHP Fatal error: Uncaught Exception: The message is not valid because it contains no message or html field. in /home/username/public_html/wp-content/plugins/wp-mail-logging/src/WPML_MailExtractor.php:37 Stack trace: #0 /home/username/public_html/wp-content/plugins/wp-mail-logging/src/WPML_MailExtractor.php(19): No3x\WPML\WPML_MailExtractor->extractMessage(false) #1 /home/username/public_html/wp-content/plugins/wp-mail-logging/src/WPML_Plugin.php(214): No3x\WPML\WPML_MailExtractor->extract(false) #2 /home/username/public_html/wp-includes/class-wp-hook.php(287): No3x\WPML\WPML_Plugin->log_email(false) #3 /home/username/public_html/wp-includes/plugin.php(206): WP_Hook->apply_filters(false, Array) #4 /home/username/public_html/wp-includes/pluggable.php(182): apply_filters('wp_mail', Array) #5 /home/username/public_html/wp-content/plugins/wp-mail-smtp/src/Admin/Pages/TestTab.php(172): wp_mail('[email protected]', 'WP Mail SMTP: H...', '\t\t<!doctype htm...', Array) #6 /home/username/public_html/wp-content/plugins/wp-mail-smtp/src/Admin in /home/username/public_html/wp-content/plugins/wp-mail-logging/src/WPML_MailExtractor.php on line 37It seems to be interfering with the latest version of “WP Mail Logging by MailPoet” plugin. Disabling either plugin prevents the error from happening and allows the emails to be sent successfully, however the logging functionality is important for me, so I have disabled the “Check & Log Email” plugin until this gets fixed.