Unified Diff: lib/synchronizer.js
Issue 29340834:
Issue 3991 - Fixed logic setting homepage URL automatically (Closed)
Patch Set:
Created April 26, 2016, 4:08 p.m.
Use n/p to move between diff chunks;
N/P to move between comments.
« no previous file with comments
|
« no previous file
|
no next file »
|
no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Hide Comments ('s')
Expand Comments ('e') | Collapse Comments ('c') | Hide Comments ('s')
| Index: lib/synchronizer.js |
| =================================================================== |
| --- a/lib/synchronizer.js |
| +++ b/lib/synchronizer.js |
| @@ -211,17 +211,17 @@ var Synchronizer = exports.Synchronizer |
| { |
| url = new URL(params.homepage); |
| } |
| catch (e) |
| { |
| url = null; |
| } |
| - if (url && (url.protocol == "http" || url.protocol == "https")) |
| + if (url && (url.protocol == "http:" || url.protocol == "https:")) |
| subscription.homepage = url.href; |
| } |
| if (params.title) |
| { |
| subscription.title = params.title; |
| subscription.fixedTitle = true; |
| } |
« no previous file with comments
|
« no previous file
|
no next file »
|
no next file with comments »
