conedevelopment / simplepay-gateway

OTP SimplePay bővítmény WooCommerce (WordPress) áruházak részére.
https://simplepay.conedevelopment.com
MIT License
57 stars 22 forks source link

Nem felel meg a bővítmény működése a dokumentációban leírtaknak #130

Closed zbacsi closed 1 year ago

zbacsi commented 1 year ago

Leírás

Sikertelen fizetés után a visszatérő oldalon a fejlesztési dokumentációban meghatározott szövegezést kellene használni, ami a következő: Sikertelen tranzakció. SimplePay tranzakció azonosító: 5xxxxxxxx Kérjük, ellenőrizze a tranzakció során megadott adatok helyességét. Amennyiben minden adatot helyesen adott meg, a visszautasítás okának kivizsgálása kapcsán kérjük, szíveskedjen kapcsolatba lépni kártyakibocsátó bankjával.

Megszakított fizetés után (amikor a vásárló a SimplePay fizető oldalon megnyomja a "vissza a kereskedő oldalára" gombot), hiányzik a vásárló szöveges tájékoztatása: "Megszakított fizetés!"

Sikeres tranzakció után a visszatérő oldalon hiányzik a SimplePay tranzakció azonosító: 5xxxxxxxx

Mindhárom esetben jelenleg a pénztár oldalra irányít vissza az oldal.

Reprodukáláshoz szükséges lépések

Teszteld a fenti eseteket és láthatod, hogy nem írja ki a szükséges szövegeket, adatokat, csak simán visszadob a kosárhoz.

A korábbi hibajelentésemre az volt a válasz, hogy ebben az ingyenes verzióban ezt már nem javítjátok. Ha ez egy megszüntetésre szánt plugin, akkor miért nem írjátok ki a readmeben egyértelműen, hogy ne használjuk, ne építsünk rá, hanem keressünk egy fizetőset?

iamgergo commented 1 year ago

Teszteld a fenti eseteket és láthatod, hogy nem írja ki a szükséges szövegeket, adatokat, csak simán visszadob a kosárhoz.

Nekem mind három esetben a megfelelő működés lép fel a teszt során.

A korábbi hibajelentésemre az volt a válasz, hogy ebben az ingyenes verzióban ezt már nem javítjátok.

Ez egy Open Source bővítmény. Ha úgy érzed, hogy van olyan megoldásod a fenti problémára ami biztosan mást nem tör meg, akkor szeretettel várjuk a PR-t.

Ha ez egy megszüntetésre szánt plugin, akkor miért nem írjátok ki a readmeben egyértelműen, hogy ne használjuk, ne építsünk rá, hanem keressünk egy fizetőset?

Majd ha jónak látjuk ezt kiírjuk. Ha úgy érzed, hogy egy ingyenes, MIT license alatt futó megoldás nem tud annyit amit szeretnétek, tudok javasolni pár alternatívát:

zbacsi commented 1 year ago

A hiba oka még mindig az inkompabilitás a WooCommerce Sequential Order Numbers Pro pluginnal és kikapcsolhatatlan prefix. A wc_get_order nem fogja megtalálni a rendelést a fenti plugin által generált azonosítóval és az erőszakosan eléragasztott wc- prefixxel.

A megoldásom közöltem múltkor is, arra kaptam azt a választ, hogy ez ebben az ingyenes verzióban már nem lesz javítva. Pedig semmit nem árt, csak egy plusz lehetőség, ha nincs meg a rendelés. Mutatom újra:

if (! $order instanceof WC_Order) { // search for the order by custom order number $query_args = array( 'numberposts' => 1, 'meta_key' => '_order_number_formatted', 'meta_value' => preg_replace("/^wc-/","", $payload['o']), 'post_type' => 'shop_order', 'post_status' => 'any', 'fields' => 'ids', );

         $posts            = get_posts( $query_args );
         list( $order_id ) = ! empty( $posts ) ? $posts : null;            
         // order was found
         if ( $order_id !== null ) {
             $order=wc_get_order($order_id);
         }
   }

Át fogom nézni a kódot, hogy hány helyen van még ilyen rendelés beazonosítási probléma és menni fog a PR, ahogy időm engedi.