From ecaa01d3b28a3bbab696acd88c5957754fd69960 Mon Sep 17 00:00:00 2001 From: Doug Kearns Date: Tue, 15 Oct 2013 14:01:12 +1100 Subject: [PATCH] Don't run the YouTube URL shortner on non-video pages. --- common/modules/buffer.jsm | 9 +++++---- 1 file changed, 5 insertions(+), 4 deletions(-) diff --git a/common/modules/buffer.jsm b/common/modules/buffer.jsm index 1b8695dd..f4092a35 100644 --- a/common/modules/buffer.jsm +++ b/common/modules/buffer.jsm @@ -2708,10 +2708,11 @@ Buffer.addPageInfoSection("s", "Security", function (verbose) { }); // internal navigation doesn't currently update link[rel='shortlink'] -Buffer.addURIShortener("youtube.com", (uri, doc) => - util.newURI("http://youtu.be/" + - array.toObject(uri.query.split("&") - .map(p => p.split("="))).v)); +Buffer.addURIShortener("youtube.com", (uri, doc) => { + let video = array.toObject(uri.query.split("&") + .map(p => p.split("="))).v; + return video ? util.newURI("http://youtu.be/" + video) : null; +}); // catch(e){ if (!e.stack) e = Error(e); dump(e.fileName+":"+e.lineNumber+": "+e+"\n" + e.stack); }