Skip to content

PostgreSQL - tag similar module error#8240

Merged
wilsonge merged 1 commit intojoomla:stagingfrom
alikon:patch-47
Nov 2, 2015
Merged

PostgreSQL - tag similar module error#8240
wilsonge merged 1 commit intojoomla:stagingfrom
alikon:patch-47

Conversation

@alikon
Copy link
Copy Markdown
Contributor

@alikon alikon commented Nov 2, 2015

Steps to reproduce the issue

Set the tag similar module parameter Order Results to random

Click on Similar Tags item from the fronted left menu All Modules

Actual result

you got a generic error message

After Patch

no error message

Addictional Comments

similar to #8238

LetsUseTheApiThanWeCanSuccess

was used RAND() instead of the api $query->Rand()

#### Steps to reproduce the issue
set the tag similar module parameter Order Results to random

Click on Similar Tags item from the fronted left menu All Modules

#### Actual result
#### After Patch
#### Addictional Comments
#LetsUseTheApiThanWeCanSuccess
@waader
Copy link
Copy Markdown
Contributor

waader commented Nov 2, 2015

I have tested this item ✅ successfully on 12ca87d

Thanks!


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

@zero-24
Copy link
Copy Markdown
Contributor

zero-24 commented Nov 2, 2015

I have tested this item ✅ successfully on 12ca87d

hehe 😃 Thanks. Still works on mysql. -> RTC


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

@zero-24 zero-24 added this to the Joomla! 3.5.0 milestone Nov 2, 2015
@zero-24
Copy link
Copy Markdown
Contributor

zero-24 commented Nov 2, 2015

RTC


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

@joomla-cms-bot joomla-cms-bot added the RTC This Pull Request is Ready To Commit label Nov 2, 2015
wilsonge added a commit that referenced this pull request Nov 2, 2015
PostgreSQL - tag similar module error
@wilsonge wilsonge merged commit 9531959 into joomla:staging Nov 2, 2015
@joomla-cms-bot joomla-cms-bot removed the RTC This Pull Request is Ready To Commit label Nov 2, 2015
@alikon alikon deleted the patch-47 branch November 2, 2015 13:02
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants