[nodebb-plugin-poll] Poll plugin

[Mega]

@chas Probably it's because of that line
https://github.com/Schamper/nodebb-plugin-poll/blob/master/lib/backend.js#L40
a callback should be passed there, but there is no.

[baris]

Latest master won't crash if callback isn't supplied.

dont crash if callback isnt supplied to list methods · NodeBB/NodeBB@749ce5f

Node.js based forum software built for the modern web - dont crash if callback isnt supplied to list methods · NodeBB/NodeBB@749ce5f

GitHub (github.com)

[Schamper]

I swear I'll update this soon Just have to finish a large report in the next 1.5 weeks and I'll have spare time for plugin development

[JonDoe12]

Any update on this @Schamper ?

[Bobber]

@Schamper Please add Russian Language Support during Poll creation UI

[Gordon Gu]

@psychobunny I tried to localize the poll plugin, but the translation doesn't happen (nodebb v0.6.x). I saw the client side js use window.templates.parse(pathToTpl, data, callback ) to process the tpl and translate string such as "[[poll.votes]]" not being replaced. Any suggestion? I guess tpl translation only happens at server side code?

[yariplus]

I got the translator working by using a ':' to split the translation filename and variable, e.g.:

"[[poll:votes]]"

and my languages/ru/poll.json has:

{
 "votes": "голосов"
}

[Gordon Gu]

@yariplus what the string to put in the tpl? I tried following:

<a class="poll-view-result-votecount" href="#">{options.votecount} [[poll:votes]]</a>

but translator didn't work:(

[yariplus]

I have

<a class="poll-view-result-votecount" href="#"> "[[poll:votes]]"</a>

(without the quotes)

[yariplus]

err, it keeps parsing out the text

This is what i'm using
https://gist.github.com/yariplus/8fbed0308828c980e117

[Gordon Gu]

@yariplus said:

gist:8fbed0308828c980e117

GitHub Gist: instantly share code, notes, and snippets.

Gist (gist.github.com)

no luck:(

my poll.json

The result

[yariplus]

@Gordon-Gu said:

@yariplus said:

gist:8fbed0308828c980e117

GitHub Gist: instantly share code, notes, and snippets.

Gist (gist.github.com)

no luck:(

my poll.json

The result

Ah okay, I should have posted my whole code. Looks like the client doesn't load the language files automatically, you have to you have to call load() first in the poll init and translate() on the template. If you can wait a bit I'm going to make a PR with everything setup.

edit: hmm, actually it does load the language automatically when you call translate(), maybe someone who knows more about the translator knows why it's not translating that template. All the admin templates work fine for me cause their parsed on the server.

edit 2: Okay! I got it working. PR coming...

[yariplus]

GitHub - yariplus/nodebb-plugin-poll at translator

Poll plugin for NodeBB. Contribute to yariplus/nodebb-plugin-poll development by creating an account on GitHub.

GitHub (github.com)

Comes with Russian translations.

[Leonardo Galani]

This works with 0.7.1?

[Kowlin]

@Leonardo-Galani Just gave this a shot on 0.7.3 and it appears to be broken on my end.

Edit: It appears to work but it doesn't display the poll on the topic.

[ogerly]

with me is the following problem.
when you create is the poll not load properly with the.
the theme appears is but empty.
the code is but since.

[Kowlin]

@ogerly Yup I got basically the same.

[phil]

@Kowlin There is a pull request (not authored by me) which fixes the problem. The Javascript problem is easy to fix. However, that in and of itself does not wholly fix the problem. The second part is that the Persona theme is laid out a bit different in HTML and the poll client javascript expects to insert the poll into a certain HTML element. Altering the poll code to work correctly for Persona means breaking all other themes. I have my own fixes for these issues but haven't pushed anything because of the theme compatibility problem.

How I fixed the JS problem:

index 5eb6a09..7b78147 100644
--- a/public/js/poll/main.js
+++ b/public/js/poll/main.js
@@ -1,7 +1,7 @@
 (function() {
        window.Poll = {
                load: function(data) {
-                       Poll.sockets.emit.load(data.pollid, function(err, poll)
+                       Poll.sockets.emit.load(data, function(err, poll) {
                                if (!err) {
                                        Poll.view.init(poll, function(pollView)
                                                if (parseInt(poll.info.deleted,
diff --git a/templates/poll/notify.tpl b/templates/poll/notify.tpl
index a4c5354..fc693c2 100644
--- a/templates/poll/notify.tpl
+++ b/templates/poll/notify.tpl
@@ -1 +1 @@
-<script>Poll.load({ pollid: {pollid} })</script>
\ No newline at end of file
+<script>Poll.load({pollid})</script>

And how I fixed the Persona theme problem:

diff --git a/public/js/poll/view.js b/public/js/poll/view.js
index 1d586ad..900f3e4 100644
--- a/public/js/poll/view.js
+++ b/public/js/poll/view.js
@@ -63,7 +63,7 @@
                },
                insertPoll: function(poll, callback) {
                        View.parsePoll(poll, function(html) {
-                               $('#post-container .post-row[data-index="0"] .po
+                               $('.posts .clearfix[data-index="0"] .content').p
                                callback();
                        });
                },

[ogerly]

@phil said:

•                           $('.posts .clearfix[data-index="0"] .content').p
  

missing since what? p = poll ??

[Tanguy Bodin-Hullin]

@Schamper This plugin would be very usefull in fact. I'm pretty sure that some people are moving from NodeBB to Discourse because of this problem. About me I spent some time installing nodeBB, which is fabulous, but the BIG problem is that I didn't find any working poll system on NodeBB until now, and I'm thinking about migrating to Discourse.. Sad, don't you think ?