Only call getGroupACLRoles if contactID is present

[eileenmcnaughton]

Overview

Only call getGroupACLRoles if contactID is present

Before

I don't see how the query could ever return anything if contactID is not a number

After

Not called if contactID is not a number

Technical Details

@seamuslee001 I kept this small as it needs a bit of thinking through but I can't see a reason why it would make sense to call the function with no contactID

Comments

[civibot]

(Standard links)

• If this is your first pull-request for CiviCRM, please browse CONTRIBUTING.md for information about the development and testing processes.
• If you are reviewing this pull-request, you may wish to consult the test sites and the Review Standards (long template, short template).

[seamuslee001]

@eileenmcnaughton if the user is Anonymous then what would have been $contact_id here 0?

[eileenmcnaughton]

@seamuslee001 that's why I moved that part back out of the function & to the calling function

[eileenmcnaughton]

@seamuslee001 did ^^ answer your question?

[seamuslee001]

yep i think this is ok merging