Skip to content

[CodingStyle] Add ManualJsonStringToJsonEncodeArrayRector#1745

Closed
TomasVotruba wants to merge 2 commits intomasterfrom
json-mix-to-json
Closed

[CodingStyle] Add ManualJsonStringToJsonEncodeArrayRector#1745
TomasVotruba wants to merge 2 commits intomasterfrom
json-mix-to-json

Conversation

@TomasVotruba
Copy link
Copy Markdown
Member

@TomasVotruba TomasVotruba commented Jul 16, 2019 •
edited
Loading 

-$someJsonAsString = '{"role_name":"admin","numberz":{"id":"10"}}';

+$data = [
+    'role_name' => 'admin',
+    'numberz' => ['id' => '10']
+];
+$someJsonAsString = json_encode($data);

@TomasVotruba TomasVotruba force-pushed the json-mix-to-json branch 3 times, most recently from cfd3bc4 to 6445665 Compare July 24, 2019 10:26
@TomasVotruba TomasVotruba changed the title [WIP] [CodingStyle] Add ManualJsonStringToJsonEncodeArrayRector [CodingStyle] Add ManualJsonStringToJsonEncodeArrayRector Jul 24, 2019
@TomasVotruba
Copy link
Copy Markdown
Member Author

TomasVotruba commented Jul 24, 2019

Replaced by #1762

@TomasVotruba TomasVotruba deleted the json-mix-to-json branch July 24, 2019 14:25
TomasVotruba added a commit that referenced this pull request Feb 2, 2022
@TomasVotruba
Updated Rector to commit 29c634f
6695b02 
rectorphp/rector-src@29c634f [Php74] Add inlinePublic configurable for TypedPropertyRector (#1745)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

No reviews

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

1 participant

@TomasVotruba