woocommerce / woocommerce-admin

(Deprecated) This plugin has been merged to woocommerce/woocommerce
https://woocommerce.github.io/woocommerce-admin/#/
Other
362 stars 144 forks source link

[Globalstep- WooCommerce Admin 2.2.0 Beta 2 ] Mercado Pago Payments option fails to stay active for all the country addresses. #6648

Closed gglobalstep closed 3 years ago

gglobalstep commented 3 years ago

Bug Description:

Mercado Pago Payments option fails to stay active for all the country addresses.

Test Instructions:

https://github.com/woocommerce/woocommerce-admin/blob/release/2.2.0-beta.1/TESTING-INSTRUCTIONS.md#payments-task-include-mercado-pago-6572

Environment:

Woocommerce Version : WooCommerce 5.1 WordPress version: (v5.7) WooCommerce Admin version: v2.2.0-Beta-2

PC: Windows 10, Mac 10.14.6 Chrome(Version 88.0.4324.104) Firefox(Version 86.0) Safari: v13.1.1 IE 11(Version 11.55.17763.0)

Steps To Reproduce:

  1. Create any test site using JN site.
  2. Install and activate all the required plugins
  3. Complete the onboarding setup wizard and enter the store address among the following options: Mexico Brazil, Argentina, Chile, Colombia, Peru and Uruguay
  4. Go to "Choose payments section" on "WooCommerce-Home" page
  5. Click on "Setup" button for Mercado Pago payments for WooCommerce plugin.
  6. Click on "Continue' button
  7. Note that now it appears active, but if i visit WooCommerce-Settings-Payments, it displays none of the Mercado payments are active
  8. Go to Plugins-Installed Plugins, Note the plugin for Mercado is installed and activated
  9. Now if i go back to "Choose payments" section on "WooCommerce->Home" page, the plugin is automatically in inactive state
  10. Observe that Mercado Pago Payments option fails to stay active for all the country addresses.

Note: For Windows: It fails to display as an option in "Choose payments section" on "WooCommerce-Home" page For Windows and Mac: Also observe that the user is unable to turn on "Mercado payments" from "WooCommerce-Settings-Payments" tab

Actual Result:

Mercado Pago Payments option fails to stay active for all the country addresses.

Expected Result:

Mercado Pago Payments option should stay active for all the country addresses after its setup is completed.

Videos:

https://user-images.githubusercontent.com/41110392/112187847-771e7f80-8c28-11eb-9f5d-e8da3751189d.mp4

https://user-images.githubusercontent.com/41110392/112187886-8271ab00-8c28-11eb-8c99-805b2bdcbae3.mp4

Isolating the problem (mark completed items with an [x]):

``` ` ### WordPress Environment ### WordPress address (URL): https://ethical-kite.jurassic.ninja Site address (URL): https://ethical-kite.jurassic.ninja WC Version: 5.1.0 REST API Version: ✔ 5.1.0 WC Blocks Version: ✔ 4.4.3 Action Scheduler Version: ✔ 3.1.6 WC Admin Version: ✔ 2.2.0-beta.2 Log Directory Writable: ✔ WP Version: 5.7 WP Multisite: – WP Memory Limit: 256 MB WP Debug Mode: ✔ WP Cron: ✔ Language: en_US External object cache: – ### Server Environment ### Server Info: Apache/2.4.46 (Unix) OpenSSL/1.0.2g PHP Version: 7.4.16 PHP Post Max Size: 1 GB PHP Time Limit: 30 PHP Max Input Vars: 5000 cURL Version: 7.47.0 OpenSSL/1.0.2g SUHOSIN Installed: – MySQL Version: 5.7.33-0ubuntu0.16.04.1-log Max Upload Size: 512 MB Default Timezone is UTC: ✔ fsockopen/cURL: ✔ SoapClient: ✔ DOMDocument: ✔ GZip: ✔ Multibyte String: ✔ Remote Post: ✔ Remote Get: ✔ ### Database ### WC Database Version: 5.1.0 WC Database Prefix: wp_ Total Database Size: 11.00MB Database Data Size: 8.36MB Database Index Size: 2.64MB wp_woocommerce_sessions: Data: 0.02MB + Index: 0.02MB + Engine InnoDB wp_woocommerce_api_keys: Data: 0.02MB + Index: 0.03MB + Engine InnoDB wp_woocommerce_attribute_taxonomies: Data: 0.02MB + Index: 0.02MB + Engine InnoDB wp_woocommerce_downloadable_product_permissions: Data: 0.02MB + Index: 0.06MB + Engine InnoDB wp_woocommerce_order_items: Data: 0.02MB + Index: 0.02MB + Engine InnoDB wp_woocommerce_order_itemmeta: Data: 0.08MB + Index: 0.09MB + Engine InnoDB wp_woocommerce_tax_rates: Data: 0.02MB + Index: 0.06MB + Engine InnoDB wp_woocommerce_tax_rate_locations: Data: 0.02MB + Index: 0.03MB + Engine InnoDB wp_woocommerce_shipping_zones: Data: 0.02MB + Index: 0.00MB + Engine InnoDB wp_woocommerce_shipping_zone_locations: Data: 0.02MB + Index: 0.03MB + Engine InnoDB wp_woocommerce_shipping_zone_methods: Data: 0.02MB + Index: 0.00MB + Engine InnoDB wp_woocommerce_payment_tokens: Data: 0.02MB + Index: 0.02MB + Engine InnoDB wp_woocommerce_payment_tokenmeta: Data: 0.02MB + Index: 0.03MB + Engine InnoDB wp_woocommerce_log: Data: 0.02MB + Index: 0.02MB + Engine InnoDB wp_actionscheduler_actions: Data: 0.05MB + Index: 0.11MB + Engine InnoDB wp_actionscheduler_claims: Data: 0.02MB + Index: 0.02MB + Engine InnoDB wp_actionscheduler_groups: Data: 0.02MB + Index: 0.02MB + Engine InnoDB wp_actionscheduler_logs: Data: 0.02MB + Index: 0.03MB + Engine InnoDB wp_ce4wp_abandoned_checkout: Data: 0.02MB + Index: 0.02MB + Engine InnoDB wp_commentmeta: Data: 0.02MB + Index: 0.03MB + Engine InnoDB wp_comments: Data: 0.02MB + Index: 0.09MB + Engine InnoDB wp_links: Data: 0.02MB + Index: 0.02MB + Engine InnoDB wp_mailchimp_carts: Data: 0.02MB + Index: 0.00MB + Engine InnoDB wp_mailchimp_jobs: Data: 0.02MB + Index: 0.00MB + Engine InnoDB wp_mailpoet_custom_fields: Data: 0.02MB + Index: 0.02MB + Engine InnoDB wp_mailpoet_dynamic_segment_filters: Data: 0.02MB + Index: 0.02MB + Engine InnoDB wp_mailpoet_feature_flags: Data: 0.02MB + Index: 0.02MB + Engine InnoDB wp_mailpoet_forms: Data: 0.02MB + Index: 0.00MB + Engine InnoDB wp_mailpoet_log: Data: 0.02MB + Index: 0.00MB + Engine InnoDB wp_mailpoet_mapping_to_external_entities: Data: 0.02MB + Index: 0.02MB + Engine InnoDB wp_mailpoet_newsletters: Data: 0.02MB + Index: 0.03MB + Engine InnoDB wp_mailpoet_newsletter_links: Data: 0.02MB + Index: 0.05MB + Engine InnoDB wp_mailpoet_newsletter_option: Data: 0.02MB + Index: 0.02MB + Engine InnoDB wp_mailpoet_newsletter_option_fields: Data: 0.02MB + Index: 0.02MB + Engine InnoDB wp_mailpoet_newsletter_posts: Data: 0.02MB + Index: 0.02MB + Engine InnoDB wp_mailpoet_newsletter_segment: Data: 0.02MB + Index: 0.02MB + Engine InnoDB wp_mailpoet_newsletter_templates: Data: 2.52MB + Index: 0.00MB + Engine InnoDB wp_mailpoet_scheduled_tasks: Data: 0.02MB + Index: 0.03MB + Engine InnoDB wp_mailpoet_scheduled_task_subscribers: Data: 0.02MB + Index: 0.02MB + Engine InnoDB wp_mailpoet_segments: Data: 0.02MB + Index: 0.02MB + Engine InnoDB wp_mailpoet_sending_queues: Data: 0.02MB + Index: 0.03MB + Engine InnoDB wp_mailpoet_settings: Data: 0.02MB + Index: 0.02MB + Engine InnoDB wp_mailpoet_statistics_clicks: Data: 0.02MB + Index: 0.05MB + Engine InnoDB wp_mailpoet_statistics_forms: Data: 0.02MB + Index: 0.02MB + Engine InnoDB wp_mailpoet_statistics_newsletters: Data: 0.02MB + Index: 0.03MB + Engine InnoDB wp_mailpoet_statistics_opens: Data: 0.02MB + Index: 0.08MB + Engine InnoDB wp_mailpoet_statistics_unsubscribes: Data: 0.02MB + Index: 0.05MB + Engine InnoDB wp_mailpoet_statistics_woocommerce_purchases: Data: 0.02MB + Index: 0.06MB + Engine InnoDB wp_mailpoet_stats_notifications: Data: 0.02MB + Index: 0.03MB + Engine InnoDB wp_mailpoet_subscribers: Data: 0.02MB + Index: 0.11MB + Engine InnoDB wp_mailpoet_subscriber_custom_field: Data: 0.02MB + Index: 0.02MB + Engine InnoDB wp_mailpoet_subscriber_ips: Data: 0.02MB + Index: 0.02MB + Engine InnoDB wp_mailpoet_subscriber_segment: Data: 0.02MB + Index: 0.03MB + Engine InnoDB wp_mailpoet_user_flags: Data: 0.02MB + Index: 0.02MB + Engine InnoDB wp_options: Data: 4.03MB + Index: 0.19MB + Engine InnoDB wp_postmeta: Data: 0.20MB + Index: 0.17MB + Engine InnoDB wp_posts: Data: 0.08MB + Index: 0.06MB + Engine InnoDB wp_termmeta: Data: 0.02MB + Index: 0.03MB + Engine InnoDB wp_terms: Data: 0.02MB + Index: 0.03MB + Engine InnoDB wp_term_relationships: Data: 0.02MB + Index: 0.02MB + Engine InnoDB wp_term_taxonomy: Data: 0.02MB + Index: 0.03MB + Engine InnoDB wp_usermeta: Data: 0.02MB + Index: 0.03MB + Engine InnoDB wp_users: Data: 0.02MB + Index: 0.05MB + Engine InnoDB wp_wc_admin_notes: Data: 0.02MB + Index: 0.00MB + Engine InnoDB wp_wc_admin_note_actions: Data: 0.02MB + Index: 0.02MB + Engine InnoDB wp_wc_category_lookup: Data: 0.02MB + Index: 0.00MB + Engine InnoDB wp_wc_customer_lookup: Data: 0.02MB + Index: 0.03MB + Engine InnoDB wp_wc_download_log: Data: 0.02MB + Index: 0.03MB + Engine InnoDB wp_wc_order_coupon_lookup: Data: 0.02MB + Index: 0.03MB + Engine InnoDB wp_wc_order_product_lookup: Data: 0.02MB + Index: 0.06MB + Engine InnoDB wp_wc_order_stats: Data: 0.02MB + Index: 0.05MB + Engine InnoDB wp_wc_order_tax_lookup: Data: 0.02MB + Index: 0.03MB + Engine InnoDB wp_wc_product_meta_lookup: Data: 0.02MB + Index: 0.09MB + Engine InnoDB wp_wc_reserved_stock: Data: 0.02MB + Index: 0.00MB + Engine InnoDB wp_wc_tax_rate_classes: Data: 0.02MB + Index: 0.02MB + Engine InnoDB wp_wc_webhooks: Data: 0.02MB + Index: 0.02MB + Engine InnoDB ### Post Type Counts ### attachment: 26 customize_changeset: 1 mailpoet_page: 1 nav_menu_item: 5 page: 7 post: 3 product: 19 product_variation: 7 revision: 1 shop_order: 29 ### Security ### Secure connection (HTTPS): ✔ Hide errors from visitors: ✔ ### Active Plugins (15) ### Query Monitor: by John Blackbourn – 3.6.7 Companion Plugin: by Osk – 1.18 Creative Mail by Constant Contact: by Constant Contact – 1.3.5 Facebook for WooCommerce: by Facebook – 2.3.3 Jetpack by WordPress.com: by Automattic – 9.5 Google Ads & Marketing by Kliken: by Kliken – 1.0.7 Mailchimp for WooCommerce: by Mailchimp – 2.5.1 MailPoet 3 (New): by MailPoet – 3.60.4 WooCommerce Admin: by WooCommerce – 2.2.0-beta.2 WooCommerce Beta Tester: by WooCommerce – 2.0.2 Mercado Pago payments for WooCommerce: by Mercado Pago – 5.0.1 WooCommerce Shipping & Tax: by Automattic – 1.25.9 WooCommerce: by Automattic – 5.1.0 WP Crontrol: by John Blackbourn & crontributors – 1.10.0 WP phpMyAdmin: by Puvox.software – 5.0.4.1 ### Inactive Plugins (2) ### Akismet Anti-Spam: by Automattic – 4.1.9 Hello Dolly: by Matt Mullenweg – 1.7.2 ### Dropin Plugins (1) ### db.php: Query Monitor Database Class ### Settings ### API Enabled: – Force SSL: – Currency: USD ($) Currency Position: left Thousand Separator: Decimal Separator: . Number of Decimals: 0 Taxonomies: Product Types: external (external) grouped (grouped) simple (simple) variable (variable) Taxonomies: Product Visibility: exclude-from-catalog (exclude-from-catalog) exclude-from-search (exclude-from-search) featured (featured) outofstock (outofstock) rated-1 (rated-1) rated-2 (rated-2) rated-3 (rated-3) rated-4 (rated-4) rated-5 (rated-5) Connected to WooCommerce.com: – ### WC Pages ### Shop base: #6 - /shop/ Cart: #7 - /cart/ Checkout: #8 - /checkout/ My account: #9 - /my-account/ Terms and conditions: ❌ Page not set ### Theme ### Name: Twenty Twenty-One Version: 1.2 Author URL: https://wordpress.org/ Child Theme: ❌ – If you are modifying WooCommerce on a parent theme that you did not build personally we recommend using a child theme. See: How to create a child theme WooCommerce Support: ✔ ### Templates ### Overrides: – ### Action Scheduler ### Complete: 79 Oldest: 2021-03-23 09:08:33 +0000 Newest: 2021-03-23 14:30:26 +0000 ### Status report information ### Generated at: 2021-03-23 17:00:41 +00:00 ` ```
louwie17 commented 3 years ago

Doing some research into this bug, updating the option name use from 'woocommerce_mercadopago_settings' to 'woocommerce_woo-mercado-pago-basic_settings' would fix this issue. Although as soon as the user loads the Settings > Payments screen it will be reset to disabled unless the user has added there credentials for one of the environments.

Here is the code that sets enabled back to no if either the prod or test access_token is not set. I could set it to an empty string, which should keep it as enabled (but this is not intended, so don't recommend this). https://plugins.trac.wordpress.org/browser/woocommerce-mercadopago/trunk/includes/payments/class-wc-woomercadopago-payment-abstract.php#L1709

louwie17 commented 3 years ago

@pmcpinto @elizaan36 for this instance where the credentials are required (but we don't ask for them in the set up flow), should we keep the Set up button present, until these credentials are added? If the plugin is already installed it will show step 2 right away -> Connect to your Mercado Pago account (this has a link to the store settings page).

pmcpinto commented 3 years ago

for this instance where the credentials are required (but we don't ask for them in the set up flow), should we keep the Set up button present, until these credentials are added?

Looks like a good workaround to this issue.

gglobalstep commented 3 years ago

@louwie17 This issue is still observed to be Open on the following Environment:

Environment: Woocommerce Version : WooCommerce 5.2.2 and 5.3.0 Beta WordPress version: (v5.7)

PC: Windows 10, Mac 10.14.6 Chrome(Version 89.0.4389.90) Firefox(Version 87.0) Safari: v13.1.1

louwie17 commented 3 years ago

Thanks @gglobalstep, the issue was only partially solved as part of this ticket. We have an open issue https://github.com/woocommerce/woocommerce-admin/issues/6680 that will address this more fully.