What's the crack here then with most active in this topic?

[Day_]

Shouldn't they be in order? 

1439, 967, 862, 852

9, 7, 4, 4

Seems rather random if you ask me.

[Randy Calvert]

If I had to venture a guess, it's displaying them in order of first post into the topic.  

[Marc]

Do you have a link to the page you see that please. These should indeed be in order

[Day_]

6 minutes ago, Marc Stridgen said:

Do you have a link to the page you see that please. These should indeed be in order

Sure.

https://dcfcfans.uk/topic/38855-the-administration-thread/#comments

This is the topic in the first screenshot example

One more where its all over the place.

https://dcfcfans.uk/topic/37392-wayne-rooney/

 

[Marc]

We would need to look further into this for you, however the access details on file appear to be incorrect or missing. Could you please update these details by visiting your client area, selecting the relevant purchase, then clicking "Review/Update Access Information" under the "Stored Access Information" section. 

We look forward to further assisting you. 

 

[Day_]

4 minutes ago, Marc Stridgen said:

We would need to look further into this for you, however the access details on file appear to be incorrect or missing. Could you please update these details by visiting your client area, selecting the relevant purchase, then clicking "Review/Update Access Information" under the "Stored Access Information" section. 

We look forward to further assisting you. 

Sorted, should be good to go now.

I look forward to you assisting me further 🙂 

[Nathan Explosion]

@Marc Stridgen

Just take a look at any long topic on here:

The order is based on member_id, not the number of posts.

IPS\Content\Statistics - the topPosters function...

When it gets to this point, $contributors is in the right order.

		$return = array();
		foreach( new \IPS\Patterns\ActiveRecordIterator( \IPS\Db::i()->select( '*', 'core_members', array( \IPS\Db::i()->in( 'member_id', $contributors ) ) ), 'IPS\Member' ) as $member )
		{
			$return[] = array( 'member' => $member, 'count' => $counts[ $member->member_id ] );
		}

The above ends up returning an array in member_id order.

[Day_]

3 minutes ago, Nathan Explosion said:

@Marc Stridgen

Just take a look at any long topic on here:

The order is based on member_id, not the number of posts.

IPS\Content\Statistics - the topPosters function...

When it gets to this point, $contributors is in the right order.

		$return = array();
		foreach( new \IPS\Patterns\ActiveRecordIterator( \IPS\Db::i()->select( '*', 'core_members', array( \IPS\Db::i()->in( 'member_id', $contributors ) ) ), 'IPS\Member' ) as $member )
		{
			$return[] = array( 'member' => $member, 'count' => $counts[ $member->member_id ] );
		}

The above ends up returning an array in member_id order.

Is this working as intended then, doesn’t make much sense if it is.

I changed the “Top Posters” wording on my site as members kicked off saying quantity does not = quality, so went with most active.

If it was top posters, surely it would be better going in order rather than member ID 🤔

 

 

[Marc]

Thank you for bringing this issue to our attention! I can confirm this should be further reviewed and I have logged an internal bug report for our development team to investigate and address as necessary, in a future maintenance release.

 

[Day_]

9 minutes ago, Marc Stridgen said:

Thank you for bringing this issue to our attention! I can confirm this should be further reviewed and I have logged an internal bug report for our development team to investigate and address as necessary, in a future maintenance release.

This is why you’re my favourite. Thank you Marc!

[Marc]

Just now, Day_ said:

This is why you’re my favourite. Thank you Marc!

[Marc]

This issue has been resolved in 4.7.0 which has recently been released. Please upgrade to that version to resolve this, and of course, let us know if you are still seeing any issues.

[Day_]

Just now, Marc Stridgen said:

This issue has been resolved in 4.7.0 which has recently been released. Please upgrade to that version to resolve this, and of course, let us know if you are still seeing any issues.

Yep, already installed and looking good our end. 👍

[Marc]

56 minutes ago, Day_ said:

Yep, already installed and looking good our end. 👍

Thank you for letting us know 🙂