Bug #10256

Treeview does not appear when using certain cultures

Added by Dan Gillean over 5 years ago. Updated over 5 years ago.

Status:VerifiedStart date:09/02/2016
Priority:HighDue date:
Assignee:-% Done:

0%

Category:I18N
Target version:Release 2.3.1
Google Code Legacy ID: Tested version:2.3
Sponsored:No Requires documentation:No

Description

First reported via the user forum: https://groups.google.com/d/msg/ica-atom-users/UiRO6PTo-sA/vkqeQkH1CAAJ

I followed the official instructions and installed it in a new machine with Ubuntu 16.04. Everything works fine, but somehow if I choose "pt" (Portuguese), español, français, and some other languages the treeview does not appear. When I choose English, Nederlands and, for instance, "pt_BR" language, the treeview does appear. It doesn't matter if you choose sidebar treeview or full width treeview, because some languages will show it and some won't. I tried setting the culture to "pt", "pt_BR", "en", etc, but that didn't fixed it. It is an odd behaviour so I'm reporting it here.

Tested and reproduced in 2.3

To reproduce

  • Add the following cultures to the user interface via Admin > Settings > i18n languages: German (de), Spanish (es), Portuguese (pt)
  • Remember to re-index after adding new cultures
  • Test 1: Create a record with children in the English interface, save. Flip interface to de, es, or pt.
  • Test 2: Flip user interface to de, es, or pt, and create a new description with children, save.

Resulting error

  • Treeview is not shown (regardless of full-width or sidebar setting) in specific cultures

Expected result

  • Treeview appears as expected, and as in other cultures.

Notes:

  • See also related issue for i18n use - #10173, Treeview quicksearch does not support culture fallback
  • The original report listed French (fr) as one of the languages where the treeview was not appearing, but I could not reproduce in this culture (worked for me)
  • I did not test every available culture in AtoM. Hopefully when we discover the cause, it will be a global fix and not something that has to be applied individually to each culture setting

Related issues

Related to Access to Memory (AtoM) - Bug #10469: i18n tasks attempt to translate QubitSetting fixtures Verified 10/24/2016

History

#2 Updated by Nick Wilkinson over 5 years ago

  • Assignee set to Jesús García Crespo

Hi Jesús, can you please investigate?

#3 Updated by Jesús García Crespo over 5 years ago

  • Related to Bug #10469: i18n tasks attempt to translate QubitSetting fixtures added

#4 Updated by Jesús García Crespo over 5 years ago

  • Status changed from New to Code Review
  • Assignee changed from Jesús García Crespo to José Raddaoui Marín

#6 Updated by José Raddaoui Marín over 5 years ago

  • Status changed from Code Review to Feedback
  • Assignee changed from José Raddaoui Marín to Jesús García Crespo

Looks great!

#7 Updated by Jesús García Crespo over 5 years ago

  • Status changed from Feedback to QA/Review
  • Assignee changed from Jesús García Crespo to Dan Gillean

Dan, this is now in qa/2.4.x and not yet in stable/2.3.x because I made a change in the internals that may cause unexpected issues, e.g. it caused an issue in the language menu, which is now fixed. So I would recommend doing some testing before we move to stable. The chances are low but it would be irresponsible otherwise.

#8 Updated by Dan Gillean over 5 years ago

  • Status changed from QA/Review to Verified
  • Target version set to Release 2.3.1
  • Requires documentation set to No

#9 Updated by Jesús García Crespo over 5 years ago

  • Status changed from Verified to Feedback
  • Assignee changed from Dan Gillean to José Raddaoui Marín

Radda has found a bug in the solution I wrote and he's working on it.

#10 Updated by José Raddaoui Marín over 5 years ago

  • Status changed from Feedback to Code Review
  • Assignee changed from José Raddaoui Marín to Jesús García Crespo

Dan, the issue I found happens after changing the culture back to English, when the sidebar tree-view appears even if the setting is set to the full width one.

PR 458

#11 Updated by Jesús García Crespo over 5 years ago

  • Status changed from Code Review to Feedback
  • Assignee changed from Jesús García Crespo to José Raddaoui Marín

#12 Updated by José Raddaoui Marín over 5 years ago

  • Status changed from Feedback to QA/Review
  • Assignee changed from José Raddaoui Marín to Dan Gillean

Fix added to qa/2.4.x and stable/2.3.x (alongside Sevein's commits)

#13 Updated by Dan Gillean over 5 years ago

  • Status changed from QA/Review to Verified
  • Assignee deleted (Dan Gillean)

I tried adding a bunch of languages, editing the titles of descriptions in various languages, and flipping back and forth between them and English - couldn't break it! Looks good to me. Tried with both types of treeview.

Also available in: Atom PDF