Looking for some guidance - use of constants in cacti code

[Howie]

So rather than working on Weathermap stuff, I've been playing with adding extra data types to graph tree for the last couple of days... so far so good (it's not done yet though). It brings up a couple of questions though:

Is the stuff in global_constants.php arriving or leaving? Some code in tree* uses it, and some uses a plain string for the type. I assume that it should be updated to use the constants instead?

Second, how should I submit this stuff? It's going to be fairly heavy patches, but only to a few files, plus some additional database fields in graph_tree_items.

[BSOD2600]

I'd assume patches should be submitted through a feature report in the bug tracking system.