Moving from date to date range in search widget causes fatal error

GravityKit / GravityView

The best and easiest way to display Gravity Forms entries on your website.

https://www.gravitykit.com/products/gravityview/

245 stars 63 forks source link

Moving from date to date range in search widget causes fatal error #2115

Closed omarkasem closed 2 months ago

omarkasem commented 2 months ago

Moving from date to date range in search widget causes fatal error to recreate it:

Add a field that has date and date range in search widget like https://github.com/GravityKit/Event-Field/pull/63
Do a search with a date then go back and change the type to date range
When you refresh the page with the old search parameters it causes a fatal error

mrcasual commented 2 months ago

@omarkasem, we were unable to reproduce the issue using develop on GravityView and issue/53-support-gravityviews-search-bar2 on Event Field. Could you please record a video or reproduce it on https://site.try.gravitykit.com/?

rafaehlers commented 2 months ago

I was able to reproduce it and the PR fixed it!

mrcasual commented 2 months ago

I was able to reproduce it and the PR fixed it!

@rafaehlers, which WP/PHP/GV versions are you using? Did you follow @omarkasem's steps? Any other suggestions for reproducing this consistently, given that both @Mwalek and I were unsuccessful?

rafaehlers commented 2 months ago

@mrcasual I followed Omar's steps precisely as he described it.

mrcasual commented 2 months ago

@rafaehlers, OK, since you were able to reproduce it, please come up with a changelog entry.

rafaehlers commented 2 months ago

@mrcasual

Fixed: Fatal Error that happened when a date search is performed against incorrect date fields.