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

[13.0][MIG] event_session #210

Merged
merged 24 commits into from
Apr 6, 2021
Merged

Conversation

dalonsod
Copy link
Contributor

@dalonsod dalonsod commented Apr 5, 2021

Standard migration. Depends on #194

chienandalu and others added 23 commits April 5, 2021 13:46
* Rename 'date' to 'date_begin' for being homogeneous with event fields
* Split tests
* Remove 'start_time' and 'end_time' as no needed
* Convert hours to UTC for avoiding timezone issues
- Fix tz in calculated name
- Fix tests
- Add event sessions count to reports
- Fixes computing update of stored value
- Fixes views and reports
- Fixes generator not creating last day sessions
- Totalizations in session tree view
- Security config
- Adds es_ES translations
- Fixes seats limits on sessions
- Fix report query
- Adds progress bar to sessions tree view
- Improves tests
- Fixes seats limit check
Updated by "Update PO files to match POT (msgmerge)" hook in Weblate.

Translation: event-12.0/event-12.0-event_session
Translate-URL: https://translation.odoo-community.org/projects/event-12-0/event-12-0-event_session/
The big issue with this kind of code is that if you have a user with a
certain locale being used, it will override the locale system wide and
the next request on the server may have the dates formatted with
strftime in a different language than expected...

The other issue is that some systems do not provide a set of actual
locales so on a server running in docker without manually modifying
a dockerfile to install a set of locales, it may fail to switch
the locale to a non existing one. It will just fail to compute the field.

Babel is already a dependency of odoo and it's a bit strange to not use it.

Get the locale the same way ir.qweb.fields do

This commit use the same method as ir.qweb.fields to get the locale
to use to translate a date. The previous code didn't took into account
a case where the lang is undefined and this case also use the same
default language as defined in odoo en_US.

As a result, the behaviour of event_session is more in line with
the current behaviour in odoo which is to default to en_US if locale
is undefined. With the global locale setting, the default locale
would be the system one so it may give different result or strange
behaviours or simply prevent correctly working without manual
intervention on the host.. While babel is already supported and used
by odoo.

Modified the test to actually test correctly the date formatted

As the default lang is en_US the format of a date is the following:

- month/day/year

And time format is on 12 hours so not 22:00 to 23:00 but 10:00 PM to 11:00 PM

With those changes, setting the locale to en_UK,en_CA will render the correct date using

- day/month/year and hours on 24h.

Added tests to confirm the translations work correctly on some locales

Check case when locale=None should default to en_US
Check that lang change is correctly taken into account.
Check that it works with languages returning unicode text (Russian)
Updated by "Update PO files to match POT (msgmerge)" hook in Weblate.

Translation: event-12.0/event-12.0-event_session
Translate-URL: https://translation.odoo-community.org/projects/event-12-0/event-12-0-event_session/
Updated by "Update PO files to match POT (msgmerge)" hook in Weblate.

Translation: event-12.0/event-12.0-event_session
Translate-URL: https://translation.odoo-community.org/projects/event-12-0/event-12-0-event_session/
@dalonsod dalonsod force-pushed the 13.0-mig-event_session branch 3 times, most recently from ea0d319 to 1bff4df Compare April 5, 2021 15:13
@dalonsod
Copy link
Contributor Author

dalonsod commented Apr 5, 2021

@chienandalu @sergio-teruel could you review? Thanks!

@rousseldenis rousseldenis added this to the 13.0 milestone Apr 6, 2021
@rousseldenis rousseldenis mentioned this pull request Apr 6, 2021
29 tasks
Copy link
Member

@chienandalu chienandalu left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks @dalonsod

@pedrobaeza
Copy link
Member

/ocabot merge nobump

@OCA-git-bot
Copy link
Contributor

This PR looks fantastic, let's merge it!
Prepared branch 13.0-ocabot-merge-pr-210-by-pedrobaeza-bump-nobump, awaiting test results.

@OCA-git-bot OCA-git-bot merged commit 725714e into OCA:13.0 Apr 6, 2021
@OCA-git-bot
Copy link
Contributor

Congratulations, your PR was merged at 69732a3. Thanks a lot for contributing to OCA. ❤️

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet