diff --git a/source/java/org/alfresco/web/bean/wcm/SubmitDialog.java b/source/java/org/alfresco/web/bean/wcm/SubmitDialog.java index 7a478a0d2d..914189a8a6 100644 --- a/source/java/org/alfresco/web/bean/wcm/SubmitDialog.java +++ b/source/java/org/alfresco/web/bean/wcm/SubmitDialog.java @@ -349,6 +349,15 @@ public class SubmitDialog extends BaseDialogBean String submitLabel = this.label; String submitComment = this.comment; + // crop to maximum length + if (submitLabel != null && submitLabel.length() > 255) + { + submitLabel = submitLabel.substring(0, 255); + } + if (submitComment != null && submitComment.length() > 255) + { + submitComment = submitComment.substring(0, 255); + } // note: always submit via workflow (if no workflow selected, then defaults to direct submit workflow) getSandboxService().submitListAssets(sbStoreId, relativePaths, diff --git a/source/java/org/alfresco/web/ui/common/Utils.java b/source/java/org/alfresco/web/ui/common/Utils.java index bf697ab44e..eaed64ff9b 100644 --- a/source/java/org/alfresco/web/ui/common/Utils.java +++ b/source/java/org/alfresco/web/ui/common/Utils.java @@ -568,11 +568,11 @@ public final class Utils extends StringUtils if (alt != null) { alt = Utils.encode(alt); - buf.append(" alt='") + buf.append(" alt=\"") .append(alt) - .append("' title='") + .append("\" title=\"") .append(alt) - .append("'"); + .append("\""); } else { @@ -653,11 +653,11 @@ public final class Utils extends StringUtils if (alt != null) { alt = Utils.encode(alt); - buf.append(" alt='") + buf.append(" alt=\"") .append(alt) - .append("' title='") + .append("\" title=\"") .append(alt) - .append("'"); + .append('"'); } else { diff --git a/source/java/org/alfresco/web/ui/common/renderer/ActionLinkRenderer.java b/source/java/org/alfresco/web/ui/common/renderer/ActionLinkRenderer.java index cc0be349f3..a02fd1f455 100644 --- a/source/java/org/alfresco/web/ui/common/renderer/ActionLinkRenderer.java +++ b/source/java/org/alfresco/web/ui/common/renderer/ActionLinkRenderer.java @@ -270,9 +270,9 @@ public class ActionLinkRenderer extends BaseRenderer } if (link.getTooltip() != null) { - out.write(" title='"); + out.write(" title=\""); out.write(Utils.encode(link.getTooltip())); - out.write("'"); + out.write('"'); } out.write('>'); } diff --git a/source/java/org/alfresco/web/ui/common/renderer/ImagePickerRadioRenderer.java b/source/java/org/alfresco/web/ui/common/renderer/ImagePickerRadioRenderer.java index 315525ce50..390429e3c4 100644 --- a/source/java/org/alfresco/web/ui/common/renderer/ImagePickerRadioRenderer.java +++ b/source/java/org/alfresco/web/ui/common/renderer/ImagePickerRadioRenderer.java @@ -311,9 +311,9 @@ public class ImagePickerRadioRenderer extends BaseRenderer if (tooltip != null) { - out.write(" title='"); + out.write(" title=\""); out.write(Utils.encode(tooltip)); - out.write("'"); + out.write('"'); } if (onclick != null) @@ -323,11 +323,6 @@ public class ImagePickerRadioRenderer extends BaseRenderer out.write("'"); } -// if (item.isDisabled()) -// { -// out.write(" disabled='true'"); -// } - out.write(">"); out.write("