search

keboola / php-component

General library for PHP applications running in Keboola Connection environment
MIT License
0 stars 1 forks source link

Add decode/encode functions to JsonFileHelper #52

Closed Actimel closed 5 years ago

Actimel commented 5 years ago

Zaroven jsem prejmenoval i tu tridu. Kdyz to umi i obycejny encode/decode, tak by bylo zvlastni aby se jmenovala JsonFileHelper

tomasfejfar commented 5 years ago

Haha, koukám, že jsem tady celkem posral review :)

https://github.com/keboola/php-component/blob/master/tests/JsonHelperTest.php#L151

Test tady je a testuje skutečné chování. Jenže ono by se to mělo chovat jinak! Opravuju v rámci #53

https://github.com/keboola/php-component/blob/master/src/JsonHelper.php#L51