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

Spanish Latin America Language error. #2285

Closed
cadamuroa opened this issue Jun 30, 2021 · 6 comments
Closed

Spanish Latin America Language error. #2285

cadamuroa opened this issue Jun 30, 2021 · 6 comments
Assignees
Milestone

Comments

@cadamuroa
Copy link

Describe the bug
When we created one form and register it as Spanish Latin America language, the system is not saving correctly.

To Reproduce
Steps to reproduce the behavior:

  1. Go to 'Administration -> Forms'
  2. Create a new form and put the language as 'Spanish Latin America'
  3. Look the languagem in the list the system shows es_41.

Looking the table I saw that the field language have just 5 caracters and the code of spanish for latin america is 6.

Screenshots

image

image

Additional context
I'm using the version of 9.5.5 (glpi) and form creator (2.11.4)

@cadamuroa
Copy link
Author

The code of latin america spanish is es_419.

@btry
Copy link
Collaborator

btry commented Jun 30, 2021

Hi

Indeed, the column is too short. Change it to VARCHAR(255). I'll add a fix for the next release.

btry added a commit that referenced this issue Jun 30, 2021
fix #2285

Signed-off-by: Thierry Bugier <tbugier@teclib.com>
@btry btry self-assigned this Jun 30, 2021
@btry btry added this to the 2.12.0 milestone Jun 30, 2021
@btry btry added the bug label Jun 30, 2021
btry added a commit that referenced this issue Jun 30, 2021
fix #2285

Signed-off-by: Thierry Bugier <tbugier@teclib.com>
@btry
Copy link
Collaborator

btry commented Jul 1, 2021

Hi

Did you changed the column of the table ? Does it resolve your issue ?

@cadamuroa
Copy link
Author

I changed the column of the thable and the language was saved correctly.

@cadamuroa
Copy link
Author

@btry ,

Let me know if you need something more!

Thank you

@btry
Copy link
Collaborator

btry commented Jul 1, 2021

Hi

I just needed a confirmation that the fix works as expected. It is ready for the next release.

Thank you for your feedback.

@btry btry closed this as completed Jul 1, 2021
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

2 participants