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

Check if value has start and end dates #2116

Conversation

omarkasem
Copy link
Collaborator

@omarkasem omarkasem commented Aug 18, 2024

@omarkasem omarkasem linked an issue Aug 18, 2024 that may be closed by this pull request
@omarkasem omarkasem self-assigned this Aug 18, 2024
@mrcasual
Copy link
Collaborator

@Mwalek, please test.

@mrcasual mrcasual self-assigned this Aug 18, 2024
@mrcasual mrcasual requested a review from Mwalek August 18, 2024 16:35
@Mwalek
Copy link

Mwalek commented Aug 19, 2024

@mrcasual I was not able to reproduce the issue without the fix. So I can't confirm that this is fixed.

I tested without the solution and got the following error:

<br /><b>Warning</b>:  Illegal string offset ‘start’ in <b>/.../public/wp-content/plugins/gravityview/includes/widgets/search-widget/templates/search-field-date_range.php</b> on line <b>20</b><br />0″>		<input name=

After adding the solution (using this build), the warning no longer appears.

If you can confirm that the problem is the same, then we can consider the issue resolved. Otherwise, please share additional details for reproducing the problem.

✔ Tested on PHP Version 7.4.30.

@mrcasual
Copy link
Collaborator

@omarkasem, please see @Mwalek's response.

@rafaehlers
Copy link
Contributor

I was able to reproduce it:

image

This PR fixed it ✅

@mrcasual mrcasual merged commit d8522b3 into develop Aug 22, 2024
1 check passed
@mrcasual mrcasual deleted the issue/2115-moving-from-date-to-date-range-in-search-widget-causes-fatal-error branch August 22, 2024 14:45
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

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