Skip to content

Prevent fatal error if WC()->cart is null or not an instance of WC_Cart#131

Merged
circlecube merged 1 commit intomainfrom
fix/PRESS10-166
Apr 14, 2025
Merged

Prevent fatal error if WC()->cart is null or not an instance of WC_Cart#131
circlecube merged 1 commit intomainfrom
fix/PRESS10-166

Conversation

@crodriguezbrito
Copy link
Copy Markdown
Contributor

Proposed changes

The following PR will add a control check for prevent a fatal error if WC()->cart does not exists. #PRESS10-166

Type of Change

Production

  • Bugfix (non-breaking change which fixes an issue)
  • New feature (non-breaking change which adds functionality)
  • Breaking change (fix or feature that would cause existing functionality to not work as expected)
  • Dependency update
  • Refactoring / housekeeping (changes to files not directly related to functionality)

Development

  • Tests
  • Dependency update
  • Environment update / refactoring
  • Documentation Update

Video

Checklist

  • I have read the CONTRIBUTING doc
  • I have viewed my change in a web-browser
  • Linting and tests pass locally with my changes
  • I have added tests that prove my fix is effective or that my feature works
  • I have added necessary documentation (if appropriate)

Further comments

@crodriguezbrito crodriguezbrito requested review from a team and Copilot April 7, 2025 09:08
Copy link
Copy Markdown
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Copilot wasn't able to review any files in this pull request.

Files not reviewed (1)
  • includes/Listeners/WonderCart.php: Language not supported

@circlecube circlecube merged commit 2097334 into main Apr 14, 2025
13 of 14 checks passed
@circlecube circlecube deleted the fix/PRESS10-166 branch April 14, 2025 15:53
@circlecube circlecube mentioned this pull request Apr 14, 2025
14 tasks
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.

3 participants