Skip to:

Opened 9 years ago

Closed 9 years ago

#6207 closed defect (bug) (fixed)

BP user blogs count display in user nav not classed.

Reported by: hnla's profile hnla Owned by: johnjamesjacoby's profile johnjamesjacoby
Milestone: 2.3 Priority: low
Severity: minor Version:
Component: Blogs Keywords: has-patch early commit


The user navs display counts in spans on various nav tabs e.g notifications, these span elements are classed either as 'no-count' or 'count'.

The Sites tab link has never had these classes added to it's span count.

This patch corrects this for uniformity.

@lead-devs please check this as I had to cast bp_get_total_blog_count_for_user as an '(int)' as in replicating other examples of how we do this a strict comparison is used but that failed on my re-working originally I used a < 1 as a check but wanted to mirror how the others handled it but 0=== failed when it shouldn't have, tracking the function back wasn't clear what the value being passed was. As it's a clear count it seemed appropriate to cast it to an int and it's use doesn't move outside the scope it's used in here.

Attachments (1)

6207-01.patch (801 bytes) - added by hnla 9 years ago.

Download all attachments as: .zip

Change History (4)

9 years ago

#1 @johnjamesjacoby
9 years ago

  • Keywords early commit added
  • Milestone changed from 2.2.1 to 2.3

Good fix for 2.3. Let's get this in early.

This ticket was mentioned in Slack in #buddypress by jjj. View the logs.

9 years ago

#3 @johnjamesjacoby
9 years ago

  • Owner set to johnjamesjacoby
  • Resolution set to fixed
  • Status changed from new to closed

In 9531:

Blogs: Update main-nav text to include classes, to match other similar components. Props hnla. Fixes #6207.

Note: See TracTickets for help on using tickets.