liferay / liferay-aui-upgrade-tool

Upgrade themes/plugins from Liferay Portal 6.1 to 6.2
13 stars 94 forks source link

Quotation marks being added to scriptlets #15

Closed ipeychev closed 11 years ago

ipeychev commented 11 years ago

Note: this issue was imported, but it was originally created by Robert-Frampton...

This issue seems to be caused by commit cf4fb14 - Fix #9 Change "handler" to "on: click" for Toolbar

Before:

cssClass='<%= showConnectedRequestedIcon ? "disabled" : "disabled aui-helper-hidden" %>'

After:

cssClass='<%=' showConnectedRequestedIcon ? "disabled" : "disabled helper-hidden" %>'

Notice the extra single quotation after the opening of the scriptlet. Same thing is happening with double quotations.

Before:

<li class="<%= phone.isPrimary() ? "primary" : "" %>">

After:

<li class="<%=" phone.isPrimary() ? "primary" : "" %>">

This appears to only happen in 'class', 'cssClass', and 'className' properties. For example.

<aui:input cssClass="<%= scriptlet %>">
<div class="<%= scriptlet %>">
<liferay-ui:ratings
    className="<%= scriptlet %>"
/>

These scriptlets would all receive the extra quotation mark.

However, the issue does not occur with the 'classPK' property.

ipeychev commented 11 years ago

Note: this comment was imported, but it was originally made by Robert-Frampton...

Hey @ipeychev,

I updated the description of the issue, I narrowed it down to three properties that receive the extra quotation.