? ? ? Success

User tests: Successful: Unsuccessful:

avatar xaddict
xaddict
3 Apr 2015

I've added in random ordering in this module since it's something I think should be in the core and it's one of the first things I change in new Joomla installations.

avatar xaddict xaddict - open - 3 Apr 2015
avatar joomla-cms-bot joomla-cms-bot - change - 3 Apr 2015
Labels Added: ?
avatar brianteeman
brianteeman - comment - 3 Apr 2015

Please provide full test instructions otherwise people will just ignore it.

A good starting point would be to state which module it is that you have edited


This comment was created with the J!Tracker Application at issues.joomla.org/joomla-cms/6648.
avatar xaddict
xaddict - comment - 3 Apr 2015

modules/mod_articles_category/mod_articles_category.xml

avatar xaddict xaddict - change - 3 Apr 2015
Title
Update for random ordering
Update for random ordering in mod_articles_category
avatar xaddict
xaddict - comment - 3 Apr 2015

There are no test instructions for previous state as there was no random ordering,

In the new situation one can choose "Random" (if it also gets pulled) in ordering at the backend and the selected articles from mod_articles_category will be shown randomly.

avatar zero-24 zero-24 - change - 3 Apr 2015
Category Modules
avatar xaddict
xaddict - comment - 7 Apr 2015

I added the other options into this one. It should now be sufficient to add the "Random" ordering option to the Articles Category module.

avatar brianteeman brianteeman - change - 17 Jun 2015
Labels Added: ? ?
avatar brianteeman
brianteeman - comment - 17 Jun 2015

Sorry but I am confused. You have added an options called Random to an option

"Article Field to order by"

So that definitely isnt the correct place for it
Would it be better to place it in the field below

"Ordering Direction"

Although that would mean that it was renamed to just Ordering


This comment was created with the J!Tracker Application at issues.joomla.org/joomla-cms/6648.

avatar brianteeman
brianteeman - comment - 24 Jul 2015

Unfortunately this will not work in Postrges which does not have a rand function


This comment was created with the J!Tracker Application at issues.joomla.org/joomla-cms/6648.

avatar brianteeman brianteeman - change - 12 Nov 2015
Status Pending Information Required
Labels
avatar brianteeman
brianteeman - comment - 12 Nov 2015

@xaddict Thanks for your contribution. The last comment on this comes from myself stating that it cannot be accepted as is because the code doesnt work with Postgres.

Are you able to update the code so that it will? If not then this will have to be closed


This comment was created with the J!Tracker Application at issues.joomla.org/joomla-cms/6648.

avatar xaddict
xaddict - comment - 25 Nov 2015

You can go ahead and close this. I'm doing this via a custom written module
now which supports random ordering, offset (take 10 items, but only show
the last 5) and splitting (one module shows items 1-3, another one 4-6, the
last shows random items but not 1-6)

On Thu, 12 Nov 2015 at 16:06 Brian Teeman notifications@github.com wrote:

@xaddict https://github.com/xaddict Thanks for your contribution. The
last comment on this comes from myself stating that it cannot be accepted
as is because the code doesnt work with Postgres.

Are you able to update the code so that it will? If not then this will

have to be closed

This comment was created with the J!Tracker Application
https://github.com/joomla/jissues at issues.joomla.org/joomla-cms/6648
https://issues.joomla.org/tracker/joomla-cms/6648.


Reply to this email directly or view it on GitHub
#6648 (comment).

avatar Bakual
Bakual - comment - 25 Nov 2015

Closing per request and since it has open issues anyway.

avatar Bakual Bakual - change - 25 Nov 2015
Status Information Required Closed
Closed_Date 0000-00-00 00:00:00 2015-11-25 10:00:50
Closed_By Bakual
Labels
avatar Bakual Bakual - close - 25 Nov 2015
avatar zero-24
zero-24 - comment - 25 Nov 2015

If there is a willing to make it work on multi database you can use "random" as option value and implement it later with

case 'random' :
    $orderby = JFactory::getDbo()->getQuery(true)->Rand();
    break;
avatar alikon
alikon - comment - 25 Nov 2015

sorry @xaddict unable to send a pr to your branch
nice feature i think , i'll submit a new one, db portable as @zero-24 suggested

avatar zero-24
zero-24 - comment - 25 Nov 2015

:+1:

avatar alikon
alikon - comment - 25 Nov 2015

see #8538

Add a Comment

Login with GitHub to post a comment