Error when value is an object - validating api data

[daycry]

For example:

{
	"configuration" :
        {
	        "consumerKey1" : "aa",
	        "consumerSecret" : "bb",
	        "oauthToken" : "cc",
	        "oauthTokenSecret" : "dd"
        }
}

Description
Explain what you have changed, and why.

Checklist:

• Securely signed commits
• Component(s) with PHPdocs
• Unit testing, with >80% coverage
• User guide updated
• Conforms to style guide

[samsonasik]

Please provide unit test and run cs fix:

vendor/bin/php-cs-fixer fix system/Validation/Validation.php

[daycry]

Done.

[samsonasik]

please run cs fix

[daycry]

Done, sorry. El mar, 28 sept 2021 a las 14:50, Abdul Malik Ikhsan (< ***@***.***>) escribió:

…

please run cs fix — You are receiving this because you authored the thread. Reply to this email directly, view it on GitHub <#5142 (comment)>, or unsubscribe <https://github.com/notifications/unsubscribe-auth/ABZ5DPZUYWTLJJHW6D2BISLUEG2YNANCNFSM5E2RNU7Q> . Triage notifications on the go with GitHub Mobile for iOS <https://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675> or Android <https://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub>.

[daycry]

Done, sorry.

[samsonasik]

cs still error, see https://github.com/codeigniter4/CodeIgniter4/pull/5142/checks?check_run_id=3733816746#step:8:20

[daycry]

ok,
I have launched the processes for fix it

• vendor/bin/php-cs-fixer fix .\tests\system\Validation\ValidationTest.php
• vendor/bin/php-cs-fixer fix .\tests_support\Validation\TestRules.php

[daycry]

If you try to validate an object and it fails, the application fails because it only accepts strings and arrays, it does not do a json_encode if it is an object, because when you work with rest, you may have to validate objects.

…

On Wed, 29 Sep 2021, 16:48 John Paul E. Balandan, CPA, < ***@***.***> wrote: ***@***.**** commented on this pull request. ------------------------------ In tests/system/Validation/ValidationTest.php <#5142 (comment)> : > + $data = (object) ['configuration' => (object) ['first' => 1, 'second' => 2]]; + $this->validation->run((array) $data); Hmm. Fair enough. Thanks for the explanation. — You are receiving this because you authored the thread. Reply to this email directly, view it on GitHub <#5142 (comment)>, or unsubscribe <https://github.com/notifications/unsubscribe-auth/ABZ5DPYBOBG5OGAKCANSHGDUEMRK5ANCNFSM5E2RNU7Q> . Triage notifications on the go with GitHub Mobile for iOS <https://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675> or Android <https://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub>.