Skip to content
This repository has been archived by the owner on Feb 24, 2020. It is now read-only.

Handling NULL in requests #65

Merged
merged 11 commits into from
Jun 26, 2017
Merged

Handling NULL in requests #65

merged 11 commits into from
Jun 26, 2017

Conversation

mickadoo
Copy link
Collaborator

Because of the way the database layer handles null values and string null we need to handle either string 'null' or true NULL input.

Since string 'null' is used to unset values it can't be used. When you pass true NULL as a value it will be ignored. To fix these problems this PR provides this workaround:

  • If you set true NULL as a value it will be converted to string 'null' so the value will be unset
  • If you set string 'null' as a value it will be converted to 'Null' since that's the only variation of string null that is allowed.

@colemanw colemanw merged commit 15088e4 into civicrm:master Jun 26, 2017
@mickadoo mickadoo deleted the 53_much_ado_about_null branch July 6, 2017 19:51
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants