LoadingController not injected in the component and remains undefined

I have this strange situation where I don’t understand why the LoadingController is not injected.
this is my code:

  selector: 'app-login',
  templateUrl: './login.page.html',
  styleUrls: ['./login.page.scss'],
export class LoginPage implements OnInit {

  constructor( private loadingController: LoadingController) {

  ngOnInit() {

  showLoading(show: boolean) {
        message: "Login...",
        animated: true,
        spinner: "circles",
        duration: 5000
      }).then(loadEl => {
        if (show) {
        } else {

When I run my app I have this error:

ERROR TypeError: Cannot read properties of undefined (reading 'create')
    at SafeSubscriber.showLoading [as _next] (login.page.ts:61)

LoadingController is undefined, what am I doing wrong?

Hmm, can you put this into a quick demo app? Trying this localy I cannot replicate

Right, I’ll do it tomorrow it’s 9:30pm here :grin: ! Do I need to import anything special in the LoginModule? Currently IonicModule is well imported so I don’t understand where the issue comes from.

That should be all you need, but without more context, it’s hard to know

I don’t see the point of the show == false code path of showLoading. Why create a loading indicator just to murder it before it is ever presented?

It’s just a snippet to show the principle. In my app the boolean comes from an Observable.
@mhartington I tried to reproduce the issue in a sample app with the same folder structure but I can’t, sadly the LoadingController is well injected, I have to find the reason it is not injected in my real app! :unamused:

@mhartington I finaly found the cause of my issue, it’s due to my lack of JS understanding.
I used the showLoading method in an observable as a function reference, something like:

ngOnInit() {

Replacing that line by:

ngOnInit() {

solved the issue.
I am a Java developer trying to build an Ionic app with Angular.

Hi there

I am not an expert on js but I wouldn’t use bind in this case

Would use a fat arrow function:


Maybe a bit more readable and reusable

Doesnt bind(this) require you to look somewhere else what will be done as a result of an item on the stream. ?

I guess that bind(this), the this in this context refers to the LoginPage class.

And in this case it looks pretty concise

You would use bind as well for the error and final handler of a stream (if relevant?)

Something like subscribe(bind(this),bind(this),bind(this))? Would that work?

Yes it would work. If the method has the expected signature, it would work.

Can’t say it looks particularly pretty :grinning: