Remove clean_action_name

[rossjones]

clean_action_name ( https://github.com/ckan/ckan/blob/master/ckan/new_authz.py#L109-L113 ) is only called on IAuthFunctions implementations and renames them replacing 'package' with 'dataset'. This can end up with a user implementing an auth function test_package() which then if called with check_access('test_package') will fail because test_dataset could not be found.

This is horrible, it should be removed and documentation used to suggest people use the new naming scheme if it is important.

[wardi]

+1

[seanh]

Good idea

[davidread]

Do it