Fix: Trim leading zeros and keep decimal digits in formatted fields

[CarolineDenis]

Fixes #7451

Checklist

• Self-review the PR after opening it to make sure the changes look good and
  self-explanatory (or properly documented)
• Add relevant issue to release milestone

Testing instructions

• Create a new field formatter for cat num and check 'trim leading zero'
• Create a new field formatter for locality, add longitude and latitude to the fields, check trim zeros for each.
• Open the QB with CO as base table
• Add CO formatted and locality formatted
• Verify that the leading 0s are removed for CO and that the decimals after the longitude and latitude are kept.

[alesan99]

• Verify that the leading 0s are removed for CO and that the decimals after the longitude and latitude are kept.

---

The formatting is working as expected 👍

The frontend formatter remains unchanged, but I think that's fine since it trims the trailing zeros anyway. Just making a note.

specify7/specifyweb/frontend/js_src/lib/components/Formatters/formatters.ts

Line 200 in a67df61

formatted = Number.isNaN(Number(formatted))

---

Though since the setting isn't just used to remove leading zeros anymore, could it be renamed to "Trim zeros"?

[Iwantexpresso]

• Verify that the leading 0s are removed for CO and that the decimals after the longitude and latitude are kept.

nice this looks good!

[acwhite211]

Looks good, might need a small change for handling different formats.

[acwhite211]

I think this needs a small change in the trimzeros logic for handling different formatting.

Looks like the decimal loss case for numeric values is fixed, but it also removes the old guard that only normalized values when the entire value was numeric. As a result, values that only look numeric at the start or end can now be rewritten unexpectedly.

We can keep this fix, but only apply it when the full value matches the numeric pattern:

def apply_stringish(expr):
    e = expr

    if fieldNodeAttrib.get('trimzeros') == 'true':
        e = cast(e, types.String())
        trimmed = e

        # remove leading zeros
        trimmed = func.regexp_replace(trimmed, r'^(-?)0+([0-9])', r'\1\2')

        # remove trailing zeros after decimal
        trimmed = func.regexp_replace(trimmed, r'(\.[0-9]*?)0+$', r'\1')

        # remove trailing decimal point
        trimmed = func.regexp_replace(trimmed, r'\.$', '')

        e = case((e.op('REGEXP')(r'^-?[0-9]+(\.[0-9]+)?$'), trimmed), else_=e)

    fmt = fieldNodeAttrib.get('format')
    if fmt is not None:
        e = self.pseudo_sprintf(fmt, e)

    sep = fieldNodeAttrib.get('sep')
    if sep is not None:
        e = concat(sep, e)

    return e

Examples cases:

Input	Current PR result	New result
0001-A	1-A	0001-A
ABC.2300	ABC.23	ABC.2300
INV0007.000	INV0007	INV0007.000

Numeric inputs would continue to work as intended:

Input	Current PR result	New result
001.230400	1.2304	1.2304
0000	0	0

Let me know if that would be a good change or not?