Skip to content

Conversation

@sebalix
Copy link
Contributor

@sebalix sebalix commented Jan 8, 2016

This is a common issue with OpenERP/Odoo since a long time: the translation of months and days in non-english dates.
The PR is not a nice fix (a proper fix should be global, and not in this module especially), so it's OK if it is refused.
What do you think?

@sebalix sebalix force-pushed the 8.0_datetime_formatter_locale branch 3 times, most recently from 8066864 to 28651bf Compare January 8, 2016 12:01
@hbrunn
Copy link
Member

hbrunn commented Jan 13, 2016

I think your approach is okay. A module that monkey patches this into core would be nicer, but well.

Two questions:

  • resetlocale sets the locale to the default locale. Wouldn't it be better to save the current locale and reset that? There might be scenarios where some other code set the locale to whatever else
  • I think LC_TIME wins against LC_ALL. Wouldn't it be better to set/reset that?

@sebalix
Copy link
Contributor Author

sebalix commented Jan 14, 2016

@hbrunn yep, LC_TIME is a better option, fixed. Travis error is related to base_external_dbsource.

@hbrunn
Copy link
Member

hbrunn commented Jan 14, 2016

👍 verified pylint errors are unrelated

@hbrunn hbrunn removed the question label Jan 14, 2016
@hbrunn
Copy link
Member

hbrunn commented Feb 1, 2016

@sebalix please rebase for green builds

@sebalix sebalix force-pushed the 8.0_datetime_formatter_locale branch from fafa299 to cd5940b Compare February 1, 2016 16:45
@sebalix
Copy link
Contributor Author

sebalix commented Feb 1, 2016

Rebased

@github-actions
Copy link

github-actions bot commented Dec 5, 2021

There hasn't been any activity on this pull request in the past 4 months, so it has been marked as stale and it will be closed automatically if no further activity occurs in the next 30 days.
If you want this PR to never become stale, please ask a PSC member to apply the "no stale" label.

@github-actions github-actions bot added the stale PR/Issue without recent activity, it'll be soon closed automatically. label Dec 5, 2021
@github-actions github-actions bot closed this Jan 9, 2022
SiesslPhillip pushed a commit to grueneerde/OCA-server-tools that referenced this pull request Nov 20, 2024
Syncing from upstream OCA/server-tools (11.0)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

8.0 needs review stale PR/Issue without recent activity, it'll be soon closed automatically.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants