Bug 1056301

Summary:	Using a single quote in a Homepage Content tag causes menu displacement
Product:	[Retired] Zanata	Reporter:	Damian Jansen <djansen>
Component:	Component-UI	Assignee:	Sean Flanigan <sflaniga>
Status:	CLOSED CURRENTRELEASE	QA Contact:	Ding-Yi Chen <dchen>
Severity:	medium	Docs Contact:
Priority:	unspecified
Version:	3.2	CC:	dchen, sflaniga, zanata-bugs
Target Milestone:	---
Target Release:	3.7
Hardware:	Unspecified
OS:	Unspecified
Whiteboard:
Fixed In Version:	3.7.0-SNAPSHOT (git-jenkins-zanata-server-github-pull-requests-3566)	Doc Type:	Bug Fix
Doc Text:		Story Points:	---
Clone Of:		Environment:
Last Closed:	2015-07-22 02:20:17 UTC	Type:	Bug
Regression:	---	Mount Type:	---
Documentation:	---	CRM:
Verified Versions:		Category:	---
oVirt Team:	---	RHEL 7.3 requirements from Atomic Host:
Cloudforms Team:	---	Target Upstream Version:
Embargoed:
Bug Depends On:	1198433
Bug Blocks:

Description Damian Jansen 2014-01-21 22:57:55 UTC

Description of problem:
If the admin user accidentally does something like
<a href='http://should.be.double.quotes' /> 
when editing their home page code, the actions menu will be pushed under the homepage

Version-Release number of selected component (if applicable):
3.3

How reproducible:
Easy always

Steps to Reproduce:
1. Sign in as admin, click Home button
2. Click Edit Page Code
3. Enter into the field
<a href='http://should.be.double.quotes' />
4. Press Update

Actual results:
Menu is now under the home page area, rather than to the right.
Console shows: Seam Text parse error: expecting a doublequote \", found '''

Expected results:
No change in menu placement

Additional info:
Only seems to affect the application homepage, not the project home pages

Comment 1 Sean Flanigan 2015-05-15 05:02:37 UTC

Eliminating Seam Text should eliminate Seam Text parser errors: https://github.com/zanata/zanata-server/pull/732

Comment 2 Ding-Yi Chen 2015-05-18 07:08:35 UTC

Tested with Zanata 3.7.0-SNAPSHOT (git-jenkins-zanata-server-github-pull-requests-3566)

The Seam Text to CommonMark failed to convert the text:
<a href='http://should.be.double.quotes' />

However, it printed 

The following text failed conversion from Seam Text to CommonMark by Zanata.  It may require editing to fix formatting. 

in the about page, which prompt project maintainer to fix.

Comment 3 Ding-Yi Chen 2015-05-18 08:20:45 UTC

VERIFIED with Zanata 3.7.0-SNAPSHOT (git-jenkins-zanata-server-github-pull-requests-3566)