Editing a topic to add a Poll crashes Discourse!


(sin) #1

So it goes like this:

  1. User A: Creates an ordinary topic.

  2. Admin: Comes in to edit the topic by adding a Poll object to it.

  3. Discourse: Freezes for ever, immediately after adding the poll.

  4. Chrome: Eats all the memory there is to eat before suicide.

  5. End result: image


EDIT: by an admin adding a poll :bar_chart:

  • :banana:
  • :apple:
  • :lemon:

0 voters


(Régis Hanol) #2

Works fine here…

Do you know what poll the admin tried to add?
Do you know if there are any non-official plugins installed?


#3

I cannot replicate it in my test environment.

Is there anything in your logs?


(sin) #4

@mikechristopher @zogstrip

Found the issue it’s certainly a bug or limitation.

So the Poll plugin crashes without any warning and all Post changes are lost, if you try to add any float numbers to the numbers poll.

Like:

 [poll type=number min=1 max=20 step=0.5 public=true]
[/poll]

It only crashes if step is float number! even in this forum too, can be replicated.


(Régis Hanol) #5

Let’s try it

[poll type=number min=1 max=20 step=0.5 public=true]
[/poll]
  • 1
  • 2
  • 3
  • 4
  • 5
  • 6
  • 7
  • 8
  • 9
  • 10
  • 11
  • 12
  • 13
  • 14
  • 15
  • 16
  • 17
  • 18
  • 19
  • 20

0 voters

@Yassine_Yousfi seems to be working here just fine…


(sin) #6

Awkward!

It used to break! could it be breaking only on topics (first post) not in all posts!

Also noticed that in this discourse, it shows minimum step being 1, on mine not. and it broke for me when i was writing the topic as well!


(Joshua Rosenfeld) #7

Doesn’t crash on topics either. See Testing Polls with float steps - Demo (note that try resets daily so it won’t be there forever.


(sin) #8

Somehow my discourse doesn’t auto update anymore, not even from the admin side! but it had an SSL issue and had to rebuild the app from ssh again. and now i got latest update!
And the error doesn’t seem to occur anymore :slight_smile:

Thank you for your prompt replies guys!

Best


(Jeff Atwood) #9