codebytere / codebytere/codebytere.github.io

localStorage usage is buggy

Ouverte
#16 1 commentaire 0 réactions 0 personnes assignées Voir sur GitHub
Langage dominant
JavaScript
Étoiles
546
Forks
105
Métriques de merge des PR
Aucune PR mergée en 30 j

Description

Cool site. Seems the usage of `localStorage` is buggy though. In particular, in this line:

https://github.com/codebytere/codebytere.github.io/blob/3d5854bea758dfe0266b6ffc4b77a0b03a0cb782/js/shell.js#L42

`localStorage.inHistory` will always evaluate to be `true`. That's because everything put into `localStorage` is coerced to a string:

> [The keys and the values are **always strings** (note that, as with objects, integer keys will be automatically converted to strings).](https://developer.mozilla.org/en-US/docs/Web/API/Window/localStorage)

So this line:

https://github.com/codebytere/codebytere.github.io/blob/3d5854bea758dfe0266b6ffc4b77a0b03a0cb782/js/shell.js#L12

is actually setting `inHistory` to the string `"false"`, and `Boolean("false") === true`.

Doesn't look like your usage of `localStorage` persists between page loads (everything resets) so there's no real need to use it.

Guide de contribution

Aucun guide de contribution indexé pour ce dépôt

Piste de recherche

Commencez par inspecter js/shell.js aux lignes 12 et 42, puis reproduisez le comportement de localStorage décrit dans l’issue, notamment la manière dont la chaîne "false" est évaluée. Confirmez le comportement prévu de l’historique lors des chargements successifs de la page et assurez-vous que la modification terminée ne traite plus la valeur false stockée comme true.

Rédigé par le modèle d'indexation à partir du texte de l'issue.

Évaluation

Stack technique
javascript
Domaine
frontend
Type d'issue
Bug
Difficulté
2/5
Temps estimé
1-3 heures
Activité
À l'abandon
Clarté
Plutôt claire
Accessibilité débutants
42/100

Recevez les nouvelles issues par e-mail

Un résumé court des issues GitHub adaptées aux débutants.