Details
-
Type:
Bug
-
Status: Done/Fixed
-
Priority:
Trivial
-
Resolution: Fixed/Completed
-
Affects Version/s: 4.7
-
Fix Version/s: 4.7
-
Component/s: None
-
Labels:None
-
Documentation Required?:None
-
Funding Source:Needs Funding
Description
The form function $this->getContactID actually sets the contact ID as well as retrieving it.
I propose to rename it setContactID & to add a function getContactID that just wraps it to reduce code change. Leaving it as getContactID risks the function call being removed when the resulting variable is not required