Cannot add JavaScript to a site when the logo is changed

Description

A user is unable to add JavaScript to a page when the default logo is replaced.

Here are the steps to reproduce:

1. Go to Control Panel
2. Go to Site Pages
3. Go to Logo and change the logo to a different image
4. Click "Save
5. Go to JavaScript
6. Type the following line into the text field: var test = document.getElementById('footer');test.innerHTML = "Hello World";
7. Click "Save"
8. The user will see exceptions and an error message. Also, the "Powered by Liferay" line at the bottom of all site pages will not be replaced by "Hello World"

If the user clicks "Save" after both changing logo and adding the JavaScript (clicking save at step 4 and step 7), the error will appear. If the user clicks "Save" only at step 8, and not at step 4, the error will not be reproduced.

Environment

Tomcat 7.0.27 + MySQL 5. Portal 6.1.x EE GIT ID: 90a8e1919860e253dc32cfcdbc6bb7486b973243. Tomcat 7.0.27 + MySQL 5. Portal 6.2.x GIT ID: daf320a35f5cb0a6a6327c5625c112419eb38f3c.

Activity

Show:

ginson.renJune 25, 2012 at 2:50 AM

PASSED Manual Testing following the steps in the description.

Reproduced on:
Tomcat 7.0 + MySQL 5. Portal 6.1.x EE GIT ID: 5508b31daae05330d8248d8837b19cfb84befe70.

The user will see exceptions and an error message. Also, the "Powered by Liferay" line at the bottom of all site pages will not be replaced by "Hello World".

Fixed on:
Tomcat 7.0 + MySQL 5. Portal 6.1.x CE GIT ID: a6bb8318c74e89e66c95f96492f9ccaf793d390d.
Tomcat 7.0 + MySQL 5. Portal 6.1.x EE GIT ID: 7f511cfcd3b2d650941684c4ecbfb1d745bda676.
Tomcat 7.0 + MySQL 5. Portal 6.2.x GIT ID: ba8883b6a82800614c845a9be6e3f02a72948369.

There are no exceptions and error message. Also, the "Powered by Liferay" line at the bottom of all site pages will be replaced by "Hello World".

Michael SaechangJune 22, 2012 at 5:14 PM

Committed on:
Portal 6.1.x CE GIT ID: 2e07051b59b5b3149de55a052e1d6b9f0baf8ecb.
Portal 6.2.x GIT ID: 248ceedab92a0dece47138106203985cb20f2ffc.

Ken DuenwaldJune 20, 2012 at 5:14 PM

Downgrading to minor due to a workaround present.

Ken DuenwaldJune 20, 2012 at 12:37 PM

Attempting to locate commit that caused the regression

Fixed

Details

Assignee

Reporter

Labels

Branch Version/s

6.1.x

Backported to Branch

Committed

Fix Priority

5

Git Pull Request

Bug Type

Regression Bug

Components

Affects versions

Priority

Zendesk Support

Created June 20, 2012 at 12:35 PM
Updated June 24, 2023 at 3:52 PM
Resolved June 22, 2012 at 10:13 AM
Loading...