kylewm / redwind

Flask-based personal website #IndieWeb #OwnYourData
Other
45 stars 5 forks source link

Temporary fix for a strange tag cloud bug #30

Closed thedod closed 9 years ago

thedod commented 9 years ago

Tags appeared several times, with the count split among them E.g. lol(5) and lol(7), and if you click on lol, you'll see 12 items.

This patch aggregates them (not a heavy operation), but I wonder how this should be done properly.

kylewm commented 9 years ago

Oh bummer, this looks good as a stopgap, can you let look at the DB itself and see whether the two tags are the exact same "name" or if there is a discrepancy in capitalization, whitespace, etc?

On Wed, Dec 10, 2014, at 06:10 AM, The Dod wrote:

Tags appeared several times, with the count split among them E.g. lol(5) and lol(7), and if you click on lol, you'll see 12 items.

This patch aggregates them (not a heavy operation), but I wonder how this should be done properly. You can merge this Pull Request by running:

git pull https://github.com/thedod/redwind weird-tag-fix

Or you can view, comment on it, or merge it online at:

https://github.com/kylewm/redwind/pull/30

-- Commit Summary --

  • Temporary fix for a strange tag cloud bug

-- File Changes --

M redwind/views.py (7)

-- Patch Links --

https://github.com/kylewm/redwind/pull/30.patch https://github.com/kylewm/redwind/pull/30.diff


Reply to this email directly or view it on GitHub: https://github.com/kylewm/redwind/pull/30

thedod commented 9 years ago

See https://dubiousdod.org/indie/static/dump.sql.xz (the whole thing except for OAuth secrets and such :wink:)