Author: dmaliarevich Date: 2008-02-20 07:43:40 -0500 (Wed, 20 Feb 2008) New Revision: 6461 Modified: trunk/vpe/plugins/org.jboss.tools.vpe/src/org/jboss/tools/vpe/editor/template/VpeDataTableCreator.java Log: http://jira.jboss.com/jira/browse/JBIDE-1744, code adjustment Modified: trunk/vpe/plugins/org.jboss.tools.vpe/src/org/jboss/tools/vpe/editor/template/VpeDataTableCreator.java =================================================================== --- trunk/vpe/plugins/org.jboss.tools.vpe/src/org/jboss/tools/vpe/editor/template/VpeDataTableCreator.java 2008-02-20 12:40:55 UTC (rev 6460) +++ trunk/vpe/plugins/org.jboss.tools.vpe/src/org/jboss/tools/vpe/editor/template/VpeDataTableCreator.java 2008-02-20 12:43:40 UTC (rev 6461) @@ -42,6 +42,11 @@ private VpeExpression footerClassExpr; private VpeExpression rowClassesExpr; private VpeExpression columnClassesExpr; + + private static final String ATTR_STYLE = "style"; + private static final String ATTR_WIDTH = "width"; + private static final String ATTR_BORDER = "border"; + private static final String HIDDEN_BORDER_STYLE = "border: 0px hidden;" ; private List propertyCreators; @@ -206,9 +211,9 @@ // To create appropriate visual appearance // borders of the body cell and content table // were set via styles. - outterTD.setAttribute("style", "border: 0px hidden;"); - visualTable.setAttribute("width", "100%"); - visualTable.setAttribute("style", "border: 0px hidden;"); + outterTD.setAttribute(ATTR_STYLE, HIDDEN_BORDER_STYLE); + visualTable.setAttribute(ATTR_WIDTH, "100%"); + visualTable.setAttribute(ATTR_STYLE, HIDDEN_BORDER_STYLE); outterTD.appendChild(visualTable); outterTR.appendChild(outterTD); @@ -235,8 +240,8 @@ if (null == attr.getNodeValue() || "".equalsIgnoreCase(attr.getNodeValue())) { continue; } - if ("border".equalsIgnoreCase(attr.getNodeName())) { - visualTable.setAttribute("border", attr.getNodeValue()); + if (ATTR_BORDER.equalsIgnoreCase(attr.getNodeName())) { + visualTable.setAttribute(ATTR_BORDER, attr.getNodeValue()); } outterTable.setAttributeNode(attr); }