Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

module related item - postgresql #5243

Closed
wants to merge 2 commits into from
Closed

module related item - postgresql #5243

wants to merge 2 commits into from

Conversation

alikon
Copy link
Contributor

@alikon alikon commented Nov 29, 2014

Install 3.4alpha with the testing sample data on postgressql
select related items from the All modules menu
you'll get an Database Error query failed
cause of DATE_FORMAT() not supported by postgres
and cause of line 114 crpytic one

Install 3.4alpha with the testing sample data on postgressql
select related items from the All modules menu 
you'll get an Database Error query failed
cause of DATE_FORMAT() not supported by postgres
and cause of line 114 crpytic one
@waader
Copy link
Contributor

waader commented Nov 29, 2014

@test success with mysql and postgresql

@brianteeman
Copy link
Contributor

@test - issue and fix confirmed

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

@brianteeman
Copy link
Contributor

Setting RTC

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

@brianteeman brianteeman added the RTC This Pull Request is Ready To Commit label Nov 30, 2014
@Bakual
Copy link
Contributor

Bakual commented Dec 2, 2014

Removing RTC label for now. @alikon needs to fix some code which slipped in 😉

@Bakual Bakual removed the RTC This Pull Request is Ready To Commit label Dec 2, 2014
@jissues-bot jissues-bot added the RTC This Pull Request is Ready To Commit label Dec 2, 2014
@brianteeman brianteeman removed the RTC This Pull Request is Ready To Commit label Dec 2, 2014
please retest
@alikon
Copy link
Contributor Author

alikon commented Dec 3, 2014

should be fixed now 😇

@waader
Copy link
Contributor

waader commented Dec 3, 2014

@test works!

@Bakual Bakual closed this in d9538d3 Dec 3, 2014
@Bakual
Copy link
Contributor

Bakual commented Dec 3, 2014

Merged, thanks!

@roland-d roland-d added this to the Joomla! 3.4.0 milestone Feb 3, 2015
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.

None yet

6 participants