BuddyPress.org

I have done this by

1. If user is not is_site_admin() it calls remove_action

2. Added a new action to my own function, which basically is a copy of BuddyPress function except the code to add bp_core_add_subnav_item

Is it a good practice to avoid any hacking to the core? Please suggest.