Skip to content

Fix PHP deprecated semicolon syntax in case statement#529

Closed
Copilot wants to merge 4 commits intomasterfrom
copilot/fix-deprecated-case-statements
Closed

Fix PHP deprecated semicolon syntax in case statement#529
Copilot wants to merge 4 commits intomasterfrom
copilot/fix-deprecated-case-statements

Conversation

Copy link
Copy Markdown

Copilot AI commented Mar 24, 2026

PHP 8.x deprecates using a semicolon (;) instead of a colon (:) after case labels. This triggers a visible deprecation notice on every page load after plugin installation.

Change

  • classes/fields.class.php line 528: Replace ; with : in the case 'paired-palettes' statement
// Before
case 'paired-palettes';

// After
case 'paired-palettes':
Original prompt

This section details on the original issue you should resolve

<issue_title>Deprecated: Case statements followed by a semicolon (;) are deprecated, use a colon (:) instead in /s5-bowbubble/wordpress/wp-content/plugins/woocommerce-product-addon/classes/fields.class.php on line 528</issue_title>
<issue_description>### Description

Deprecated: Case statements followed by a semicolon (;) are deprecated, use a colon (:) instead in /s5-bowbubble/wordpress/wp-content/plugins/woocommerce-product-addon/classes/fields.class.php on line 528

Step-by-step reproduction instructions

Just install the plugin
Image

Screenshots, screen recording, code snippet or Help Scout ticket

No response

Environment info

No response

Is the issue you are reporting a regression

No</issue_description>

Comments on the Issue (you are @copilot in this section)


📍 Connect Copilot coding agent with Jira, Azure Boards or Linear to delegate work to Copilot in one click without leaving your project management tool.

Copilot AI changed the title [WIP] Fix deprecated case statements by replacing semicolons with colons Fix PHP deprecated semicolon syntax in case statement Mar 24, 2026
Copilot AI requested a review from rodica-andronache March 24, 2026 16:30
@rodica-andronache rodica-andronache requested review from Soare-Robert-Daniel and removed request for rodica-andronache March 24, 2026 16:32
@rodica-andronache rodica-andronache removed their assignment Mar 24, 2026
@rodica-andronache
Copy link
Copy Markdown

@copilot change the PR to the development branch, instead of master

Copy link
Copy Markdown
Author

Copilot AI commented Mar 25, 2026

@copilot change the PR to the development branch, instead of master

The branch has been rebased on top of development. However, the PR base branch setting on GitHub cannot be changed programmatically with the available credentials (403 Forbidden). Could you please manually update the PR base from master to development in the GitHub UI? The branch already contains development's history, so the merge should be clean.

Warning

Firewall rules blocked me from connecting to one or more addresses (expand for details)

I tried to connect to the following addresses, but was blocked by firewall rules:

  • https://api.github.com/graphql
    • Triggering command: /usr/bin/gh gh pr edit 529 --base development (http block)
    • Triggering command: /usr/bin/gh gh pr view 529 --json baseRefName,headRefName (http block)
    • Triggering command: /usr/bin/gh gh auth status (http block)
  • https://api.github.com/repos/Codeinwp/woocommerce-product-addon/pulls/529
    • Triggering command: /usr/bin/gh gh api --method PATCH /repos/Codeinwp/woocommerce-product-addon/pulls/529 -f base=development (http block)

If you need me to access, download, or install something from one of these locations, you can either:

@rodica-andronache
Copy link
Copy Markdown

@copilot close this PR, and create a new one, containing the same fix. However that PR should have the development branch as base, instead of master

Copy link
Copy Markdown
Author

Copilot AI commented Mar 25, 2026

@rodica-andronache I've opened a new pull request, #531, to work on those changes. Once the pull request is ready, I'll request review from you.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

3 participants