`no-array-method-this-argument` false positives on non-array methods

Metriche repository

Star

5022 stelle (5022 stelle)

Metriche merge PR

Merge medio 4h 30m (Merge medio 4h 30m)26 PR mergiate in 30 g (26 PR mergiate in 30 g)

Descrizione

export interface ErrorMapperInterface {
  /**
   * Maps an error to a string.
   * @param {string} key the error key. i.e., required, etc
   * @param error the error object
   * @param control control Object (AbstractControl for Angular, NgModel for AngularJS)
   * @returns {string}
   */
  map(key: string, error?: any, control?: any): string;
}

declare const mapper: ErrorMapperInterface

mapperService.map(firstError, errors[firstError])

Guida contributor

Direzione di ricerca

Il bug si trova nella regola `no array method this argument`. La regola probabilmente controlla le chiamate di metodo sugli array ma segnala erroneamente altre chiamate di metodo come `mapperService.map(firstError, errors[firstError])`. Indaga l'implementazione della regola nel codice sorgente, in particolare il visitatore `CallExpression` e la sua logica per determinare se l'oggetto chiamato è un array. Esamina il file di test della regola per comprendere i casi esistenti e aggiungi un test fallito che riproduca questo falso positivo. I commenti dell'issue potrebbero contenere ulteriori approfondimenti da parte dei manutentori o degli utenti.

Tech stack

javascripttypescript

Dominio

developer experiencetooling

Tipo issue

Bug

DifficoltàQuanto dovrebbe essere impegnativa per un nuovo contributor.

2

Tempo stimatoTempo stimato per completare e verificare uno scope piccolo.

1-3 ore

Stato attivitàQuanto è probabile che la issue sia ancora attiva e reviewable.

Attiva

ChiarezzaQuanto chiaramente la issue descrive problema, scope e risultato atteso.

Chiara

Prerequisiti

Node.jsESLint

Adatta ai principiantiQuanto la issue è adatta a contributor alla prima esperienza.

75