Commit a4c97b2c28b8bb16232a873162d6d9992353cedc
Check link type before getting message
cel committed on 10/17/2016, 9:05:51 PMParent: 9018a0991550b3edcb6e308bf5957a8024df4487
Files changed
modules/message-link.js | changed |
modules/notifications.js | changed |
modules/message-link.js | ||
---|---|---|
@@ -1,5 +1,6 @@ | ||
1 | 1 … | var h = require('hyperscript') |
2 … | +var ref = require('ssb-ref') | |
2 | 3 … | |
3 | 4 … | var sbot_get = require('../plugs').first(exports.sbot_get = []) |
4 | 5 … | |
5 | 6 … | exports.message_link = function (id) { |
@@ -8,8 +9,9 @@ | ||
8 | 9 … | throw new Error('link must be to message id') |
9 | 10 … | |
10 | 11 … | var link = h('a', {href: '#'+id}, id.substring(0, 10)+'...') |
11 | 12 … | |
13 … | + if(ref.isMsg(id)) | |
12 | 14 … | sbot_get(id, function (err, value) { |
13 | 15 … | if(err) { |
14 | 16 … | if (err.name == 'NotFoundError') |
15 | 17 … | link.textContent += ' (missing)' |
modules/notifications.js | |||
---|---|---|---|
@@ -4,8 +4,9 @@ | |||
4 | 4 … | var Scroller = require('pull-scroll') | |
5 | 5 … | var paramap = require('pull-paramap') | |
6 | 6 … | var plugs = require('../plugs') | |
7 | 7 … | var cont = require('cont') | |
8 … | +var ref = require('ssb-ref') | ||
8 | 9 … | ||
9 | 10 … | var message_render = plugs.first(exports.message_render = []) | |
10 | 11 … | var sbot_log = plugs.first(exports.sbot_log = []) | |
11 | 12 … | var sbot_get = plugs.first(exports.sbot_get = []) | |
@@ -30,8 +31,9 @@ | |||
30 | 31 … | ||
31 | 32 … | function isOurMsg(id, cb) { | |
32 | 33 … | if (!id) return cb(null, false) | |
33 | 34 … | if (typeof id === 'object' && typeof id.link === 'string') id = id.link | |
35 … | + if (!ref.isMsg(id)) return cb(null, false) | ||
34 | 36 … | sbot_get(id, function (err, msg) { | |
35 | 37 … | if (err && err.name == 'NotFoundError') cb(null, false) | |
36 | 38 … | else if (err) cb(err) | |
37 | 39 … | else if (msg.content.type === 'issue' || msg.content.type === 'pull-request') |
Built with git-ssb-web