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

Crash while opening Options dialog after switching locale #50239

Closed
2 tasks done
DelazJ opened this issue Sep 19, 2022 · 7 comments · Fixed by #50263
Closed
2 tasks done

Crash while opening Options dialog after switching locale #50239

DelazJ opened this issue Sep 19, 2022 · 7 comments · Fixed by #50263
Assignees
Labels
Bug Either a bug report, or a bug fix. Let's hope for the latter! Crash/Data Corruption

Comments

@DelazJ
Copy link
Contributor

DelazJ commented Sep 19, 2022

What is the bug or the crash?

I'm on a French locale based machine but used to play with QGIS in English. In order to test a fix involving locale, I unchecked the "override locale" option, restarted QGIS, and while trying to reopen the Settings dialog, I got a crash. Here's so far what I had in the console. I tried with a new profile but still the crash.

src/core/settings/qgssettingsentryenumflag.h:82 : (convertFromVariant) [5356ms] Invalid enum/flag key/s 'JoinStyleRound' for settings key 'qgis/digitizing/offset_join_style'
Fatal: ASSERT failure in QgsOptions::checkPageWidgetNameMap(): "QgisApp::optionsPagesMap() is outdated, contains too many entries", file /media/delazj/TRAVAIL/dev/github/QGIS/src/app/options/qgsoptions.cpp, line 1289
QGIS died on signal -1src/app/qgscrashhandler.cpp:71 : (handleCrash) [158ms] /tmp/qgis-crash-info-38841
src/app/qgscrashhandler.cpp:124 : (handleCrash) [0ms] /media/delazj/TRAVAIL/dev/github/QGIS/build-ninja/output/bin/qgiscrashhandler
Extra Info File: /tmp/qgis-crash-info-38841

Trying to open "Custom projection" also triggers the crash (probably because it opens the settings dialog? I can't remember if it is the case)

Steps to reproduce the issue

See above

Versions

<style type="text/css"> p, li { white-space: pre-wrap; } </style>
Version de QGIS 3.27.0-Master Révision du code 6f9da19
Version de Qt 5.15.3
Version de Python 3.10.4
Version de GDAL/OGR 3.4.1
Version de Proj 8.2.1
Version de la base de données du registre EPSG v10.041 (2021-12-03)
Version de GEOS 3.10.2-CAPI-1.16.0
Version de SQLite 3.37.2
Version de PDAL 2.3.0
Version du client PostgreSQL unknown
Version de SpatiaLite 5.0.1
Version de QWT 6.1.4
Version de QScintilla2 2.11.6
Version de l'OS Ubuntu 22.04.1 LTS
       
Cette copie de QGIS dispose d'une sortie de débogage.
       
Extensions Python actives
db_manager 0.1.20
grassprovider 2.12.99
MetaSearch 0.3.6
processing 2.12.99
sagaprovider 2.12.99

Supported QGIS version

  • I'm running a supported QGIS version according to the roadmap.

New profile

  • I tried with a new QGIS profile

Additional context

No response

@DelazJ DelazJ added Bug Either a bug report, or a bug fix. Let's hope for the latter! Crash/Data Corruption labels Sep 19, 2022
@DelazJ
Copy link
Contributor Author

DelazJ commented Sep 19, 2022

Opening QGIS with ./output/bin/qgis --lang en I can reopen the Settings dialog but ./output/bin/qgis --lang fr still fails

@elpaso
Copy link
Contributor

elpaso commented Sep 20, 2022

I've seen this multiple times with non-english locales, it is probably related to missing translations for the entries in the panel.
I guess we should make it more tolerant and not crash or at least have a meaningful message that points to the source of the issue. I'll have a look.

@elpaso elpaso self-assigned this Sep 20, 2022
@elpaso
Copy link
Contributor

elpaso commented Sep 20, 2022

@DelazJ new message:
Fatal: ASSERT failure in QgsOptions::checkPageWidgetNameMap(): "QgisApp::optionsPagesMap() is outdated, contains too many entries (diff: Rendu)"

"Rendu" is missing, does it make any sense to you?

@DelazJ
Copy link
Contributor Author

DelazJ commented Sep 20, 2022

Maybe this one not being translated? (I assume the Rendering tab in Options dialog)image

@DelazJ
Copy link
Contributor Author

DelazJ commented Sep 20, 2022

hmmm I can see the tab translated in 3.26
Ah... scratch above, the Rendering tab has been revamped in 3.27, so yes the translation might have been lost

@DelazJ
Copy link
Contributor Author

DelazJ commented Sep 20, 2022

@elpaso Do you want me to translate and give it a day to see how it plays (hmmm... translations are daily pulled in master or only at the release time? @jef-n )

@jef-n
Copy link
Member

jef-n commented Sep 20, 2022

translations are daily pulled in master or only at the release time? @jef-n

at release time. but the nightlies pull on each build.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Bug Either a bug report, or a bug fix. Let's hope for the latter! Crash/Data Corruption
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants