I take the code back in so someone else can deal with it as I will be upgrading to 5.2 and dave advised me to not remove code but try and fix the bug, which, taken that i will upgrade anyway, wont happen.
Mass commit. Made sure that role_two and object_id_two work with the new relationship adding. Made sure the callback for relationships contains the package_id, I18N groupnames
replacing foreach with util_unlist for telecom_number and postal_address widget types since it cause a bug, also move attribute::help_text proc to the ad_form_widget part of the request switch so that the lookup only takes place when it needs to and adding help_text message to mobile_number, aim and skype widget types