Bug 38031 - Справка на старом языке после смены
Summary: Справка на старом языке после смены
Status: CLOSED FIXED
Alias: None
Product: Sisyphus
Classification: Development
Component: alterator-sysconfig (show other bugs)
Version: unstable
Hardware: x86_64 Linux
: P5 normal
Assignee: manowar@altlinux.org
QA Contact: qa-sisyphus
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2020-02-05 11:33 MSK by Sergey V Turchin
Modified: 2021-03-12 18:06 MSK (History)
9 users (show)

See Also:


Attachments
Каждый язык на своём языке в списке. (10.60 KB, image/png)
2021-02-18 16:39 MSK, Sergey V Turchin
no flags Details
screen (178.43 KB, image/png)
2021-03-10 11:31 MSK, Vera Blagoveschenskaya
no flags Details

Note You need to log in before you can comment on or make changes to this bug.
Description Sergey V Turchin 2020-02-05 11:33:13 MSK
При установке на этапе выбора языка при смене языка текст у виджетов меняется, а справка остается на старом языке.
Comment 1 Sergey V Turchin 2020-02-13 15:26:41 MSK
Он должен ещё раз "help" послать, но с новым значением.
Comment 2 manowar@altlinux.org 2020-03-31 14:42:57 MSK
Понял. Сейчас посмотрю.
Comment 3 manowar@altlinux.org 2020-04-06 18:16:34 MSK
Если добавить в alterator-sysconfig одну такую строчку:

 ui/sysconfig/base/index.scm:
 ...
 (define (change-translations)
 ...
+(simple-notify document:root 'action "language" 'value (current-language))
 ...)

то при смене языка, кроме смены заголовков виджетов, дополнительно передаётся action "language":


CMD: (auth-request user "anonymous" session-id "89786033" content (alterator-request action "event" name "changed" widget-id "w5" state ((w1 (current-action . __undefined__) (current-step . 0)) (w5 (current . 3)) (w8 (current . 2)))))

ANS: (auth-answer user "anonymous" session-id "89786033" content (("command" widget-id w2 action "language" value "uk_UA") ("command" widget-id w1 action "set" name "steps-clear" value "") ("command" widget-id w1 action "set" name "steps" value "Мова;steps/sysconfig-base;Preparing for install;steps/greeting;Ліцензійна угода;steps/notes-license;Дата й час;steps/datetime;Підготовка дисків;steps/vm;System installation;steps/install;Встановлення завантажувача;steps/lilo;Адміністратор системи;steps/root;Системний користувач;steps/users;Встановлення паролю на LUKS-підрозділи;steps/vm;Finish installation;steps/finish") ("command" widget-id w1 action "set" name "current-step" value "0") ("command" widget-id w1 action "set" name "action-text" value "help;Довідка") ("command" widget-id w1 action "set" name "action-text" value "forward;Далі") ("command" widget-id w4 action "set" name "text" value "Оберіть Вашу мову:") ("command" widget-id w7 action "set" name "text" value "Оберіть варіант переключення розкладки клавіатури:") ("command" widget-id w8 action "set" name "rows" value "Клавіша Control+Shift;") ("command" widget-id w8 action "set" name "current" value "0")))

В результате чего вызывается метод Browser::changeLanguage(language) с нужным значением:

Thread 1 "alterator-brows" hit Breakpoint 1, Browser::changeLanguage (
    this=0x7ffffa9b0830, language=...) at browser.cc:374
374	{
(gdb) p language == "uk_UA"
$10 = true

После нажатия на кнопку "Довiдка" срабатывает конструктор HelpWidget::HelpWidget(), в котором я вижу

    setPopupTitle(tr("Help"));

то-есть вроде бы установку заголовка окна на текущем языке, который после changeLanguage("uk_UA") уже должен быть украинским. Но в заголовке всё равно выводится "Справка".


P.S. Для alterator-browser-qt5 не формируется debuginfo-пакет.
Comment 4 Sergey V Turchin 2020-04-06 18:35:29 MSK
> Но в заголовке всё равно выводится "Справка".
Это я посмотрю. Если подсказка на кнопке "Next" меняет перевод, то остальное я в alterator-browser допилю.
Comment 5 Sergey V Turchin 2020-04-06 19:55:52 MSK
(Ответ для Sergey V Turchin на комментарий #4)
> Если подсказка на кнопке "Next" меняет перевод
Хотя, может и там баг, т.к. проверял только в тестовой програмульке.
В общем, если Browser::changeLanguage сработал, то этого достаточно.

Главное, чтоб "help" новый прилетел.
Comment 6 manowar@altlinux.org 2020-04-07 13:22:21 MSK
Подсказка "Next" перевод не меняет, хотя changeLanguage() вызывается. Ты посмотришь?
Comment 7 Sergey V Turchin 2020-04-07 13:42:57 MSK
(Ответ для manowar@altlinux.org на комментарий #6)
> Подсказка "Next" перевод не меняет, хотя changeLanguage() вызывается.
Да. Уже нашел баг со сменой языка внутрях.
Comment 8 Sergey V Turchin 2020-04-07 16:43:17 MSK
http://webery.altlinux.org/task/249536 "Help" теперь переводится соответственно выбранному языку с изменением в ui/sysconfig/base/index.scm .
Comment 9 Repository Robot 2020-04-14 12:15:53 MSK
alterator-sysconfig-1.3.3-alt1 -> sisyphus:

 Thu Apr 09 2020 Paul Wolneykien <manowar@altlinux> 1.3.3-alt1
 - Fix: Added "language" notification on language change (closes: 38031).
Comment 10 Sergey V Turchin 2020-04-14 13:45:34 MSK
А зачем
+Conflicts: alterator-browser-qt5 < 3.1.5-alt1
?

P.S.
Лишние конфликты ведут к лишним проблемам обновления. Особенно актуально с бранча на бранч.
Comment 11 Sergey V Turchin 2020-04-14 13:48:16 MSK
Я даже больше скажу: в alterator-browser-qt4 это работало скорее всего.

Если никто не против, я уберу конфликт.
Comment 12 manowar@altlinux.org 2020-04-14 14:35:14 MSK
(Ответ для Sergey V Turchin на комментарий #11)
> Я даже больше скажу: в alterator-browser-qt4 это работало скорее всего.
> 
> Если никто не против, я уберу конфликт.

  Но без него оно не будет closes: 38031.
Comment 13 Sergey V Turchin 2020-04-14 14:46:20 MSK
(Ответ для manowar@altlinux.org на комментарий #12)
>   Но без него оно не будет closes: 38031.
Будет. alterator-browser-qt5 > 3.1.5-alt1 (релиз-то зачем включать!) уже в репозитории.

Если нужны лишние формальности, то нужно завести отдельный баг на alterator-browser-qt5 и поставить зависимость _в_багзиле_ , а не в пакете.
Comment 14 Sergey V Turchin 2020-04-14 14:51:26 MSK
Кстати, это был починен баг #37914 .
Comment 15 manowar@altlinux.org 2020-04-14 15:30:34 MSK
(Ответ для Sergey V Turchin на комментарий #13)
> (Ответ для manowar@altlinux.org на комментарий #12)
> >   Но без него оно не будет closes: 38031.
> Будет. alterator-browser-qt5 > 3.1.5-alt1 (релиз-то зачем включать!) уже в
> репозитории.

Я точечное обновление имел в виду. Вот человек читает в changelog, что данная версия закрывает ошибку. Ставит пакет (никаких дополнительных он не вытягивает за собой), проверяет — ошибка воспроизводится.
Comment 16 Sergey V Turchin 2020-04-14 15:34:15 MSK
(Ответ для manowar@altlinux.org на комментарий #15)
> Я точечное обновление имел в виду.
У нас считается, что для выяснения, существует ли проблема, перед точечным обновлением необходимо сделать dist-upgrade.
Comment 17 Sergey V Turchin 2020-04-14 15:35:48 MSK
(Ответ для manowar@altlinux.org на комментарий #15)
> человек читает в changelog, что данная версия закрывает ошибку
Какую ошибку она закрывает, можно узнать только в багзиле, а там написано, от какого бага тот баг зависит.
Comment 18 Sergey V Turchin 2020-04-14 15:43:21 MSK
(Ответ для manowar@altlinux.org на комментарий #15)
> никаких дополнительных он не вытягивает за собой
Конкретная проблема больше в том, что он не "вытягивает", а "удаляет". Да и просто без особой необходимости.

В бороде из несколько подобных конфликтов оказывется пакет с кривыми зависимостями, после чего dist-upgrade с бранча на бранч или невозможен или выносит пол-системы. И никакого "точечного" не дадут. ;-)
Comment 19 Repository Robot 2020-04-16 13:48:13 MSK
alterator-sysconfig-1.3.3-alt2 -> p9:

 Tue Apr 14 2020 Sergey V Turchin <zerg@altlinux> 1.3.3-alt2
 - remove unnecessary conflict with alterator-browser-qt5
 Thu Apr 09 2020 Paul Wolneykien <manowar@altlinux> 1.3.3-alt1
 - Fix: Added "language" notification on language change (closes: 38031).
Comment 20 manowar@altlinux.org 2020-04-16 13:56:35 MSK
А чего ты тогда написал: "Кстати, это был починен баг #37914" ?
Comment 21 Sergey V Turchin 2020-04-16 14:08:07 MSK
Это робот сработал при отправке в p9.
Comment 22 manowar@altlinux.org 2021-02-18 15:28:01 MSK
А знаете какой ещё баг есть? Список языков всегда на русском языке! Может, стоит просто названия каждого языка писать на самом этом языке?
Comment 23 manowar@altlinux.org 2021-02-18 15:28:58 MSK
А смену языка справки я починил (сейчас приедет). Однако, справки на английском языке для этого модуля просто нет — тоже бага.
Comment 24 Repository Robot 2021-02-18 16:16:38 MSK
alterator-sysconfig-1.3.4-alt1 -> sisyphus:

 Thu Feb 18 2021 Paul Wolneykien <manowar@altlinux> 1.3.4-alt1
 - Send help URL on language change (closes: 38031).
Comment 25 Sergey V Turchin 2021-02-18 16:38:27 MSK
> Список языков всегда на русском языке! 
> Может, стоит просто названия каждого языка писать на самом этом языке?
Дык, так и есть.
Comment 26 Sergey V Turchin 2021-02-18 16:39:15 MSK
Created attachment 9205 [details]
Каждый язык на своём языке в списке.
Comment 27 manowar@altlinux.org 2021-02-18 16:44:30 MSK
(Ответ для Sergey V Turchin на комментарий #26)
> Создано вложение 9205 [details] [подробности]
> Каждый язык на своём языке в списке.

В образе c9 это почему-то не так. Хотя, может быть у меня старый образ.
Comment 28 Sergey V Turchin 2021-02-18 17:03:33 MSK
(Ответ для manowar@altlinux.org на комментарий #27)
> В образе c9 это почему-то не так.
Видимо, там баг, т.к. я такого ни разу не видел.
Comment 29 Vera Blagoveschenskaya 2021-03-10 11:29:29 MSK
(Ответ для manowar@altlinux.org на комментарий #23)
> А смену языка справки я починил (сейчас приедет). Однако, справки на
> английском языке для этого модуля просто нет — тоже бага.

alterator-sysconfig-1.3.4-alt1
Текст справки меняется, а заголовок "Справка" остается на Русском языке.
При этом если пойти в установщике Вперед, то и заголовок подтягивается.
ALT Workstation K 9.1 RC5
Comment 30 Vera Blagoveschenskaya 2021-03-10 11:31:09 MSK
Created attachment 9234 [details]
screen
Comment 31 Sergey V Turchin 2021-03-10 12:03:55 MSK
> Текст справки меняется, а заголовок "Справка" остается на Русском языке.
Да, сломалось. Перестало приходить событие "language" при смене языка, поэтому и всплывающая подсказка кнопки "Далее" тоже не меняется.
Comment 32 manowar@altlinux.org 2021-03-10 13:08:03 MSK
А как вы это проверяете? Последнюю регулярку нужно взять или как-то иначе?
Comment 33 Sergey V Turchin 2021-03-10 13:46:49 MSK
(Ответ для manowar@altlinux.org на комментарий #32)
> А как вы это проверяете?
Я прям локально alterator-wizard настроил и подсунул ему шаг выбора языка.
Comment 34 Sergey V Turchin 2021-03-10 13:50:05 MSK
> подсунул ему шаг выбора языка.
В /usr/share/install2/installer-steps и /usr/share/alterator/steps
Comment 35 manowar@altlinux.org 2021-03-10 14:08:06 MSK
(Ответ для Sergey V Turchin на комментарий #31)
> > Текст справки меняется, а заголовок "Справка" остается на Русском языке.
> Да, сломалось. Перестало приходить событие "language" при смене языка,
> поэтому и всплывающая подсказка кнопки "Далее" тоже не меняется.

Подтверждаю. В новом исправлении банально удаляется строчка от предыдущего исправления. Я работал со старой версией alterator-sysconfig, а пропажу в diff не заметил. Сейчас верну.
Comment 36 manowar@altlinux.org 2021-03-10 14:39:17 MSK
Wed Mar 10 2021 Paul Wolneykien <manowar@altlinux> 1.3.5-alt1
- Restore a lost commit from v1.3.3.
- Fix: Added "language" notification on language change (closes:
  38031).

Автозакрытие по второму разу не работает, что-ли?
Comment 37 Vera Blagoveschenskaya 2021-03-12 18:06:41 MSK
С заданием 
http://git.altlinux.org/tasks/267585/
alterator-sysconfig-1.3.5-alt1
ошибка исправлена.