Closed jakejackson1 closed 1 year ago
Merging #1454 (c222ee0) into development (f9649f0) will increase coverage by
17.58%
. The diff coverage isn/a
.:exclamation: Current head c222ee0 differs from pull request most recent head b6fa1d8. Consider uploading reports for the commit b6fa1d8 to get more accurate results
@@ Coverage Diff @@
## development #1454 +/- ##
==================================================
+ Coverage 75.71% 93.30% +17.58%
==================================================
Files 243 57 -186
Lines 12828 1464 -11364
Branches 370 370
==================================================
- Hits 9713 1366 -8347
+ Misses 3107 90 -3017
Partials 8 8
see 186 files with indirect coverage changes
Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.
Description
The conditional logic JSON was being passed through
sanitize_text_field()
, instead of\GFFormsModel::sanitize_conditional_logic()
. This update ensures the JSON data is correctly decoded, sanitized, and then recoded.Testing instructions
Let automated tests run.
Screenshots
Checklist:
Additional Comments