to control flow

[surreal olive]

following the automatic conversion of an old application I find myself with an error that I cannot correct. can you help me please

ng generate @angular/core:control-flow

@for (line of datas; track line) {
  ...
  @for (column of columnsToDisplay; track column; let i = $index) {
    ...
    @if (column.classCss) {
      <div [ngClass]="column.classCss(line[column.field])">     // HERE ERROR  --> see picture
        ...

datas

export class TableV2LoadData<T> implements OnInit, AfterViewInit {
  public datas: T[];
...

column ( is type of TableColumn)

export interface TableColumn {
...
  field?: string;
...
  classCss?: () => {};
...

[muted solstice]

The function in classCss says it does not accept any parameters, why are you handing it a parameter?

[surreal olive]

with the old code, it was like that, with a parameter it was planned like that.

[muted solstice]

Then remove the parameter. That looks like a type error that for some reason your TS didn't catch before, or tsconfig settings were changed or the like

[surreal olive]

Ah OK thank
I hope there will be no regression

[muted solstice]

That's what you have unit tests for.
If you don't, take it as a lesson that you should have unit tests

[surreal olive]

It’s a big project that I inherited. and my boss will never want to give me 3 months to do unit tests.. we always have constraints that prevent us from doing things well 😦

[muted solstice]

Start learning to do convincing work.
Show them that this will now take longer and this will be the case every time you do any upgrade of any sort. You have no guarantees that things will work and you have a massive risk that things break over time

It's not like you have to do them all at once, just start writing unit tests as part of your ticket or fully unit-test out all components you touch

[surreal olive]

yes I understand well.
and what's more, I'm all alone lol