global: general improvements

[drjova]

DO NOT merge yet

[jirikuncar]

👍

[drjova]

The PR has been updated with a more DRY approach for different use cases, it's possible to change the data and clean logic with a service decorator:

.config([ '$provide', function($provide) {

  $provide.decorator('InvenioRecordsAPI', [
    '$delegate',
    function myServiceDecorator($delegate) {

      // keep the existing function
      var getData = $delegate.getData;

      function newCleanData(data, unwanted) {
        // unicorns and stuff
        return data;
      }

      function newGetData(model, extraParams, endpoints) {
        var _model = {
            metadata: model
        };
        // More unicorns and stuff

        // Pass the arguments to the existing function without re-writting the code
        getData.apply($delegate, _model, extraParams, endpoints);
      }

      // Override the cleanData
      $delegate.cleanData = newCleanData;

      // Override the getData
      $delegate.getData = newGetData;

      // return the delegate
      return $delegate;
    }
  ]);
}]);

[pamfilos]

👍 LGTM!

[ioannistsanaktsidis]

This one works for me and @pamfilos. I think it can go on.
cc @jirikuncar , @lnielsen