Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Advanced filters throw an error #6533

Closed
nicdwilson opened this issue Jun 14, 2023 · 16 comments
Closed

Advanced filters throw an error #6533

nicdwilson opened this issue Jun 14, 2023 · 16 comments
Labels
category: core WC Payments core related issues, where it’s obvious. component: transactions Issues related to Transactions priority: high The issue/PR is high priority—it affects lots of customers substantially, but not critically. type: bug The issue is a confirmed bug.

Comments

@nicdwilson
Copy link

Describe the bug

Clicking on Transactions > Advanced Filters results in unexpected response

To Reproduce

  1. Go to Payments > Transactions
  2. Click on Advanced Filters
  3. See error

Actual behavior

dAFUmr.png

Expected behavior

Advanced filters should work

Desktop (please complete the following information):

  • OS: MacOS 13,4
  • Browser Firefox
  • Version 115.0b4

Additional context

6409605-zen

@nicdwilson nicdwilson added the type: bug The issue is a confirmed bug. label Jun 14, 2023
@zmaglica
Copy link
Contributor

I wasn't able to reproduce the issue. It is working fine on my end. Are there more steps that is missing? Can you maybe test the issue on your end using JN site?

@zmaglica zmaglica added category: core WC Payments core related issues, where it’s obvious. component: transactions Issues related to Transactions labels Jun 15, 2023
@nicdwilson
Copy link
Author

Hi @zmaglica This behaviour was reported by the customer in 6409605-zd-woothemes and replicated on a Pressable site. There are no steps to replicate because I simply clicked on the advanced filter, and got the result on my screen which you see in the screenshot above.

I have added creds for the test site to 6409605-zd-woothemes

@johndcoy
Copy link

I'm seeing the same error as @nicdwilson on one Pressable site:

Screenshot
Screenshot link: https://d.pr/i/OPOW7C

But not on another Pressable site:

Screenshot
Screenshot link: https://d.pr/i/OhhWyl

The difference between these two sites is the one with the issue has CDN caching enabled and is running PHP 8.1. The one without issues has CDN disabled and is running PHP 7.4

@mattsherman
Copy link
Member

mattsherman commented Jun 20, 2023

This looks related to an update made in woocommerce/woocommerce#37967 for WooCommerce 7.8.

@daniyalahmadk
Copy link

Another report on this ticket: 6441052-zen. I can replicate the issue on our WCPay pressable staging site:

image

@sarbouw
Copy link

sarbouw commented Jun 21, 2023

6443725-zd

@foosantos
Copy link
Member

6452781-zen

@scotthopkins
Copy link

scotthopkins commented Jun 22, 2023

I also have this on my site... for any advanced filter type

Woocommerce 7.8.0
Woocommerce Payments Version 6.0.0
PHP 8.1

CleanShot 2023-06-22 at 16 43 42

@druesome
Copy link

druesome commented Jun 23, 2023

334584-zen

6423865-zen

I was also able to replicate this on my Atomic site, but strangely, not on my Pressable site.

@AbhinavSakalle
Copy link

Reported in 6457727-zen.

I was able to replicate it on my Pressable site.

@csmcneill csmcneill changed the title Transaction advanced filters throw an error Advanced filters throw an error Jun 27, 2023
@csmcneill
Copy link
Contributor

Renamed this issue because it does not affect just transaction filters—it also affects dispute filters.

Markup on 2023-06-26 at 19:40:31

Markup on 2023-06-26 at 19:41:42

@francoishvz
Copy link

6474495-zen

@kyajfx
Copy link

kyajfx commented Jun 28, 2023

Same issue after updating WooCommerce 7.8.1 and WC Payments 6.0
Screenshot 2023-06-28 223421

@melek melek added the priority: high The issue/PR is high priority—it affects lots of customers substantially, but not critically. label Jun 28, 2023
@melek
Copy link

melek commented Jun 28, 2023

6476791-zen

@daquinons
Copy link
Member

We've been debugging this issue here p1688006903600379-slack-CGGCLBN58

I can confirm WC 7.8 is the root cause of the issue.

My local environment was showing advanced filters with 7.6 perfectly and it wasn't until I updated it to 7.8.1 that it displayed the error.

@ricardo
Copy link
Member

ricardo commented Jun 29, 2023

Fixed in #6617.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
category: core WC Payments core related issues, where it’s obvious. component: transactions Issues related to Transactions priority: high The issue/PR is high priority—it affects lots of customers substantially, but not critically. type: bug The issue is a confirmed bug.
Projects
None yet
Development

No branches or pull requests