Magmadiver

Προ? του? developers, γιατ? δε φτι?χνετε ?να github να ανεβ?ζουμε τι? διορθ?σει?; Η δομ? του plugin ?χει ?πω? ?γραψα διαφορ? λ?θη σε δι?φορα σημε?α και νομ?ζω θα σα? ωφελο?σε κ?τι τ?τοιο. Μια πρ?ταση ε?ναι λ?γα λεπτ? πα?ρνει και θα σα? γλιτ?σει απ? κ?πο.

δυστυχ?? δεν ?χω κρατ?σει το version θα πρ?πει να το δω στα backup μου π?λι. Ε?ναι το 1.7.1.

Δεν ξ?ρω αν ενδιαφ?ρονται οι developers αλλ? οι τελευτα?ε? αλλαγ?? που ?γιναν βρ?θουν λαθ?ν, κ?ποια τα ?χουμε επισημ?νει αλλ? δεν ?χουν διορθωθε? και αφορο?ν το UX, αλλ? σε μεταφρ?σει?, αλλ? σε θ?ματα ?πω? αυτ?. Αν ?χετε backup η τελευτα?α πλ?ρω? λειτουργικ? ?κδοση ε?ναι η 1.7.1

κ?ποια παρακ?μφθηκαν και κ?ποια μπαλ?θηκαν, ?χω κ?νει fork το plugin για τη δικ? μου σελ?δα με βελτι?σει? γιατ? ορισμ?να πρ?γματα πχ οι μεταφρ?σει? δεν ε?ναι σωστ?? σε κεφαλα?α μικρ? και πρ?σθεσα την αυτ?ματη μετατροπ? του cardholder name σε Αγγλικ? το γρ?φει δε το γρ?φει ?τσι ο πελ?τη?.

Λογικ? μελλοντικ? θα κ?νουν κι ?λλο patch

@gibeladakis μια καλ? ιδ?α για τα translations θα ?ταν η ?παρξη εν?? pot file που τα δι?φορα plugin θα ?καναν ο?τω? ? ?λλω? sync οπ?τε δε θα ?ταν ?μεση η αν?γκη υποστ?ριξη?.

Κανονικ? ε?ναι ?λα required αλλ? οι πληρωμ?? περν?νε και με το 1/3 των στοιχε?ων ω? τ?ρα.

please don’t forget

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 6 months, 3 weeks ago by Magmadiver.

@gibeladakis

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 6 months, 3 weeks ago by Magmadiver.
• This reply was modified 6 months, 3 weeks 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 6 months, 4 weeks ago by Magmadiver.

Nα σημει?σω εδ? ?τι ακ?μα και mandatory να ε?ναι, θα ?πρεπε να ε?ναι τουλ?χιστον translatable.

δυστυχ?? το θ?μα δε φα?νεται να ?χει επιλυθε?.

Σε πολλ? καταστ?ματα με virtual products δεν ?χει ν?ημα να δ?δονται ?λα τα στοιχε?α και γ?νεται edit με φ?λτρο ? με plugin στα checkout fields, περαιτ?ρω ακ?μα κι ?τσι, το cardholder name mandatory δεν ε?ναι μεταφρασμ?νο ? δε μπορε? να μεταφραστε? αν δεν επ?μβει κανε?? στα αρχε?α του plugin, καθ?? δεν ε?ναι στα αρχε? μετ?φραση? και το plugin δεν ?χει pot. https://postimg.cc/XpYZLBYW

?

ναι φυσικ? βρε?τε το εδ?

https://easyupload.io/u8k0fb

Η τελευτα?α ?κδοση που λειτουργε? σωστ? ε?ναι η 1.7.1 η οπο?α δεν ε?ναι διαθ?σιμη στο repository. Ωστ?σο μπ?ρεσα και τη βρ?κα σε backup. Θα πρ?τεινα να κ?νετε push ξαν? αυτ? μ?χρι να λ?σετε το θ?μα.

Hi,

thanks for the great plugin. Any updates on this?
I think it will greatly upgrade the functionality.

Thanks!

• This reply was modified 3 years, 3 months ago by Magmadiver.

You could actually start by helping me here. Unfortunately the field is not present in the php file.

• This reply was modified 4 years, 3 months ago by Magmadiver.