[Liam W] (RIP) Post Count Rebuild 2.0.3

[Liam W]

Liam W submitted a new resource:

Post Count Rebuild - Correct those post counts!

This is a simple add-on that adds a rebuilder to the 'Rebuild caches' page to rebuild your boards users message counts.

This takes into account whether or not the forum counts posts to the post count.

Read more about this resource...

 

[Kevin]

Any chance of being able to run this from the CLI?

 

[anon666]

Any chance of being able to run this from the CLI?

I'd like to know that too, and thank you so much for the add-on, mr. @Liam W

 

[Nirjonadda]

@Liam W Rebuild user post counts, does this excluded from option Count messages posted in this forum toward user total? Also Please can add Cron entries for run automatically and re-check user total post count?

 

[Liam W]

I'm not sure what you're asking? This add-on will respect that option, so any posts in those nodes will not be counted.

Liam

 

[SeToY]

Can this be used to upgrade from the XF1.5 version or is it recommended to remove the XF1.5 version beforehand?

 

[Nirjonadda]

@Liam W Does this addon work with Xenforo 2.1.1?

 

[ivp]

It doesn't take into account whether posts/threads are visible.

Here is the fix:

$user->message_count = $this->app->finder('XF:Post')->where('user_id', $id)
	->where('Thread.Forum.count_messages', 1)->total();

should be replaced with:

$db = $this->app->db();
$user->message_count = $db->fetchOne("
	SELECT COUNT(*)
	FROM xf_post AS post
	LEFT JOIN xf_thread AS thread ON (thread.thread_id = post.thread_id)
	LEFT JOIN xf_forum AS forum ON (forum.node_id = thread.node_id)
	WHERE post.user_id = ?
	AND post.message_state = 'visible' AND thread.discussion_state = 'visible'
	AND forum.count_messages = 1
", $id);

 

[Liam W]

I could’ve sworn I’d already changed that... will take a look later this evening.

Thanks!

 

[Liam W]

Just checked, and I changed this locally when adding support for a CLI rebuilder, but never released it ‍... whoops!

 

[Nirjonadda]

@Liam W Do you forget adding Developer info?

 

[Liam W]

Yeah, this was one of the earlier XF2 add-ons I made, I missed a lot of metadata

Liam

 

[Liam W]

Liam W updated Post Count Rebuild with a new update entry:

Bug Fixes & CLI

• Add CLI rebuild command
• Improve rebuild
  • Rebuild now considers visibility of posts and thread

Read the rest of this update entry...

 

[ivp]

Many thanks for the update.

Both @Brogan and @Jake Bunce used LEFT JOIN instead of INNER JOIN, probably we should go that road:

Rebuild user post counts (query)

Jake Bunce submitted a new resource: Rebuild user post counts (query) (version 1.x) - A database query to rebuild all user post counts. This will recalculate and update the post counts for all users at once. This is a large query that may take a while to finish. UPDATE xf_user AS user SET...

xenforo.com

Rebuild user post counts (query)

Suppose a user has 1000 messages but 300 of them were deleted for one reason or another. I'd like that in the postbit of the user there are 700 messages and not 1000 so I'm trying to make XF recount the users's messages not including the deleted ones...

xenforo.com

 

[SeToY]

Brogan also filtered "AND forum.node_id IS NOT NULL" whereas the other one didn't. Wonder if it will make a difference.

Edit: Found one. https://xenforo.com/community/threa...all-user-post-counts-query.19736/#post-252980

 

[Liam W]

Brogan also filtered "AND forum.node_id IS NOT NULL" whereas the other one didn't. Wonder if it will make a difference.

It won't, actually, as this checks to ensure the forum counts messages, so if there is no forum defined for a post, then that value will be null, and will not match the condition.

It should also be noted that that condition was specifically intended to exclude orphaned threads, which was an XF1 bug and therefore should no longer be a major issue.

 

[Liam W]

Additionally, the use of INNER JOIN also prevents the existence of orphaned threads, as it requires that both the join columns exist for the row to be provided.

 

[JoyFreak]

Does this work with 2.1?

 

[Liam W]

Does this work with 2.1?

Should do. Let me update the listing.

 

[JoyFreak]

Should do. Let me update the listing.

Wow, talk about swift response haha. Thanks!