refactor(http): Removed deprecated HTTP_PROVIDERS and JSONP_PROVIDERS

[chuckjaz]

Please check if the PR fulfills these requirements

• The commit message follows our guidelines: https://github.com/angular/angular/blob/master/CONTRIBUTING.md#commit-message-format
• Tests for the changes have been added (for bug fixes / features)
• Docs have been added / updated (for bug fixes / features)

What kind of change does this PR introduce? (check one with "x")

[ ] Bugfix
[ ] Feature
[ ] Code style update (formatting, local variables)
[x] Refactoring (no functional changes, no api changes)
[ ] Build related changes
[ ] CI related changes
[ ] Other... Please describe:

What is the current behavior? (You can also link to an open issue here)

The http module exports HTTP_PROVIDERS and JSONP_PROVIDERS that need to be removed.

What is the new behavior?

The previously deprecated HTTP_PROVIDERS and JSONP_PROVIDERS are removed.

Does this PR introduce a breaking change? (check one with "x")

[x] Yes
[ ] No

If this PR contains a breaking change, please describe the impact and migration path for existing applications: ...

The previously deprecated HTTP_PROVIDERS and JSONP_PROVIDERS are removed.

Other information:

BREAKING CHANGE: previously deprecated HTTP_PROVIDERS and JSONP_PROVIDERS were removed; see deprecation notice for migration instructions.

[robertlevy]

can you provide a link to the deprecation notice for this?

[dagilleland]

The deprecation notice in the release notes says that it is deprecated, but gives no guidance as to what we should replace it with.

Specifically, I have discovered that the migration guidance should be:

1. Remove HTTP_PROVIDERS and JSONP_PROVIDERS from your application module
2. Use the HttpModule and JsonpModule in your application module's imports.

e.g.: Instead of this

@NgModule({
  declarations: [AppComponent],

  providers:    [
    HTTP_PROVIDERS,
    JSONP_PROVIDERS,
      ],
   ...
})
export class AppModule { }

use this

@NgModule({
  declarations: [AppComponent],

  imports: [
    HttpModule,
    JsonpModule
  ],
   ...
})
export class AppModule { }

[maartentibau]

Could you also provide an example if you want to "overwrite" the HTTP_PROVIDERS from the HttpModule?

e.g. in my RC.4 app I had this custom provider.

import {ReflectiveInjector} from '@angular/core';
import {Http, HTTP_PROVIDERS} from '@angular/http';
import {HttpAuth} from '../services/http-auth.service';

let relativeInjector = ReflectiveInjector.resolveAndCreate([HTTP_PROVIDERS]);
let http = relativeInjector.get(Http);

export const HTTP_AUTH_PROVIDERS: any[] = [
  {provide: 'httpAuth', useValue: http},
  {provide: Http, useClass: HttpAuth},
];

That way the original HTTP_PROVIDERS could be overwritten and I could inject my own HTTP_AUTH_PROVIDERS in the main.ts and still use the normal import {Http} from '@angular/http'; statement in my other services.

[angular-automatic-lock-bot]

This issue has been automatically locked due to inactivity.
Please file a new issue if you are encountering a similar or related problem.

Read more about our automatic conversation locking policy.

_{This action has been performed automatically by a bot.}