Closed aksm closed 1 year ago
In the below line, the message gets triggered whether simulating or not and has the simulate configuration value, which is an int.
https://github.com/esmero/strawberryfield/blob/7166e579434dc0590a5fab2c0b1f2bcf2a9719c8/src/Plugin/Action/StrawberryfieldJsonPatch.php#L218
The message needs to be moved to the simulate check further down and should match the one for text-based, i.e.:
https://github.com/esmero/ami/blob/f6bf9dd33ad94a44255fe2e30c031ef9f50a0b31/src/Plugin/Action/AmiStrawberryfieldJsonAsText.php#L137-L138
What?
In the below line, the message gets triggered whether simulating or not and has the simulate configuration value, which is an int.
https://github.com/esmero/strawberryfield/blob/7166e579434dc0590a5fab2c0b1f2bcf2a9719c8/src/Plugin/Action/StrawberryfieldJsonPatch.php#L218
The message needs to be moved to the simulate check further down and should match the one for text-based, i.e.:
https://github.com/esmero/ami/blob/f6bf9dd33ad94a44255fe2e30c031ef9f50a0b31/src/Plugin/Action/AmiStrawberryfieldJsonAsText.php#L137-L138