Details
-
Type:
Improvement
-
Status: Done/Fixed
-
Priority:
Trivial
-
Resolution: Fixed/Completed
-
Affects Version/s: 4.7.19
-
Fix Version/s: 4.7.23
-
Component/s: Core CiviCRM
-
Labels:
-
Versioning Impact:Patch (backwards-compatible bug fixes)
-
Documentation Required?:None
-
Funding Source:Needs Funding
-
Verified?:No
Description
Using issue numbers as variables is not very self-explanatory and needs research or prior understanding just to understand the code.
In the Civi/API/Subscriber/ChainSubscriber.php one of the variables is named after an issue and I think it would be more readable if we gave it a meaningful name that doesn't require more research.
