Contributor count on tag pages not reflecting number of contributors

[steviepubliclab]

I was looking for the number of posts and contributors to this tag (https://publiclab.org/tag/oil-and-gas) and saw that the number was listed here as Zero:

But there are actually 73 contributors and 38 posts. This can be found here https://publiclab.org/contributors/oil-and-gas:

[ebarry]

great catch!

[jywarren]

So, here are the queries used for the 0 posts by 0 contributors | 0 followers area:

plots2/app/views/tag/show.html.erb

Lines 57 to 61 in 8128a30

	<p class="contributor-info"><%= Tag.find_nodes_by_type(tag.name, 'note', false).count %> posts by
	<a href="/contributors/<%= params[:id] %>"><%= Tag.contributor_count(tag.name) %> contributors</a> |
	<a href="/contributors/<%= params[:id] %>"><%= Tag.follower_count(tag.name) %> followers</a>
	<a href="https://publiclab.org/wiki/contributors"><i class="fa fa-question-circle-o" aria-hidden="true"></i></a>
	</p>

Here's somewhere they seem to be counting correctly, in the lower right of the main image:

plots2/app/views/tag/show/_header.html.erb

Line 18 in 8128a30

<%= Tag.follower_count(params[:id]) %> following

Is it possible that tag.name is not correct?

Yes! This relates to the error on

plots2/app/views/tag/show.html.erb

Line 28 in 8128a30

<a rel="tooltip" title="<%= t('sidebar._post_button.share_your_work') %>" data-placement="bottom" href="/post?tags=<%= tag.name %>" class="btn btn-primary requireLogin">New post <i class="fa fa-plus fa-white"></i></a>

This has the same error! #6592

We should be able to solve both by replacing tag.name with params[:id] for these lines.

---

This has been marked as a good candidate for becoming a first-timers-only issue like these, meaning that it's simple, self-contained, and with some extra formatting, could be a great entry point for a new contributor. If you're familiar enough with this code, please consider reformatting or reposting it as a first-timers-only issue, and then ping @publiclab/reviewers to get it labelled. Or, if this is not your first time, try to solve it yourself!

[nicoleiocana]

I would also like to piggy-back off of @jywarren to mention that the entire show.html.erb code should be refactored to include the instance variable @title since the tag_controller.rb file has @title = params[:id]. Since we should be practicing DRY, it is redundant to have params[:id] in the show.html.erb file when it was passed along as the @title instance variable from the tag_controller.rb.

[jywarren]

Hi, ah, i see - so @title is used to set the HTML document title field, so I am a little hesitant about using it for this logic just as it's currently named for a different purpose -- it could cause some confusion. If we'd like to independently use a new instance variable like @TagName, that could work, but it may not save us much code. Thanks a lot!

…

On Mon, Nov 4, 2019 at 2:11 PM nicoleiocana ***@***.***> wrote: I would also like to piggy-back off of @jywarren <https://github.com/jywarren> to mention that the entire show.html.erb code should be refactored to include the instance variable @title <https://github.com/title> since the tag_controller.rb file has @title = params[:id]. Since we should be practicing DRY, it is redundant to have params[:id] in the show.html.erb file when it was passed along as the @title <https://github.com/title> instance variable from the tag_controller.rb. — You are receiving this because you were mentioned. Reply to this email directly, view it on GitHub <#6675?email_source=notifications&email_token=AAAF6J4T3LZU5SI3AYYSH6DQSBXUDA5CNFSM4JIWPCCKYY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOEDAL6JA#issuecomment-549502756>, or unsubscribe <https://github.com/notifications/unsubscribe-auth/AAAF6J5QCPZFUW55BW3F2RDQSBXUDANCNFSM4JIWPCCA> .

[nicoleiocana]

You are very welcome. I totally agree with you...... trust me. But since this isn't my code, I didn't want to edit out their usage of certain definitions; I was just using what they gave me. I'll submit this PR using @title, then, if someone wants to refactor the code to redefine @title to mean something else, I'll just let them have that responsibility. =)

EDIT: Due to my conversation with @jywarren in #6592, the PR for this issue will replace @title with params[:id] in locations where the code is referencing the params passed from the previous link, not where it refers to the page header title.