From: Benoît Pin Date: Mon, 7 Apr 2014 08:13:00 +0000 (+0200) Subject: jslint X-Git-Url: https://scm.cri.mines-paristech.fr/git/Portfolio.git/commitdiff_plain/de796b93caf39451cab93560b5978620a21db2f3?ds=sidebyside jslint --- diff --git a/skins/photo_film_viewer.js b/skins/photo_film_viewer.js index bda6428..cfd7df5 100644 --- a/skins/photo_film_viewer.js +++ b/skins/photo_film_viewer.js @@ -20,7 +20,7 @@ FilmSlider = function(filmBar, slider, ctxInfos, image, toolbar, breadcrumbs) { var thisSlider = this; this.filmBar = filmBar; var film = filmBar.firstChild; - if (film.nodeType == 3) + if (film.nodeType === 3) film = film.nextSibling; this.film = film; this.slider = slider; @@ -209,12 +209,12 @@ FilmSlider.prototype.loadSibling = function(previous) { var slide = null; if (previous) { slide = this.selectedSlide.parentNode.previousSibling; - if (slide && slide.nodeType==3) + if (slide && slide.nodeType===3) slide = slide.previousSibling; } else { slide = this.selectedSlide.parentNode.nextSibling; - if (slide && slide.nodeType==3) + if (slide && slide.nodeType===3) slide = slide.nextSibling; } @@ -303,7 +303,7 @@ FilmSlider.prototype.thumbnailClickHandler = function(evt) { if (target.tagName != 'A') return; else { - if (this.viewMode == 'full') { + if (this.viewMode === 'full') { this.mosaique.unload(); this.mosaique = null; this.viewMode = 'medium'; @@ -335,7 +335,7 @@ FilmSlider.prototype.thumbnailClickHandler = function(evt) { var toggleSelectionBtn = this.buttons['toggle_selection']; var toggleSelectionLink = toggleSelectionBtn.parentNode; - this.selectedSlideInSelection = (target.className=='selected'); + this.selectedSlideInSelection = (target.className==='selected'); if (this.selectedSlideInSelection) { toggleSelectionBtn.src = portal_url() + '/unselect_flag_btn.gif'; toggleSelectionBtn.alt = toggleSelectionLink.title = 'Retirer de la sélection'; @@ -379,7 +379,7 @@ FilmSlider.prototype.thumbnailClickHandler = function(evt) { break; case 4 : hideProgressImage(); - if (req.status == '200') + if (req.status === '200') thisFS.populateViewer(req); else //window.location.href = target.href; @@ -399,7 +399,7 @@ FilmSlider.prototype.thumbnailClickHandler = function(evt) { for (i in classes) { name = classes[i]; - if (name == 'displayed') + if (name === 'displayed') continue; else newClasses.push(name); @@ -418,7 +418,7 @@ FilmSlider.prototype.thumbnailClickHandler = function(evt) { FilmSlider.prototype.toolbarClickHandler = function(evt) { var target = getTargetedObject(evt); - if(target.tagName == 'IMG' && target.getAttribute('name')){ + if(target.tagName === 'IMG' && target.getAttribute('name')){ switch(target.getAttribute('name')) { case 'previous' : disableDefault(evt); @@ -440,7 +440,7 @@ FilmSlider.prototype.toolbarClickHandler = function(evt) { disableDefault(evt); disablePropagation(evt); target.parentNode.blur(); - if (this.viewMode == 'full') { + if (this.viewMode === 'full') { this.mosaique.unload(); this.mosaique = null; this.viewMode = 'medium'; @@ -516,7 +516,7 @@ FilmSlider.prototype.toolbarClickHandler = function(evt) { disableDefault(evt); disablePropagation(evt); target.blur(); - if (this.viewMode == 'full') { + if (this.viewMode === 'full') { this.mosaique.unload(); this.mosaique = null; this.viewMode = 'medium'; @@ -576,7 +576,7 @@ FilmSlider.prototype.keyDownHandler = function(evt) { FilmSlider.prototype.keyPressHandler = function(evt) { var target = getTargetedObject(evt); - if (target.tagName == 'INPUT' || target.tagName== 'TEXTAREA') + if (target.tagName === 'INPUT' || target.tagName=== 'TEXTAREA') return; var evt = getEventObject(evt); var charPress = String.fromCharCode((evt.keyCode) ? evt.keyCode : evt.which); @@ -602,9 +602,9 @@ FilmSlider.prototype.populateViewer = function(req) { link.href = element.getAttribute('backToContextUrl'); link = this.buttons['show_buyable'].parentNode; var buyable = element.getAttribute('buyable'); - if(buyable == 'True') + if(buyable === 'True') link.className = null; - else if(buyable == 'False') + else if(buyable === 'False') link.className = 'hidden'; this.image.alt = element.getAttribute('alt'); this.updateBreadcrumbs(element.getAttribute('lastBcUrl'), @@ -640,7 +640,7 @@ FilmSlider.prototype.startSlideShow = function() { FilmSlider.prototype.slideShowNext = function() { var nextSlide = this.slideShowSlide.parentNode.nextSibling; - if (nextSlide && nextSlide.nodeType==3) + if (nextSlide && nextSlide.nodeType===3) nextSlide = nextSlide.nextSibling; if (nextSlide) { @@ -651,7 +651,7 @@ FilmSlider.prototype.slideShowNext = function() { else { var row = this.slideShowSlide.parentNode.parentNode; var first = row.firstChild; - if (first.nodeType==3) + if (first.nodeType===3) first = first.nextSibling; this.pendingSlideShowSlide = first.getElementsByTagName('a')[0]; return this.pendingSlideShowSlide.href; @@ -660,7 +660,7 @@ FilmSlider.prototype.slideShowNext = function() { FilmSlider.prototype.slideShowPrevious = function() { var previousSlide = this.slideShowSlide.parentNode.previousSibling; - if (previousSlide && previousSlide.nodeType==3) + if (previousSlide && previousSlide.nodeType===3) previousSlide = previousSlide.previousSibling; if (previousSlide) { @@ -671,7 +671,7 @@ FilmSlider.prototype.slideShowPrevious = function() { else { var row = this.slideShowSlide.parentNode.parentNode; var last = row.lastChild; - if (last.nodeType==3) + if (last.nodeType===3) last = last.previousSibling; this.pendingSlideShowSlide = last.getElementsByTagName('a')[0]; return this.pendingSlideShowSlide.href;