Misinformation in the inline code comment documentation for get_term_children

[simonwheatley]

The inline documentation around get_term_children suggests that it will take a term name, the function argument $term suggests it would take a Term object; neither is true, it takes a term ID and returns an array of term IDs.

I propose the attached patch which changes the function arg name to $term_id, and amends the inline documentation.

The only addition, potentially mildly contentious I suppose, is that the diff uses absint to ensure the term ID passed is definitely an integer.

I think you should use intval instead of absint. Although term ids are not going to be negative in most cases (thanks to auto-increment), as far as I know nothing precludes them from being negative integers.

Docs clarification, arg var name change, term ID arg cast to intval

Thanks for the suggestion filosofo, diff changed accordingly.

(In [11115]) get_term_children phpdoc and var naming improvements. Props simonwheatley. fixes #9667