Odoo Help


strftime %T not working in base web_editor module

on 1/4/16, 6:56 AM 335 views

Inside the main controller file within the base module web_editor, inside the following function:

export_icon_to_png (line 76)

there are the following two lines that throw an error due to the %T directive.

The lines in question are line 125 and 126:

response.headers['Date'] = time.strftime("%a, %d-%b-%Y %T GMT", time.gmtime()) 

response.headers['Expires'] = time.strftime("%a, %d-%b-%Y %T GMT", time.gmtime(time.time()+604800*60))

From some research it appears that %T is not within python's strftime library, or perhaps it is just an OS specific issue, but on windows it definitely doesn't work and quite possibly on some Linux distributions. in PHP strftime api references it would seem it represents the same as "%H:%M:%S".

I'm creating this question/ticket just to make the Odoo team aware of it if they aren't aware already, perhaps the longhand version (%H:%M:%S) should be used for safety in the base module in question, in case the instance of odoo in being used is running on an OS that doesn't support some of the directives in the strftime library.

Any further info would be greatly appreciated, for now I've just removed the %T in order to get export_icon_to_png function to work as it should.



Your Answer

Please try to give a substantial answer. If you wanted to comment on the question or answer, just use the commenting tool. Please remember that you can always revise your answers - no need to answer the same question twice. Also, please don't forget to vote - it really helps to select the best questions and answers!

About This Community

This platform is for beginners and experts willing to share their Odoo knowledge. It's not a forum to discuss ideas, but a knowledge base of questions and their answers.


Odoo Training Center

Access to our E-learning platform and experience all Odoo Apps through learning videos, exercises and Quizz.

Test it now

Question tools

1 follower(s)


Asked: 1/4/16, 6:56 AM
Seen: 335 times
Last updated: 1/8/16, 5:34 AM