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.

Sign up for GitHub

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

Jump to bottom

Documents in some languages produce a server error #748

Closed
mmo opened this issue Jan 17, 2022 · 0 comments · Fixed by #749
Closed

Documents in some languages produce a server error #748

mmo opened this issue Jan 17, 2022 · 0 comments · Fixed by #749
Assignees
@Garfield-fr
Labels
bug Breaks something but is not blocking p-High To set a high priority!

Comments

@mmo
Copy link
Collaborator

mmo commented Jan 17, 2022 •
edited
Loading

When accessing https://sonar.ch/global/documents/309058, a 500 Internal server error message is displayed and the document cannot be shown in the user interface. The document is in Polish :

"language": [{ "type": "bf:Language", "value": "pol" }]

Given that Polish is not in the list SONAR_APP_LANGUAGES_MAP, an exception is raised in get_language_from_bibliographic_code().

The data model supports dozens of languages, but only a few of them are included in the SONAR_APP_LANGUAGES_MAP variable.
@mmo mmo added bug Breaks something but is not blocking p-High To set a high priority! labels Jan 17, 2022
Garfield-fr added a commit to Garfield-fr/sonar that referenced this issue Jan 17, 2022
@Garfield-fr
languages: add some mapping alpha-3 to alpha-2
373db97 
* Closes rero#748.

Co-Authored-by: Bertrand Zuchuat <bertrand.zuchuat@rero.ch>
@Garfield-fr Garfield-fr mentioned this issue Jan 17, 2022
Merged
7 tasks
@Garfield-fr Garfield-fr self-assigned this Jan 18, 2022
Garfield-fr added a commit to Garfield-fr/sonar that referenced this issue Jan 18, 2022
@Garfield-fr
languages: add some mapping alpha-3 to alpha-2
8ccbef3 
* Closes rero#748.

Co-Authored-by: Bertrand Zuchuat <bertrand.zuchuat@rero.ch>
Garfield-fr added a commit to Garfield-fr/sonar that referenced this issue Jan 18, 2022
@Garfield-fr
languages: add some mapping alpha-3 to alpha-2
db58efb 
* Closes rero#748.

Co-Authored-by: Bertrand Zuchuat <bertrand.zuchuat@rero.ch>
Garfield-fr added a commit to Garfield-fr/sonar that referenced this issue Jan 18, 2022
@Garfield-fr
languages: add some mapping alpha-3 to alpha-2
5a9095d 
* Closes rero#748.

Co-Authored-by: Bertrand Zuchuat <bertrand.zuchuat@rero.ch>
rerowep pushed a commit that referenced this issue Jan 20, 2022
@Garfield-fr @rerowep
languages: add some mapping alpha-3 to alpha-2
203693c 
* Closes #748.

Co-Authored-by: Bertrand Zuchuat <bertrand.zuchuat@rero.ch>
@mmo mmo closed this as completed in #749 Jan 31, 2022
mmo pushed a commit that referenced this issue Jan 31, 2022
@Garfield-fr @mmo
languages: add some mapping alpha-3 to alpha-2
5df26dd 
* Closes #748.

Co-Authored-by: Bertrand Zuchuat <bertrand.zuchuat@rero.ch>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@Garfield-fr Garfield-fr

Labels
bug Breaks something but is not blocking p-High To set a high priority!
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

languages: add some mapping alpha-3 to alpha-2 Garfield-fr/sonar
2 participants
@Garfield-fr @mmo