search

jesusangel / wc-sermepa

Sermepa payment gateway plugin for WooCommerce
Other
28 stars 16 forks source link

Php Notice: status se llamo incorrectamente.... #36

Closed SUGO-es closed 5 years ago

SUGO-es commented 6 years ago

Hola!

No es error como tal, pero va dejando aviso.

Hay una llamada en la 720 a status de la orden que va a dejar de ser compatible.

// Check order not already completed
if ( $order->status == 'completed' ) {

Cuando pueda la cambia por...

// Check order not already completed
if ( $order->get_status() == 'completed' ) {

Saludos cordiales y gracias por el plugin! Javier

SUGO-es commented 5 years ago

Hola,

Esta nota se escribió en Enero, no se ha contestado ni solucionado, el plugin al final dejará de funcionar correctamente...ademas de dejar cantidad de avisos en el log.

El error se produce en: /wp-content/plugins/woocommerce-sermepa-payment-gateway/wc_redsys_payment_gateway.php - line:720

El aviso lo indica como: PHP Notice: status se llamó incorrectamente. Order properties should not be accessed directly. Backtrace: require('wp-blog-header.php'), wp, WP->main, WP->parse_request, do_action_ref_array, WP_Hook->do_action, WP_Hook->apply_filters, WC_API->handle_api_requests, do_action('woocommerce_api_wc_myredsys'), WP_Hook->do_action, WP_Hook->apply_filters, WC_MyRedsys->check_notification, WC_Abstract_Legacy_Order->__get, wc_doing_it_wrong Por favor, visita Depuración en WordPress para más información. (Este mensaje se añadió en la versión 3.0.) in /wp-includes/functions.php on line 4161

Cada nueva instalación tengo que ir corrigiendo esto para evitar los avisos.

¿Seria tan amable de corregirlo en el plugin?

Saludos cordiales.

jesusangel commented 5 years ago

Hola.

Hecho. También puedes desactivar los avisos (notices) de PHP en la configuración de éste para que no aparezcan en los logs.

Saludos cordiales.