Προς τους developers, γιατί δε φτιάχνετε ένα github να ανεβάζουμε τις διορθώσεις; Η δομή του plugin έχει όπως έγραψα διαφορά λάθη σε διάφορα σημεία και νομίζω θα σας ωφελούσε κάτι τέτοιο. Μια πρόταση είναι λίγα λεπτά παίρνει και θα σας γλιτώσει από κόπο.
δυστυχώς δεν έχω κρατήσει το version θα πρέπει να το δω στα backup μου πάλι. Είναι το 1.7.1.
Δεν ξέρω αν ενδιαφέρονται οι developers αλλά οι τελευταίες αλλαγές που έγιναν βρίθουν λαθών, κάποια τα έχουμε επισημάνει αλλά δεν έχουν διορθωθεί και αφορούν το UX, αλλά σε μεταφράσεις, αλλά σε θέματα όπως αυτό. Αν έχετε backup η τελευταία πλήρως λειτουργική έκδοση είναι η 1.7.1
κάποια παρακάμφθηκαν και κάποια μπαλώθηκαν, έχω κάνει fork το plugin για τη δική μου σελίδα με βελτιώσεις γιατί ορισμένα πράγματα πχ οι μεταφράσεις δεν είναι σωστές σε κεφαλαία μικρά και πρόσθεσα την αυτόματη μετατροπή του cardholder name σε Αγγλικά το γράφει δε το γράφει έτσι ο πελάτης.
@gibeladakis μια καλή ιδέα για τα translations θα ήταν η ύπαρξη ενός pot file που τα διάφορα plugin θα έκαναν ούτως ή άλλως sync οπότε δε θα ήταν άμεση η ανάγκη υποστήριξης.
state/county and form it as state / county for consistency.
Furthermore considering transliteration of the billing name to the cardholder name if the client leaves this empty, even though this is counterproductive as your developer noted in the PHP file of the plugin.
Lastly, I still do not consider the loop necessary. WooCommerce already fires a notice when a mandatory field is empty by default. Format your mandatory fields as per WooCommerce requirements and the loop is unnecessary and might lead to unwanted notices, or double notices.
This reply was modified 1 year, 8 months ago by Magmadiver.
The main problem is not that the Piraeus Bank plugin is marking the “Cardholder Name” as mandatory (even though it is not really set as mandatory as it is marked false, if left empty you handle this on the back end) but that when it is not filled out, due to a validation loop an error notice appears. Additionally, hidden fields are being incorrectly flagged as mandatory as well. The loop checks each field even if it’s not visible and mistakenly adds these errors. A suggested fix involves reverting to version 1.7.1. You may need to modify the validation loop that handles visibility and field requirements to prevent this! Furthermore, the notices are not translatable as they are not added to the po file of the plugin.
So the main issue seems to be the following loop.
foreach ($requiredFields as $field => $info) { if (!isset($_POST[$field]) || trim($_POST[$field]) === '') { wc_add_notice(__("$info is a mandatory field!", 'woocommerce'), 'error'); } }
that shoots out untranslated and non translatable notices like this and prevents checkout
This reply was modified 1 year, 8 months ago by Magmadiver.
This reply was modified 1 year, 8 months ago by Magmadiver.
@gibeladakis μπορώ να στείλω αναλυτικά σε screen capture το ζήτημα που έχουμε εμείς και άλλοι χρήστες αν χρειαστεί. Ωστόσο τα προβλήματα είναι 2 πρώτον το loop που δημιουργεί όλα αυτά τα notices και μάλλον δεν είναι απαραίτητο, ή δεν έχει εφαρμοστεί σωστά, δεύτερο οκ βγαίνουν τα notices, δεν είναι στα po files του plugin.
Αυτό το Loop που προστέθηκε στο νέο αρχείο μάλλον δημιουργεί το πρόβλημα, δεδομένου ότι έχουν αλλάξει σε required τα fields.
foreach ($requiredFields as $field => $info) { if (!isset($_POST[$field]) || trim($_POST[$field]) === '') { wc_add_notice(__("$info is a mandatory field!", 'woocommerce'), 'error'); } }
Περαιτέρω, δεδομένου ότι έχετε αφήσει σαν false τελικώς το cardname holder και τα χειρίζεστε στο backend, μήπως θα άξιζε να κάνετε και μετατροπή σε αγγλικούς χαρακτήρες πχ με το
// μετατροπή σε λατινικούς χαρακτήρες με χρήση του iconv $cardholderName = iconv('UTF-8', 'ASCII//TRANSLIT//IGNORE', $cardholderName);
This reply was modified 1 year, 8 months ago by Magmadiver.
Σε πολλά καταστήματα με virtual products δεν έχει νόημα να δίδονται όλα τα στοιχεία και γίνεται edit με φίλτρο ή με plugin στα checkout fields, περαιτέρω ακόμα κι έτσι, το cardholder name mandatory δεν είναι μεταφρασμένο ή δε μπορεί να μεταφραστεί αν δεν επέμβει κανείς στα αρχεία του plugin, καθώς δεν είναι στα αρχεί μετάφρασης και το plugin δεν έχει pot. https://postimg.cc/XpYZLBYW
Η τελευταία έκδοση που λειτουργεί σωστά είναι η 1.7.1 η οποία δεν είναι διαθέσιμη στο repository. Ωστόσο μπόρεσα και τη βρήκα σε backup. Θα πρότεινα να κάνετε push ξανά αυτή μέχρι να λύσετε το θέμα.
