From a92dc673fbc741d25d91fdc6cc481012fc4b33f5 Mon Sep 17 00:00:00 2001 From: =?utf8?q?Benoi=CC=82t=20Pin?= Date: Mon, 7 Apr 2014 09:54:55 +0200 Subject: [PATCH 1/1] =?utf8?q?Bugfix=C2=A0:=20quand=20on=20a=20une=20pr?= =?utf8?q?=C3=A9sentation=20sans=20breadcrumbs,=20il=20ne=20faut=20pas=20q?= =?utf8?q?ue=20=C3=A7a=20plante.?= MIME-Version: 1.0 Content-Type: text/plain; charset=utf8 Content-Transfer-Encoding: 8bit --- skins/photo_film_viewer.js | 29 +++++++++++++++++++++-------- 1 file changed, 21 insertions(+), 8 deletions(-) diff --git a/skins/photo_film_viewer.js b/skins/photo_film_viewer.js index 70f8b99..bda6428 100644 --- a/skins/photo_film_viewer.js +++ b/skins/photo_film_viewer.js @@ -1,5 +1,5 @@ /* -* © 2008 Benoît Pin – Centre de recherche en informatique – École des mines de Paris +* © 2008-2014 Benoît Pin – Centre de recherche en informatique – MINES ParisTech * http://plinn.org * Licence Creative Commons http://creativecommons.org/licenses/by-nc/2.0/ * @@ -36,10 +36,16 @@ FilmSlider = function(filmBar, slider, ctxInfos, image, toolbar, breadcrumbs) { this.buttons = new Array(); this.toolbar = toolbar; - var bcElements = breadcrumbs.getElementsByTagName('a'); - this.lastBCElement = bcElements[bcElements.length-1]; - var imgSrcParts = image.src.split('/'); - this.lastBCElement.innerHTML = imgSrcParts[imgSrcParts.length-2]; + if (breadcrumbs) { + var bcElements = breadcrumbs.getElementsByTagName('a'); + this.lastBCElement = bcElements[bcElements.length-1]; + var imgSrcParts = image.src.split('/'); + this.lastBCElement.innerHTML = imgSrcParts[imgSrcParts.length-2]; + this.hasBreadcrumbs = true; + } + else { + this.hasBreadcrumbs = false; + } var buttons = toolbar.getElementsByTagName('img'); var b, name; @@ -589,7 +595,7 @@ FilmSlider.prototype.populateViewer = function(req) { switch (element.nodeName) { case 'fragment' : var dest = document.getElementById(element.getAttribute('id')); - dest.innerHTML = element.firstChild.nodeValue; + if (dest) { dest.innerHTML = element.firstChild.nodeValue; } break; case 'imageattributes' : var link = this.buttons['back_to_portfolio'].parentNode; @@ -601,8 +607,8 @@ FilmSlider.prototype.populateViewer = function(req) { else if(buyable == 'False') link.className = 'hidden'; this.image.alt = element.getAttribute('alt'); - this.lastBCElement.href = element.getAttribute('lastBcUrl'); - this.lastBCElement.innerHTML = element.getAttribute('img_id'); + this.updateBreadcrumbs(element.getAttribute('lastBcUrl'), + element.getAttribute('img_id')); break; } } @@ -620,6 +626,13 @@ FilmSlider.prototype.refreshImage = function() { this.image.parentNode.className = ''; }; +FilmSlider.prototype.updateBreadcrumbs = function(url, title) { + if (this.hasBreadcrumbs) { + this.lastBCElement.href = element.getAttribute('lastBcUrl'); + this.lastBCElement.innerHTML = element.getAttribute('img_id'); + } +}; + FilmSlider.prototype.startSlideShow = function() { this.slideShowSlide = this.pendingSlideShowSlide = this.selectedSlide; return this.slideShowSlide.href; -- 2.20.1