Update php snippet due to PSR-12 #548

oldy777 · 2025-02-03T14:58:32Z

The opening brace for the class MUST go on its own line; the closing brace for the class MUST go on the next line after the body.

Definition for methods

Method and function names MUST NOT be declared with space after the method name. The opening brace MUST go on its own line, and the closing brace MUST go on the next line following the body. There MUST NOT be a space after the opening parenthesis, and there MUST NOT be a space before the closing parenthesis.

OkelleyDevelopment

Hi @oldy777, the changes look good. I do have a question about one (see my other comment). Once that's clarified, then I'll approve and merge. Thanks!

OkelleyDevelopment · 2025-04-20T21:48:49Z

snippets/php/php.json

@@ -174,12 +191,12 @@
        "description": "Foreach loop"
    },
    "$… = array (…)": {
-        "prefix": "array",
+        "prefix": "arrayold",


What is the purpose for renaming these array initializers?

Hi, because the short array syntax is the default today, and the old syntax is only used for older versions of PHP.

php psr-12 update

cea5f00

OkelleyDevelopment reviewed Apr 20, 2025

View reviewed changes

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

Update php snippet due to PSR-12 #548

Update php snippet due to PSR-12 #548

oldy777 commented Feb 3, 2025

OkelleyDevelopment left a comment

OkelleyDevelopment Apr 20, 2025

oldy777 Apr 21, 2025 •

edited

Loading

Update php snippet due to PSR-12 #548

Are you sure you want to change the base?

Update php snippet due to PSR-12 #548

Conversation

oldy777 commented Feb 3, 2025

OkelleyDevelopment left a comment

Choose a reason for hiding this comment

OkelleyDevelopment Apr 20, 2025

Choose a reason for hiding this comment

oldy777 Apr 21, 2025 • edited Loading

Choose a reason for hiding this comment

oldy777 Apr 21, 2025 •

edited

Loading