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

Tage manager not workign in dutch language #5380

Closed
jtesser opened this issue Apr 14, 2014 · 1 comment
Closed

Tage manager not workign in dutch language #5380

jtesser opened this issue Apr 14, 2014 · 1 comment
Assignees
@dotcmsCodeReview
Labels
Merged QA : Approved Type : Defect
Milestone
2.5.5

Comments

@jtesser
Copy link
Contributor

jtesser commented Apr 14, 2014

The issue is a ' in the property. This needs to be escaped properly for the HTML/JS
@jtesser jtesser added this to the 3.0 milestone Apr 14, 2014
@jtesser jtesser added the bug label Apr 14, 2014
@ravit ravit closed this as completed in b332eb2 Apr 16, 2014
jtesser pushed a commit that referenced this issue Apr 16, 2014
Merge pull request #5405 from dotCMS/issue-5380-Tag-manager-not-worki…
289b86e 
…ng-in-dutch

fixes #5380 Tag manager not working in dutch
@jtesser jtesser assigned dotcmsCodeReview and unassigned ravit Apr 16, 2014
@bryanboza
Copy link
Member

Fixed, Tested on Nightly Build 2014-04-25

ravit added a commit that referenced this issue May 19, 2014
@ravit
fixes #5380 Tag manager not working in dutch
128d907
@jtesser jtesser modified the milestones: 2.5.5, 3.0 May 19, 2014
ravit added a commit that referenced this issue Sep 10, 2014
@ravit
fixes #5380 Tag manager not working in dutch
3e8ef7a 
Former-commit-id: 27e5282bf3a6b0484c8d9ac2735969a332fdcf9a
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@dotcmsCodeReview dotcmsCodeReview

Labels
Merged QA : Approved Type : Defect
Projects
None yet
Milestone
2.5.5
Development

No branches or pull requests
4 participants
@jtesser @dotcmsCodeReview @ravit @bryanboza