When can TableElement.currentData be undefined?

[irossimoline]

When can TableElement.currentData be undefined?

[irossimoline]

Hi @hossameldeen,

I think you have found an issue. The thing is that originalData could be undefined, but currentData shouldn't: the first one is only populated when editing an element, and the last one is the one that holds the row data, so it should not be undefined. I'm not with my PC near now, but I can fix it and push soon.

[irossimoline]

Hi @hossameldeen,

I've published a new version with this fix. :)
If you have any question please let me know.

[hossameldeen]

@irossimoline , I was writing this comment when you pushed the fix :-) Thanks for the prompt response and for the awesome package!

I've checked it, will upgrade. Nope, no more questions 👍 , you can close.

By the way, I'm a fan of strictNullChecks. So, I've tried to enable it on your project and resolve the errors. One error that particularly caught my eye is this:

  cancelOrDelete(): void {
    if (this.id == -1 || !this.editing)
      this.delete();
    else {
      this.currentData = this.originalData; // I've made currentData required while originalData optional, so an error came up here for assigning optional to required
      this.editing = false;
      this.validator.disable();
    }
  }

Of course, the if-condition above !this.editing makes sure that we're in editing mode in the other branch & thus have originalData. But how to make sure that editing == true -implies-> originalData !== undefined across the whole code?

For the purposes of technical curiosity, I've made a solution & reached it to passing compilation (didn't even test). I probably don't recommend merging it into the package since it'll break backward compatibility and may need a bit refactoring. But anyway, I'm opening a PR with it in case you want to check it out :-)

Thanks again :-)

[irossimoline]

Cool @hossameldeen!

I'm gonna close this issue, and lets talk about the code improvement proposed directly on the PR.
Thank you in advance for contributing with this project! 😃