{"push_id":1114445693,"size":2,"distinct_size":2,"ref":"refs/heads/Issue2271","head":"b5754903ad99e3c21319677e1f064bd2d2480c4d","before":"b429d4aee7ed27912df71500d88b6956c0f08004","commits":[{"sha":"ecd7588425b5b05873f5dc02721c1c2e215d8a76","author":{"email":"1f3d3733d1e0b2e0f75751244eaa544479daf1ab@lightsource.ca","name":"Michael Butler"},"message":"Modified:\n\nAMControl::addChildControl had a logic bug in its conditional requiring\na child to exist in the children_ list before it could be added. I\nsuspect this would never allow a child control to be added. Since all\ninheriting classes redefine the method this likely was not a serious\nissue.\n\nUndid my initial refactors as the whole hierarchy requires work which\nwill change my approach.","distinct":true,"url":"https://api.github.com/repos/acquaman/acquaman/commits/ecd7588425b5b05873f5dc02721c1c2e215d8a76"},{"sha":"b5754903ad99e3c21319677e1f064bd2d2480c4d","author":{"email":"1f3d3733d1e0b2e0f75751244eaa544479daf1ab@lightsource.ca","name":"Michael Butler"},"message":"Modified:\n\nAMBeamlineControlGroup\nAMBeamlineControlGroupStatus\nAMCompositeControl\nAMConnectedControl\nAMPseudoMotorControl\nBioXASBiStateGroup\n\nRedefined reimplementations of addChildControl and removeChildControl in\ninherted classes to use the AMControl method to limit exposure to\nchildren_ outside of AMControl. Adjusted those implementations to also\nreturn bool true/false conditions.","distinct":true,"url":"https://api.github.com/repos/acquaman/acquaman/commits/b5754903ad99e3c21319677e1f064bd2d2480c4d"}]}