dev/core#1603 fix tangental bug on form handling of long options
authoreileen <emcnaughton@wikimedia.org>
Mon, 31 Aug 2020 06:28:20 +0000 (18:28 +1200)
committereileen <emcnaughton@wikimedia.org>
Tue, 8 Sep 2020 20:09:23 +0000 (08:09 +1200)
commit09e6f54112d5091826792bd29dc36d0e196d79fd
tree7c6a66de8198697b0be5d1afdaf5c5a32a76769a
parent270761f2bde97d6e06c2cc88d9305145b3ad9e1a
dev/core#1603 fix tangental bug on form handling of long options

This fixes the bug that derailed the fix for https://lab.civicrm.org/dev/core/-/issues/1603

On further testing I agree with Jitendra that the price field form was mis-saving the longer values (I only
tested with Euro style decimal separators but maybe on both) and it was bad data caused by this
that made the merged fix look like a regression.

I also observed that the Money functions intended to round only appeared to since it was rounding
just fine to 2 places, as tested, but it was still rounding to 2 places when we wanted more.

This fixes both the New Price field screen (when options are entered) and the edit options screen
CRM/Price/Form/Field.php
CRM/Price/Form/Option.php