Skip to content

Remove suggestion that you *must* use the new parameters

homotechsual requested to merge github/fork/eileenmcnaughton/patch-11 into master

Created by: eileenmcnaughton

The docs were incorrectly stating that only the new properties are available - as can be demonstrated in this tested code in core - https://github.com/civicrm/civicrm-core/pull/17585/files#diff-925c5486a3ef170b6613caf3da110a18R627 - the legacy params still work and while we recommend change no action is required

@mattwire @artfulrobot in my tests the propertyBag is doing what it should here so we should not say that only the new params will work.

Note this will clash with my other docs PR regarding empty not working - if one is merged I will fix the other but I think both discussions need to be open as the docs are currently saying things I can't replicate.

Merge request reports