Bug Bounty Report: remaining typos in natspec at 2.6.1 (wich/whitch/intefraces/suported/overriden)

Aperta
#301 1 commento 0 reazioni 0 assegnatari Vedi su GitHub

@Jah-yee ci sta già lavorando.

Dal 11/10/2026.

  • #302 di @Jah-yee — aperta

Valutazione

Difficoltà
1/5
Tempo stimato
1-3 ore
Idoneità per principianti
25/100
Tipo di issue
Documentazione
Chiarezza
Specificata chiaramente
Stato di attività
Attiva
Stack tecnologico
solidity
Ambito
documentation

Direzione di ricerca

The typos sit in natspec comments in src/contracts/tokens/nf-token-enumerable.sol, nf-token-metadata.sol, utils/supports-interface.sol, ownership/ownable.sol and mocks/nf-token-metadata-enumerable-mock.sol. Run the grep command from the issue to confirm the locations, then check PR #302, which is already open against this issue, before starting. Done means every listed misspelling is corrected in comments only, with no code changes, and the grep returns nothing.

Scritto dal modello di indicizzazione a partire dal testo della issue.

Descrizione

Found while reviewing src/contracts under the bug bounty program (latest release, @nibbstack/erc721 2.6.1). Typos are explicitly listed in the program scope.

Remaining misspellings in natspec/comments at tag 2.6.1:

File Line Text Should be
src/contracts/tokens/nf-token-enumerable.sol 151 "Address from wich we want to remove the NFT" which
src/contracts/tokens/nf-token-enumerable.sol 181 "Address to wich we want to add the NFT" which
src/contracts/tokens/nf-token-metadata.sol 83 "an internal function that can be overriden" overridden
src/contracts/utils/supports-interface.sol 14 "Mapping of supported intefraces" interfaces
src/contracts/utils/supports-interface.sol 27 "which interfaces are suported by this contract" supported
src/contracts/ownership/ownable.sol 5 "authorization control whitch" which
src/contracts/mocks/nf-token-metadata-enumerable-mock.sol 107, 123 "from wich we want to remove" / "Address to wich we want to add" which

Reproduce with:

grep -rniE '\bwich\b|whitch|intefraces|suported|overriden' src/contracts

Suggested fix: a single pass with a spell checker over natspec blocks (ispell/aspell), as done previously in #252 for other files.

Tested against tag 2.6.1 (commit 198dec9).

Lingua principale
Solidity
Stelle
1k
Fork
391
Metriche di merge delle PR
Nessuna PR unita negli ultimi 30g

Preparare l'ambiente

Come iniziare

  1. Leggi tutta la issue e poi la guida ai contributi del progetto.
  2. Commenta sulla issue per dire che te ne occupi tu — evita che due persone facciano lo stesso lavoro.
  3. Fai un fork del repository e lavora su un branch.
  4. Apri una pull request che faccia riferimento al numero della issue.

Altre issue di fulldecent/erc721

Tutte le issue di fulldecent/erc721

Issue simili

Altre issue su Documentation

Ricevi le nuove issue nella tua casella

Un breve riepilogo di issue GitHub adatte ai principianti.