Replace `Number()` constructors with a utility method when parsing column ids

[pavish]

Description

• Prior to Refactor table view system to use string-based column ids #4975, column ids were both numbers and strings based on the context. Eg., the table page used numbers and data explorer used strings.
• In Refactor table view system to use string-based column ids #4975, we decided to move all usage to string-based.
  • We now use the Number constructor in several places where columnId is expected to be a number.
• The Number constructor has this behaviour:
  • Number(null) and Number("") return 0.
  • Number("randomstring") is NaN
• While this doesn't cause any visible issues, this might come back to affect us in the future, as it opens a number of possible edge cases, where null needs to be supported.

Changes required

• The Number constructor usages for column.id should be replaced with a utility which handles the null, empty string, and NaN scenarios.

[undefinedIsMyLife]

Hi! I’d like to work on this issue.
Before starting, can I confirm the expected behavior of the utility function?
What should the function return when:
id is null
id is "" (empty string)
id is not numeric (e.g., "abc")
Thanks!