Bug 44699 - Typos (external feedback)
Typos (external feedback)
Status: CLOSED FIXED
Product: UCS@school
Classification: Unclassified
Component: UMC - Wizards
UCS@school 4.4
Other Linux
: P5 normal (vote)
: UCS@school 4.4 v9-errata
Assigned To: Toni Röhmeyer
Tobias Wenzel
:
Depends on:
Blocks:
  Show dependency treegraph
 
Reported: 2017-05-29 09:26 CEST by Alexander Kläser
Modified: 2021-05-06 14:10 CEST (History)
2 users (show)

See Also:
What kind of report is it?: Bug Report
What type of bug is this?: 1: Cosmetic issue or missing function but workaround exists
Who will be affected by this bug?: 5: Will affect all installed domains
How will those affected feel about the bug?: 1: Nuisance – not a big deal but noticeable
User Pain: 0.029
Enterprise Customer affected?:
School Customer affected?:
ISV affected?:
Waiting Support:
Flags outvoted (downgraded) after PO Review:
Ticket number:
Bug group (optional): Typo/text changes
Max CVSS v3 score:


Attachments

Note You need to log in before you can comment on or make changes to this bug.
Description Alexander Kläser univentionstaff 2017-05-29 09:26:10 CEST
We received feedback for the following typos:

ucsschool/ucs-school-umc-printermoderation/umc/python/printermoderation/fr.po
-----------------------------------------------------------------------------

File %r could not be printed as it does not exists (anymore). 
=>
File %r could not be printed as it does not exist (anymore). 


ucsschool/ucs-school-umc-wizards/umc/python/schoolwizards/fr.po
---------------------------------------------------------------

The %s %r does not exists or might have been removed in the meanwhile.
->
The %s %r does not exist or might have been removed in the meantime.
Comment 1 Toni Röhmeyer univentionstaff 2020-10-23 19:17:06 CEST
I fixed those typos with commits

e234b313c Bug #44699: removed typos in fr.po
88deac8cb Bug #44699: remove typos

on branch troehmey/bug44699_remove_typos
Comment 2 Tobias Wenzel univentionstaff 2020-12-07 13:50:26 CET
QA:

The correction is OK, but:

The texts which can be found in the *.po-files are updated by our poupdate-tools in the toolshed. If this is done otherwise, your changes might be overwritten with the next change. Please change the text inside the __init__.py and run the command.
Comment 3 Toni Röhmeyer univentionstaff 2021-03-11 09:50:31 CET
Applied suggestions from comment #2 with 

a7d778962 Bug #44699: fixup
d37676224 Bug #44699: fixed typos, updated po files
Comment 4 Tobias Wenzel univentionstaff 2021-03-30 13:45:32 CEST
QA → looks OK, but I need some info before closing this.

I'm sorry this is taking so long.

I just had a look at this and could reproduce the scenario like this

- Tested by opening a class in the schoolclass umc modul 
- deleting the class in the common umc modul
- saving the (already deleted) class in the school modul

The SchoolClass u'DEMOSCHOOL-testi' does not exists or might have been removed in the meanwhile.

After the fix

The SchoolClass u'DEMOSCHOOL-testi' does not exist or might have been removed in the meantime.

How did you test your changes in l.240ff?
Comment 5 Toni Röhmeyer univentionstaff 2021-04-15 16:34:16 CEST
You can simply use the test 40_schoolwizards_admin_workaround.py which expects the error messages which are affected by this bug (from ucs-school-umc-wizards).

The test needed a minor patch, which specifies the expected output language:

ee382531f Bug #44699: specifying output language in test 

In order to check if the german translation is working, just change the language in the create_umc_client function from language="en_US" to "de_DE"
Comment 6 Tobias Wenzel univentionstaff 2021-04-16 09:05:35 CEST
Reopen

Code → OK
Tests pass → OK

please squash your commits, rebase, merge & build
Comment 7 Toni Röhmeyer univentionstaff 2021-04-18 21:59:49 CEST
Merged to 4.4 with:

874b93fdb Bug #44699: added advisories
bb222e34f Bug #44699: added changelog entries
7217e0771 Bug #44699: Merge branch 'troehmey/bug44699_remove_typos' into 4.4
f1186e1d1 Bug #44699: specifying output language in test
a14c2c4c6 Bug #44699: remove typos

Builds:

Package: ucs-school-umc-printermoderation
Version: 16.0.0-3A~4.4.0.202104182151
Branch: ucs_4.4-0
Scope: ucs-school-4.4

Package: ucs-school-umc-wizards
Version: 11.0.0-14A~4.4.0.202104182154
Branch: ucs_4.4-0
Scope: ucs-school-4.4

Package: ucs-test-ucsschool
Version: 6.0.205A~4.4.0.202104182157
Branch: ucs_4.4-0
Scope: ucs-school-4.4
Comment 8 Tobias Wenzel univentionstaff 2021-04-19 08:39:03 CEST
All OK → verify

merge → OK
changelogs → OK
advisories → OK
jenkins → is happy
Comment 9 Daniel Tröder univentionstaff 2021-04-20 08:54:36 CEST
The Debian package for ucs-school-umc-printermoderation has only been imported into the build system, but has not been built.
Comment 10 Tobias Wenzel univentionstaff 2021-04-22 14:50:14 CEST
Package: ucs-school-umc-printermoderation
Version: 16.0.0-4A~4.4.0.202104221447
Branch: ucs_4.4-0
Scope: ucs-school-4.4

→ set to resolve, verify after jenkins
Comment 11 Tobias Wenzel univentionstaff 2021-04-28 08:47:40 CEST
jenkins is happy -> set to verify
Comment 12 Tobias Wenzel univentionstaff 2021-05-06 14:10:32 CEST
Errata updates for UCS@school 4.4 v9 have been released.

https://docs.software-univention.de/changelog-ucsschool-4.4v9-de.html

If this error occurs again, please clone this bug.