Openbravo Issue Tracking System - Openbravo ERP
View Issue Details
IDProjectCategoryView StatusDate SubmittedLast Update
0036857Openbravo ERPA. Platformpublic2017-09-14 13:092017-09-15 11:35
Reporteralostale 
Assigned Toalostale 
PrioritynormalSeverityminorReproducibilityhave not tried
StatusclosedResolutionfixed 
PlatformOS5OS Version
Product Version 
Target VersionFixed in Version3.0PR17Q4 
Merge Request Status
Review Assigned Tocaristu
OBNetwork customerNo
Web browser
ModulesCore
Support ticket
Regression level
Regression date
Regression introduced in release
Regression introduced by commit
Triggers an Emergency PackNo
Summary0036857: having \u2028 or \u2029 characters in recent documents title makes UI to fail
DescriptionWhen there is a document that includes on its identifier \u2028 or \u2029 characters, Openbravo fails to load.
Steps To Reproduce1. Follow steps in 0036788 creating BP with name and search key with those charactres
2. Log out
3. Log in
  -> ERROR: Openbravo is no loaded because the generated JavaScript is not valid
Proposed SolutionThe problem is caused because even the JavaScript is generated using js_string in freemarker, current version (2.3.16) doesn't escape reported ones properly. This issue is fixed since 2.3.20 [1][2].

A temporary fix will be implemented until freemarked library is updated.

--
[1] https://sourceforge.net/p/freemarker/bugs/390/ [^]
[2] https://github.com/apache/incubator-freemarker/commit/e6daec6e63b967faea9579f63bfad333bb22e9ad [^]
Additional Information
TagsNo tags attached.
Relationships
related to defect 0036788 closed alostale adding a \u2028 (line separator) character to any field makes UI to fail 
related to feature request 0036858 acknowledged Triage Platform Base update freemarker 
Attached Files
Issue History
Date ModifiedUsernameFieldChange
2017-09-14 13:09alostaleNew Issue
2017-09-14 13:09alostaleAssigned To => platform
2017-09-14 13:09alostaleOBNetwork customer => No
2017-09-14 13:09alostaleModules => Core
2017-09-14 13:09alostaleTriggers an Emergency Pack => No
2017-09-14 13:09alostaleRelationship addedrelated to 0036788
2017-09-14 13:09alostaleReview Assigned To => caristu
2017-09-14 13:09alostaleAssigned Toplatform => alostale
2017-09-14 13:13alostaleRelationship addedrelated to 0036858
2017-09-14 13:26hgbotCheckin
2017-09-14 13:26hgbotNote Added: 0099040
2017-09-14 13:26hgbotStatusnew => resolved
2017-09-14 13:26hgbotResolutionopen => fixed
2017-09-14 13:26hgbotFixed in SCM revision => http://code.openbravo.com/erp/devel/pi/rev/756e6fab2ecf9db6c03b93fb5e7b250a8be4bb23 [^]
2017-09-15 11:35caristuNote Added: 0099064
2017-09-15 11:35caristuStatusresolved => closed
2017-09-15 11:35caristuFixed in Version => 3.0PR17Q4

Notes
(0099040)
hgbot   
2017-09-14 13:26   
Repository: erp/devel/pi
Changeset: 756e6fab2ecf9db6c03b93fb5e7b250a8be4bb23
Author: Asier Lostalé <asier.lostale <at> openbravo.com>
Date: Thu Sep 14 13:25:35 2017 +0200
URL: http://code.openbravo.com/erp/devel/pi/rev/756e6fab2ecf9db6c03b93fb5e7b250a8be4bb23 [^]

fixed 36857: invalid js generated if \u2028 or \u2029 chars are in recent docs

  Implemented temporary fix until freemarker gets updated. Dynamic js resources
  now escape those special characters.

---
M modules/org.openbravo.client.kernel/src/org/openbravo/client/kernel/SessionDynamicResourceComponent.java
---
(0099064)
caristu   
2017-09-15 11:35   
Code reviewed + testing OK.