Add CellProperties as a parameter to validator function

[john-swallow-cenduit]

When using HoT in an Angular component the passing of the details of the cell being validated by scope only is limiting and working around it is a pain. In a usage scenario where a validator wants to make a decision based on data not present to the HoT instance itself you need to be able to access both "this" as the CellProperties but also "this" as the scope of the Component in which the HoT instance is being hosted - and it isn't straightforward to do both.

It would just be easier if the validator passed the CellProperties as a parameter in the first place. Instead of:
validator(value, callback) { // "this" is the CellProps }
This:
validator(value, callback, cellProperties: CellProperties) { // "this" is still the CellProps }

Then I could use an arrow function to keep my component level scope and yet still access the CellProps.
validator(value, callback, cellProperties: CellProperties) => { // "this" is the component but now I can access CellProps }

Take this as for instance:

/* file: app.component.ts */
import { Component } from '@angular/core';
import Handsontable from 'handsontable/base';

@Component({
  selector: 'app-root',
  template: `
    <div>
      <hot-table [settings]="hotSettings"></hot-table>
    </div>
  `,
})
export class AppComponent {
  const someDataThatTheValidatorNeeds = [1, 2, 3]; // Here is some data of some kind that the validator needs access to

  hotSettings: Handsontable.GridSettings = {
    startRows: 5,
    columns: [
      {
              validator(value, callback) {
                     console.log(this); // <- this will be the CellProperties
                     // But how can I get to someDataThatTheValidatorNeeds?
              }
      }
    ],
    colHeaders: true,
    colWidths: 200,
    licenseKey: 'non-commercial-and-evaluation'
  };
}

You can work around this by the usual "const self = this;" trick but its just a pain to have to do this:

hotSettings: Handsontable.GridSettings = {
  startRows: 5,
  columns: this.getColumns()
}

private getColumns() {
const self = this;
return [
    {
            validator(value, callback) {
                   console.log(this); // <- this will be the CellProperties
                   console.log(self.someDataThatTheValidatorNeeds); 
            }
    }
  ]
}

[adrianszymanski89]

Hi @john-swallow-cenduit

Thank you for pointing that out. We understand that the current workaround is not very convenient. I agree that this is something we should consider fixing in the future. Our roadmap is already planned, but I propose your request to our development team for the estimation.

Additionally, can you please confirm if this #7965 case is related to the same matter?

[john-swallow-cenduit]

Well I guess I#7965 is related insofar as both are about configuring HoT settings but otherwise I would not really consider these to be related.