git ssb - Depject expects modules that 'give' to return undefined in order for other modules to be searched for. 'False' will false circuit that search.

Gordon Martin committed on 10/15/2017, 10:59:54 PM
Parent: ed9add9c301f7f06b62492824e07772071cc5719

plugs/message/html/render/about.js	changed
plugs/message/html/render/attending.js	changed
plugs/message/html/render/channel.js	changed
plugs/message/html/render/following.js	changed
plugs/message/html/render/gathering.js	changed

plugs/message/html/render/about.jsView
		@@ -24,11 +24,9 @@
24	24
25	25	exports.create = function (api) {
26	26	const i18n = api.intl.sync.i18n
27	27	return nest('message.html', {
28		- canRender: function (msg) {
29		- return isRenderable(msg);
30		- },
	28	+ canRender: isRenderable,
31	29	render: function (msg, opts) {
32	30	if (!isRenderable(msg)) return;
33	31
34	32	var c = msg.value.content
		@@ -88,9 +86,9 @@
88	86	}
89	87	})
90	88
91	89	function isRenderable(msg) {
92		- if (msg.value.content.type !== 'about') return false
93		- else if (!ref.isFeed(msg.value.content.about)) return false
	90	+ if (msg.value.content.type !== 'about') return undefined
	91	+ else if (!ref.isFeed(msg.value.content.about)) return undefined
94	92	return true
95	93	}
96	94	}

plugs/message/html/render/attending.jsView
		@@ -17,11 +17,9 @@
17	17	})
18	18
19	19	exports.create = function(api) {
20	20	return nest('message.html', {
21		- canRender: function(msg) {
22		- return isRenderable(msg)
23		- },
	21	+ canRender: isRenderable,
24	22	render: function about(msg, opts) {
25	23	if (!isRenderable(msg)) return
26	24
27	25	var action = msg.value.content.attendee.remove ? `can't attend` : 'is attending'
		@@ -41,11 +39,11 @@
41	39	}
42	40	})
43	41
44	42	function isRenderable(msg) {
45		- if (msg.value.content.type !== 'about') return false
46		- else if (!ref.isMsg(msg.value.content.about)) return false
47		- else if (!msg.value.content.attendee) return false
48		- else if (msg.value.content.attendee.link !== msg.value.author) return false
	43	+ if (msg.value.content.type !== 'about') return undefined
	44	+ else if (!ref.isMsg(msg.value.content.about)) return undefined
	45	+ else if (!msg.value.content.attendee) return undefined
	46	+ else if (msg.value.content.attendee.link !== msg.value.author) return undefined
49	47	else return true
50	48	}
51	49	}

plugs/message/html/render/channel.jsView
		@@ -17,11 +17,9 @@
17	17
18	18	exports.create = function(api) {
19	19	const i18n = api.intl.sync.i18n
20	20	return nest('message.html', {
21		- canRender: function (msg) {
22		- return isRenderable(msg);
23		- },
	21	+ canRender: isRenderable,
24	22	render: function (msg, opts) {
25	23	if (!isRenderable(msg)) return
26	24	var element = api.message.html.layout(msg, extend({
27	25	miniContent: messageContent(msg),
		@@ -46,6 +44,6 @@
46	44	}
47	45	}
48	46
49	47	function isRenderable(msg) {
50		- return msg.value.content.type === 'channel'
	48	+ return msg.value.content.type === 'channel' ? true : undefined
51	49	}

plugs/message/html/render/following.jsView
		@@ -19,12 +19,10 @@
19	19
20	20	exports.create = function(api) {
21	21	const i18n = api.intl.sync.i18n
22	22	return nest('message.html', {
23		- canRender: function(msg) {
24		- return isRenderable(msg);
25		- },
26		- render: function (msg, opts) {
	23	+ canRender: isRenderable,
	24	+ render: function(msg, opts) {
27	25	if (!isRenderable(msg)) return
28	26
29	27	var element = api.message.html.layout(msg, extend({
30	28	miniContent: messageContent(msg),
		@@ -36,9 +34,9 @@
36	34	})
37	35	}
38	36	})
39	37
40		- function messageContent (msg) {
	38	+ function messageContent(msg) {
41	39	var following = msg.value.content.following
42	40	var blocking = msg.value.content.blocking
43	41
44	42	if (typeof blocking === 'boolean') {
		@@ -53,12 +51,12 @@
53	51	]
54	52	}
55	53	}
56	54
57		-function isRenderable(msg) {
58		- if (msg.value.content.type !== 'contact') return
59		- else if (!ref.isFeed(msg.value.content.contact)) return
60		- else if (typeof msg.value.content.following !== 'boolean' && typeof msg.value.content.blocking !== 'boolean') return
61		- return true;
62		-}
	55	+ function isRenderable(msg) {
	56	+ if (msg.value.content.type !== 'contact') return undefined
	57	+ else if (!ref.isFeed(msg.value.content.contact)) return undefined
	58	+ else if (typeof msg.value.content.following !== 'boolean' && typeof msg.value.content.blocking !== 'boolean') return undefined
	59	+ return true;
	60	+ }
63	61
64	62	}

plugs/message/html/render/gathering.jsView
		@@ -33,11 +33,9 @@
33	33	exports.create = function(api) {
34	34	var following = null
35	35
36	36	return nest('message.html', {
37		- canRender: function(msg) {
38		- return isRenderable(msg)
39		- },
	37	+ canRender: isRenderable,
40	38	render: function (msg, opts) {
41	39	if (!isRenderable(msg)) return
42	40
43	41	var yourId = api.keys.sync.id()
		@@ -157,9 +155,9 @@
157	155	}
158	156	}
159	157
160	158	function isRenderable(msg) {
161		- return (msg.value.content.type === 'gathering')
	159	+ return (msg.value.content.type === 'gathering') ? true : undefined
162	160	}
163	161
164	162	function formatTime(time) {
165	163	if (time && time.epoch) {

Matt McKegg / patchwork