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

Cannot put two content languages with "No image" #12907

Closed
andrepereiradasilva opened this issue Nov 16, 2016 · 6 comments
Closed

Cannot put two content languages with "No image" #12907

andrepereiradasilva opened this issue Nov 16, 2016 · 6 comments

Comments

@andrepereiradasilva
Copy link
Contributor

andrepereiradasilva commented Nov 16, 2016

Steps to reproduce the issue

Add two content languages with no image.

Expected result

Two content languages with no image.

Actual result

SQL Error
image

System information (as much as possible)

latest

Additional comments

there seems to be an unique index idx_image for image field in #__languages table

probably a missing part from #12051

@brianteeman
Copy link
Contributor

I can confirm this


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

@brianteeman
Copy link
Contributor

Removing the unique index resolves it - does this even need an index?


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

@mbabker
Copy link
Contributor

mbabker commented Nov 16, 2016

https://github.com/joomla/joomla-cms/blob/staging/libraries/joomla/table/language.php#L71

The code requires a unique image per language. It's not just the table indexes. I never understood it but for whatever reason I got conned to adding it onto https://developer.joomla.org/joomlacode-archive/issue-27196.html

@brianteeman
Copy link
Contributor

brianteeman commented Nov 16, 2016 via email

@andrepereiradasilva
Copy link
Contributor Author

Imho It can still check for the same image except when no image.

@andrepereiradasilva
Copy link
Contributor Author

please test #12949

rdeutz pushed a commit that referenced this issue Dec 6, 2016
…2949)

* change index type

* on installation

* remove extra space
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

4 participants