dpoll.xyz icon indicating copy to clipboard operation
dpoll.xyz copied to clipboard

Doubled tags on Steem root posts

Open espoem opened this issue 7 years ago • 2 comments

Actual behaviour

Dpoll tags are doubled at a Steem post of the poll.

image

image

Test post: https://steempeak.com/dpoll/@elear/which-perks-would-you-like-to-have-the-most-on-an-upcoming-crowdfunding-campaign-for-utopian

(tags are not doubled on steemit interface)

Expected behaviour

Each tag should be present at most once.

espoem avatar Feb 07 '19 19:02 espoem

This might be related to editing. Will investigate.

emre avatar Feb 07 '19 20:02 emre

I think so. In https://github.com/emre/dpoll.xyz/blob/d7d758f2908948b0ba4123f35036f3b2834a506c/dpoll/polls/views.py#L240 edit_poll method, code is not checking for the tags parameter to contain the settings.DEFAULT_TAGS and it is just concatenating them:

    if tags:
        tags = settings.DEFAULT_TAGS + tags
    else:
        tags = settings.DEFAULT_TAGS

kekkyojin avatar Mar 31 '19 20:03 kekkyojin