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.

Sign up for GitHub

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

Jump to bottom

BR def name_get() Issue when use Group By on Task by business-requirement #114

Closed
victormmtorres opened this issue Apr 11, 2017 · 3 comments
Closed

BR def name_get() Issue when use Group By on Task by business-requirement #114

victormmtorres opened this issue Apr 11, 2017 · 3 comments
Labels
bug
Milestone
8.0

Comments

@victormmtorres
Copy link
Collaborator

BR name_get use format() built in Python lib and if some string is unicode will happen like this:

Apr 11 10:44:25 superman docker/trunk_odoo-elico_1[806]: File "/opt/odoo/additional_addons/business-requirement/business_requirement/models/business.py", line 256, in name_get
Apr 11 10:44:25 superman docker/trunk_odoo-elico_1[806]: formatted_name = '[{}] {}'.format(br.name, br.description)
Apr 11 10:44:25 superman docker/trunk_odoo-elico_1[806]: UnicodeEncodeError: 'ascii' codec can't encode characters in position 29-37: ordinal not in range(128
@victormmtorres
Copy link
Collaborator Author

@elicoidal @seb-elico

@seb-elico
Copy link
Collaborator

@victormartinelicocorp maybe you can try with the % substitution

@victormmtorres
Copy link
Collaborator Author

Yes indeed it's need to be changed by another method. I will create an Issue as well for us.
Here I need @elicoidal add the tags bug and help wanted

@elicoidal elicoidal added this to the 8.0 milestone Apr 11, 2017
@elicoidal elicoidal changed the title [8.0] BR def name_get() Issue when use Group By on Task by business-requirement BR def name_get() Issue when use Group By on Task by business-requirement Apr 11, 2017
victormmtorres pushed a commit to victormmtorres/business-requirement that referenced this issue Apr 13, 2017
[FIX] business_requirement
69eeab9 
To fix OCA#114
@victormmtorres victormmtorres mentioned this issue Apr 13, 2017
Merged
elicoidal pushed a commit that referenced this issue Apr 20, 2017
@elicoidal
[FIX][business_requirement] Name formating improvement (#116)
c12bf62 
* [FIX] business_requirement
To fix #114

* [FIX] business_requirement
Fix name_get method as peer code review comments
Need to force string to unicode before replace
ruter-lyu pushed a commit to ruter-lyu/business-requirement that referenced this issue Mar 15, 2019
[FIX][business_requirement] Name formating improvement (OCA#116)
078fbad 
* [FIX] business_requirement
To fix OCA#114

* [FIX] business_requirement
Fix name_get method as peer code review comments
Need to force string to unicode before replace
victormmtorres pushed a commit to Tecnativa/business-requirement that referenced this issue Jun 4, 2019
[FIX][business_requirement] Name formating improvement (OCA#116)
f707f5b 
* [FIX] business_requirement
To fix OCA#114

* [FIX] business_requirement
Fix name_get method as peer code review comments
Need to force string to unicode before replace
pedrobaeza pushed a commit to Tecnativa/business-requirement that referenced this issue Jun 21, 2019
@pedrobaeza
[IMP][MIG] business_requirement (OCA#112)
23a8056 
Need to add reponsible and reviewers.
Responsible should be required and populate on upgrade version
Reviewers are users who are involved on the BR process

[FIX][business_requirement] Name formating improvement (OCA#116)

* [FIX] business_requirement
To fix OCA#114

* [FIX] business_requirement
Fix name_get method as peer code review comments
Need to force string to unicode before replace
victormmtorres pushed a commit to Tecnativa/business-requirement that referenced this issue Jun 26, 2019
[IMP][MIG] business_requirement (OCA#112)
deeda49 
Need to add reponsible and reviewers.
Responsible should be required and populate on upgrade version
Reviewers are users who are involved on the BR process

[FIX][business_requirement] Name formating improvement (OCA#116)

* [FIX] business_requirement
To fix OCA#114

* [FIX] business_requirement
Fix name_get method as peer code review comments
Need to force string to unicode before replace
Tardo pushed a commit to Tecnativa/business-requirement that referenced this issue Jan 24, 2020
@Tardo
[IMP][MIG] business_requirement (OCA#112)
b5ba877 
Need to add reponsible and reviewers.
Responsible should be required and populate on upgrade version
Reviewers are users who are involved on the BR process

[FIX][business_requirement] Name formating improvement (OCA#116)

* [FIX] business_requirement
To fix OCA#114

* [FIX] business_requirement
Fix name_get method as peer code review comments
Need to force string to unicode before replace
victoralmau pushed a commit to Tecnativa/business-requirement that referenced this issue Oct 14, 2021
@victoralmau
[IMP][MIG] business_requirement (OCA#112)
76798cb 
Need to add reponsible and reviewers.
Responsible should be required and populate on upgrade version
Reviewers are users who are involved on the BR process

[FIX][business_requirement] Name formating improvement (OCA#116)

* [FIX] business_requirement
To fix OCA#114

* [FIX] business_requirement
Fix name_get method as peer code review comments
Need to force string to unicode before replace
victoralmau pushed a commit to Tecnativa/business-requirement that referenced this issue Oct 14, 2021
@victoralmau
[IMP][MIG] business_requirement (OCA#112)
dc05be7 
Need to add reponsible and reviewers.
Responsible should be required and populate on upgrade version
Reviewers are users who are involved on the BR process

[FIX][business_requirement] Name formating improvement (OCA#116)

* [FIX] business_requirement
To fix OCA#114

* [FIX] business_requirement
Fix name_get method as peer code review comments
Need to force string to unicode before replace
MosabWadea pushed a commit to MosabWadea/business-requirement that referenced this issue Mar 5, 2022
@MosabWadea
[IMP][MIG] business_requirement (OCA#112)
077b0ab 
Need to add reponsible and reviewers.
Responsible should be required and populate on upgrade version
Reviewers are users who are involved on the BR process

[FIX][business_requirement] Name formating improvement (OCA#116)

* [FIX] business_requirement
To fix OCA#114

* [FIX] business_requirement
Fix name_get method as peer code review comments
Need to force string to unicode before replace
AntoniRomera pushed a commit to AntoniRomera/business-requirement that referenced this issue Jul 4, 2023
[IMP][MIG] business_requirement (OCA#112)
1591711 
Need to add reponsible and reviewers.
Responsible should be required and populate on upgrade version
Reviewers are users who are involved on the BR process

[FIX][business_requirement] Name formating improvement (OCA#116)

* [FIX] business_requirement
To fix OCA#114

* [FIX] business_requirement
Fix name_get method as peer code review comments
Need to force string to unicode before replace
NachoAlesLopez pushed a commit to NachoAlesLopez/business-requirement that referenced this issue Apr 28, 2024
@NachoAlesLopez
[IMP][MIG] business_requirement (OCA#112)
59b05aa 
Need to add reponsible and reviewers.
Responsible should be required and populate on upgrade version
Reviewers are users who are involved on the BR process

[FIX][business_requirement] Name formating improvement (OCA#116)

* [FIX] business_requirement
To fix OCA#114

* [FIX] business_requirement
Fix name_get method as peer code review comments
Need to force string to unicode before replace
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
bug
Projects
None yet
Milestone
8.0
Development

No branches or pull requests
3 participants
@elicoidal @seb-elico @victormmtorres