Jshint: JSHint marque "javascript:void(0);" en tant qu'URL JS

Créé le 23 juin 2014  ·  8Commentaires  ·  Source: jshint/jshint

Si j'affecte le href d'un élément à javascript:void(0); (par opposition à # ), comme ceci :

$('#foobar').attr("href", "javascript:void(0);");

JSHint récupère la construction javascript:void(0); tant qu'URL de script :

PS C:\Users\hal9000\Development\strugee.github.com> grunt lint
Running "jshint:all" (jshint) task

   js\main.js
     30 |    $('#lightbulb-icon').attr("href", "javascript:void(0);");
                                                                    ^ Script URL.

>> 1 error in 4 files

alors qu'en fait il ne fait rien et est sémantiquement valide. Il devrait y avoir une exception pour ces types d'URL de script.

picture strugee
👍1

Commentaire le plus utile

Mettez simplement cette ligne en haut de votre fichier pour désactiver l'avertissement :

/*jshint scripturl:true*/

Mais le meilleur moyen est d'utiliser "#" dans le href et de retourner "false" pour tous les événements javascript :

<a href="#" onclick="doSomething();return false;">my link</a>
picture kcampion le 9 sept. 2014
👍5

Tous les 8 commentaires

Peu importe .

Peut-être devrait-il s'agir d'un avertissement spécialisé ? Je ne peux pas être le seul à avoir été dérouté par cela.

strugee picture strugee le 23 juin 2014

Vous n'êtes pas.

weeger picture weeger le 13 août 2014

Mettez simplement cette ligne en haut de votre fichier pour désactiver l'avertissement :

/*jshint scripturl:true*/

Mais le meilleur moyen est d'utiliser "#" dans le href et de retourner "false" pour tous les événements javascript :

<a href="#" onclick="doSomething();return false;">my link</a>
kcampion picture kcampion le 9 sept. 2014
👍5

Non, le meilleur moyen est de garder votre JavaScript hors de votre balisage et attributs :P

rwaldron picture rwaldron le 9 sept. 2014
😕2 👍1

@rwaldron : rectification, en effet

$('#foobar').attr("href", "#").click(function() {
  // do it
});
kcampion picture kcampion le 9 sept. 2014

Passe jslint :
$('#foobar').attr('href','javascript'+String.fromCharCode(58)+'void(0);');

MikeGodin picture MikeGodin le 4 août 2016

@MikeGodin ressemble à un bogue distinct.

strugee picture strugee le 4 août 2016

@kcampion mettre # et retourner false n'est pas toujours souhaitable
dans mon cas, je ne veux pas que le hash apparaisse

englishextra picture englishextra le 31 oct. 2016
👍2
Cette page vous a été utile?
0 / 5 - 0 notes

Questions connexes

ghost picture ghost  ·  5Commentaires
Daniel-Hug picture Daniel-Hug  ·  3Commentaires
NemoStein picture NemoStein  ·  7Commentaires
jugglinmike picture jugglinmike  ·  6Commentaires
TheSavior picture TheSavior  ·  3Commentaires