Page MenuHomePhabricator

extension intersection (DynamicPageList) needs a patch to add 'random' sort order
Open, MediumPublic

Description

Author: xiangfu

Description:
patch for the bug

extension intersection (DynamicPageList) needs a patch to add 'random' sort order. we have to apply the attachment patch
for enalbe the 'random' sort order. more info:

http://en.qi-hardware.com/wiki/Server_setup#MediaWiki

Version: unspecified
Severity: normal

Attached:

Details

Reference
bz30654

Event Timeline

bzimport raised the priority of this task from to Medium.Nov 21 2014, 11:50 PM
bzimport set Reference to bz30654.
bzimport added a subscriber: Unknown Object (MLST).

My main concern with this patch is that when combined with caching, it only randomly re-orders stuff on a cache purge, which isn't really what people want when they specify "random" (I think anyways) Although it would work for some use-cases.

(In reply to comment #1)

My main concern with this patch is that when combined with caching, it only
randomly re-orders stuff on a cache purge, which isn't really what people want
when they specify "random"

But I think it may be what we want.

  • Bug 33306 has been marked as a duplicate of this bug. ***

okino wrote:

Some randomizer would be a great tool in DPL, and I would prefer a full randomizer, which reshuffles not only the order of pre-selected results, but also the changes the selection of results itself every time the cache is reloaded. It would be very useful for our planned Czech Wikinews page (and I believe for others as well).

So please, if anyone can, take a look at it. Thanks a lot in advance.

M.L.

sumanah wrote:

xiangfu, thank you for the patch. I am sorry for the delay in review. Are you interested in using developer access to directly suggest it into our Git source control system? That will get it reviewed faster.

https://www.mediawiki.org/wiki/Developer_access

https://www.mediawiki.org/wiki/Git/Workflow#How_to_submit_a_patch