[nodebb-plugin-ns-spoiler] NS Spoiler

darktrge

@KitsuneSolar said in [nodebb-plugin-ns-spoiler] NS Spoiler:
confirming the bug on:
nodebb v1.4.2
mongodb v3.2.1
node v4.7.1

I'm getting this error in browser console:
Error has occurred, error: Something went wrong. Spoiler content can not be found.

Nicolas

@KitsuneSolar your investigation is very helpful.

Could you disable all 3-rd party plugins so only Native Plugins and Spoiler Plugin will be active? Am I right that you don't see an issue anymore?

My theory: some plugin in the chain rewrites a payload. It is not reproducible for me because I'm using different plugins, mostly which I have developed.

KitsuneSolar

@Nicolas said in [nodebb-plugin-ns-spoiler] NS Spoiler:

Could you disable all 3-rd party plugins so only Native Plugins and Spoiler Plugin will be active?

3-rd party plugins

I have only nodebb-plugin-poll & spam-be-gone.

Nicolas

Version 3.1.0

A new version is available. Summary: Looks like investigation with Spoiler Content issue is resolved. Thanks to https://github.com/wsyscz2 user. The issue was a post cache.

Changes:

Change internal logic to use a plugins hooks
Fix issue with spoiler content

MJ

@Nicolas 3.1.0 works thanks

KitsuneSolar

Images not showing.

TypeError: e.$element is null[Learn More]  nodebb.min.js:7:25286
	t.prototype.destroy/< https://domain.com/nodebb.min.js:7:25286
	a https://domain.com/nodebb.min.js:7:22395
	Ce/r https://domain.com/nodebb.min.js:2:9962
	e.event.special.bsTransitionEnd.handle https://domain.com/nodebb.min.js:6:29279
	v.event.dispatch https://domain.com/nodebb.min.js:2:12448
	v.event.add/g.handle https://domain.com/nodebb.min.js:2:10404
	.trigger https://domain.com/nodebb.min.js:3:9401
	.trigger/< https://domain.com/nodebb.min.js:3:9937
	.each https://domain.com/nodebb.min.js:1:2988
	v.prototype.each https://domain.com/nodebb.min.js:1:1077
	.trigger https://domain.com/nodebb.min.js:3:9916
	e.fn.emulateTransitionEnd/r https://domain.com/nodebb.min.js:6:28985
TypeError: r.$element is null[Learn More]  nodebb.min.js:7:20985
	t.prototype.show/g https://domain.com/nodebb.min.js:7:20985
	Ce/r https://domain.com/nodebb.min.js:2:9962
	e.event.special.bsTransitionEnd.handle https://domain.com/nodebb.min.js:6:29279
	v.event.dispatch https://domain.com/nodebb.min.js:2:12448
	v.event.add/g.handle https://domain.com/nodebb.min.js:2:10404
	.trigger https://domain.com/nodebb.min.js:3:9401
	.trigger/< https://domain.com/nodebb.min.js:3:9937
	.each https://domain.com/nodebb.min.js:1:2988
	v.prototype.each https://domain.com/nodebb.min.js:1:1077
	.trigger https://domain.com/nodebb.min.js:3:9916
	e.fn.emulateTransitionEnd/r https://domain.com/nodebb.min.js:6:28985
TypeError: e.$element is null[Learn More]  nodebb.min.js:7:25286
	t.prototype.destroy/< https://domain.com/nodebb.min.js:7:25286
	a https://domain.com/nodebb.min.js:7:22395
	Ce/r https://domain.com/nodebb.min.js:2:9962
	e.event.special.bsTransitionEnd.handle https://domain.com/nodebb.min.js:6:29279
	v.event.dispatch https://domain.com/nodebb.min.js:2:12448
	v.event.add/g.handle https://domain.com/nodebb.min.js:2:10404
	.trigger https://domain.com/nodebb.min.js:3:9401
	.trigger/< https://domain.com/nodebb.min.js:3:9937
	.each https://domain.com/nodebb.min.js:1:2988
	v.prototype.each https://domain.com/nodebb.min.js:1:1077
	.trigger https://domain.com/nodebb.min.js:3:9916
	e.fn.emulateTransitionEnd/r https://domain.com/nodebb.min.js:6:28985

exodo

@KitsuneSolar said in [nodebb-plugin-ns-spoiler] NS Spoiler:

Images not showing.

TypeError: e.$element is null[Learn More]  nodebb.min.js:7:25286
	t.prototype.destroy/< https://domain.com/nodebb.min.js:7:25286
	a https://domain.com/nodebb.min.js:7:22395
	Ce/r https://domain.com/nodebb.min.js:2:9962
	e.event.special.bsTransitionEnd.handle https://domain.com/nodebb.min.js:6:29279
	v.event.dispatch https://domain.com/nodebb.min.js:2:12448
	v.event.add/g.handle https://domain.com/nodebb.min.js:2:10404
	.trigger https://domain.com/nodebb.min.js:3:9401
	.trigger/< https://domain.com/nodebb.min.js:3:9937
	.each https://domain.com/nodebb.min.js:1:2988
	v.prototype.each https://domain.com/nodebb.min.js:1:1077
	.trigger https://domain.com/nodebb.min.js:3:9916
	e.fn.emulateTransitionEnd/r https://domain.com/nodebb.min.js:6:28985
TypeError: r.$element is null[Learn More]  nodebb.min.js:7:20985
	t.prototype.show/g https://domain.com/nodebb.min.js:7:20985
	Ce/r https://domain.com/nodebb.min.js:2:9962
	e.event.special.bsTransitionEnd.handle https://domain.com/nodebb.min.js:6:29279
	v.event.dispatch https://domain.com/nodebb.min.js:2:12448
	v.event.add/g.handle https://domain.com/nodebb.min.js:2:10404
	.trigger https://domain.com/nodebb.min.js:3:9401
	.trigger/< https://domain.com/nodebb.min.js:3:9937
	.each https://domain.com/nodebb.min.js:1:2988
	v.prototype.each https://domain.com/nodebb.min.js:1:1077
	.trigger https://domain.com/nodebb.min.js:3:9916
	e.fn.emulateTransitionEnd/r https://domain.com/nodebb.min.js:6:28985
TypeError: e.$element is null[Learn More]  nodebb.min.js:7:25286
	t.prototype.destroy/< https://domain.com/nodebb.min.js:7:25286
	a https://domain.com/nodebb.min.js:7:22395
	Ce/r https://domain.com/nodebb.min.js:2:9962
	e.event.special.bsTransitionEnd.handle https://domain.com/nodebb.min.js:6:29279
	v.event.dispatch https://domain.com/nodebb.min.js:2:12448
	v.event.add/g.handle https://domain.com/nodebb.min.js:2:10404
	.trigger https://domain.com/nodebb.min.js:3:9401
	.trigger/< https://domain.com/nodebb.min.js:3:9937
	.each https://domain.com/nodebb.min.js:1:2988
	v.prototype.each https://domain.com/nodebb.min.js:1:1077
	.trigger https://domain.com/nodebb.min.js:3:9916
	e.fn.emulateTransitionEnd/r https://domain.com/nodebb.min.js:6:28985

Same issue. Not showing images

Master

@KitsuneSolar Same

Nicolas

Version 4.0.0

A new version is available. Summary: add compatibility with changes in NodeBB v1.1.0 and support for the images. Thanks to https://github.com/segura2010 user. The issue was a lazy loading for the images which was introduced in NodeBB v1.1.0

Changes:

Add compatibility with NodeBB v1.1.0
Change content rendering to accommodate image lazy loading

darktrge

On nodebb 1.4.5, spoiler v4.1
I keep getting error:

Uncaught TypeError: e.unloadImages is not a function
at p.<anonymous> (nodebb.min.js?v=8vda2olngi6:12)
at p.onack (nodebb.min.js?v=8vda2olngi6:6)
at p.onpacket (nodebb.min.js?v=8vda2olngi6:6)
at h.<anonymous> (nodebb.min.js?v=8vda2olngi6:6)
at h.i.emit (nodebb.min.js?v=8vda2olngi6:5)
at h.ondecoded (nodebb.min.js?v=8vda2olngi6:4)
at f.<anonymous> (nodebb.min.js?v=8vda2olngi6:6)
at f.n.emit (nodebb.min.js?v=8vda2olngi6:4)
at f.add (nodebb.min.js?v=8vda2olngi6:4)
at h.ondata (nodebb.min.js?v=8vda2olngi6:4)

Nicolas

Did you see such error before? Before 1.4.5?

Nicolas

Hm, I see now.

In 1.4.5 function was removed, so 1.4.5 has a breaking change for spoiler plugin.
Technically plugin is not compatible with 1.4.5 and higher.

Sam Nguyen

Hi Mr. @Nicolas ,

Do you try to fix it? I tried to install last version to my forum, It's still error when spoiler image

Uncaught TypeError: e.unloadImages is not a function
    at i.<anonymous> (spoiler.js:57)
    at i.onack (socket.io.js:2)
    at i.onpacket (socket.io.js:2)
    at i.<anonymous> (socket.io.js:2)
    at i.emit (socket.io.js:1)
    at i.ondecoded (socket.io.js:1)
    at a.<anonymous> (socket.io.js:2)
    at a.i.emit (socket.io.js:1)
    at a.add (socket.io.js:1)
    at i.ondata (socket.io.js:1)

I'm using NodeBB version 1.5.2

Many thanks.

P/s: Spoiler text is working normally, issue happen when spoiler a image

ToeiRei

Sorry for necroposting, but it looks like the spoiler plugin got a problem with lists at the current versionof nodebb.

:::

- some
- items
- here

:::

Master

Can someone fix issue with image in spoiler please?

Nicolas

A new version of the package nodebb-plugin-ns-spoiler (5.0.1) is available.

Plugin updated to work with the latest changes (NodeBB 1.8.x). Also, the plugin should work better with a more complex post structure (multi-list, etc.)

@Sam-Nguyen images should work well in v5.

MJ

@nicolas said in [nodebb-plugin-ns-spoiler] NS Spoiler:

A new version of the package nodebb-plugin-ns-spoiler (5.0.1) is available.

Plugin updated to work with the latest changes (NodeBB 1.8.x). Also, the plugin should work better with a more complex post structure (multi-list, etc.)

@Sam-Nguyen images should work well in v5.

Thank you for this update, everything works fine at 1.9.0

faizanzahid

@nicolas thank you very much for making it compatible with the latest releases

Bulldozer

@nicolas said in [nodebb-plugin-ns-spoiler] NS Spoiler:

A new version of the package nodebb-plugin-ns-spoiler (5.0.1) is available.
@Sam-Nguyen images should work well in v5.

Hi @nicolas There is still 4.1.0 in my ACP. I installed NodeBB v1.9.3 a week ago.

0_1529892010787_Screenshot from 2018-06-25 11-59-47.png

Brophy

This post is deleted!