SIT-3022: fix/ Rogers-crossposting: update utils.php with client code for cross posting

[YMufleh]

Description of the Change

Please inspect this updated code as received by client and advise us if there will be any breaking changes introduced.

Notes from client:
It's an issue if the function is called multiple times, the functions in those lines are in a function, which declares those functions every time the function is called.
Removing those functions from the containers functions fixes the problem.
This is a problem if you do multiple post inserts in the same script, which is the case on News because we crossposts some posts through our different sites.

The issue as actually happening because we do multiple wp_insert_posts in the same PHP script. Maybe I misspoke the right term, but basically, since we have a loop that create multiple posts, in one go, and that this function called get_post_categories has function declarations in it, there is a Fatal error in PHP since you can't declare two functions with the same name.
We crosspost stories from one site to multiple sites when the story is relevant for the other sites, most likely when the story is national.
The crossposting functionality duplicate the post on another site, and links them, so when the "parent" article gets an update, the "children" articles get the same updates
Personally, I would suggest not having named functions or not have function declarations in a function.

[YMufleh]

@jeffpaul looks like the code change is minimal here only wrapping function calls with others.

Will leave it up to you to decide on if this change is good or breaks current logic.
Thank you

[jeffpaul]

@oscarssanchez tagging you for code review here

[jeffpaul]

Relates to https://voltron.atlassian.net/servicedesk/customer/portal/41/SIT-3022

[sonarqubecloud]

Kudos, SonarCloud Quality Gate passed!   

0 Bugs
0 Vulnerabilities
0 Security Hotspots
0 Code Smells

No Coverage information
0.0% Duplication