Skip to content

fix saving empty string#77

Open
shadik42 wants to merge 1 commit intophemellc:masterfrom
shadik42:master
Open

fix saving empty string#77
shadik42 wants to merge 1 commit intophemellc:masterfrom
shadik42:master

Conversation

@shadik42
Copy link
Copy Markdown

@shadik42 shadik42 commented Feb 21, 2018

Pull request for issue #75

@zhukovsergei
Copy link
Copy Markdown

zhukovsergei commented Feb 21, 2018

What's about to save a zero 0? It's rather common case of setting. It will not pass throw empty()

@shadik42
Copy link
Copy Markdown
Author

shadik42 commented Feb 21, 2018

Correct, and if zero provided as string value this code just skip trying of json_decode value. After that the value is saved as a string.

@schmunk42
Copy link
Copy Markdown
Contributor

But is 0 currently saved a object?

@shadik42
Copy link
Copy Markdown
Author

No, 0 saved with string type

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

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants