Last modified: 2014-09-23 19:47:46 UTC

Wikimedia Bugzilla is closed!

Wikimedia migrated from Bugzilla to Phabricator. Bug reports are handled in Wikimedia Phabricator.
This static website is read-only and for historical purposes. It is not possible to log in and except for displaying bug reports and their history, links might be broken. See T32654, the corresponding Phabricator task for complete and up-to-date bug report information.
Bug 30654 - extension intersection (DynamicPageList) needs a patch to add 'random' sort order
extension intersection (DynamicPageList) needs a patch to add 'random' sort o...
Status: NEW
Product: MediaWiki extensions
Classification: Unclassified
DynamicPageList (Other open bugs)
unspecified
All All
: Normal normal with 1 vote (vote)
: ---
Assigned To: Nobody - You can work on this!
: patch, patch-need-review
: 33306 (view as bug list)
Depends on:
Blocks:
  Show dependency treegraph
 
Reported: 2011-08-31 09:48 UTC by xiangfu
Modified: 2014-09-23 19:47 UTC (History)
5 users (show)

See Also:
Web browser: ---
Mobile Platform: ---
Assignee Huggle Beta Tester: ---


Attachments
patch for the bug (562 bytes, patch)
2011-08-31 09:48 UTC, xiangfu
Details

Description xiangfu 2011-08-31 09:48:58 UTC
Created attachment 8993 [details]
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
Comment 1 Bawolff (Brian Wolff) 2011-09-05 13:07:39 UTC
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.
Comment 2 Mark A. Hershberger 2011-10-28 18:51:26 UTC
(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.
Comment 3 Bawolff (Brian Wolff) 2011-12-21 23:08:00 UTC
*** Bug 33306 has been marked as a duplicate of this bug. ***
Comment 4 Miroslav Langer 2012-02-20 00:15:56 UTC
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.
Comment 5 Sumana Harihareswara 2012-05-25 03:17:14 UTC
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

Note You need to log in before you can comment on or make changes to this bug.


Navigation
Links