X-Git-Url: http://git.argeo.org/?a=blobdiff_plain;f=org.argeo.cms.ui%2Fsrc%2Forg%2Fargeo%2Fcms%2Futil%2FCmsLink.java;h=a395bd23095eb3296336d12b7af22533e3c8ce5a;hb=eb1048ad80b735492cb0c2e153790509112ed4a6;hp=3778515dcc70c56a338ef271db0a69df304b8e5a;hpb=c5fa035468228d1f87ab5431a3fad17403eee1c3;p=lgpl%2Fargeo-commons.git

diff --git a/org.argeo.cms.ui/src/org/argeo/cms/util/CmsLink.java b/org.argeo.cms.ui/src/org/argeo/cms/util/CmsLink.java
index 3778515dc..a395bd230 100644
--- a/org.argeo.cms.ui/src/org/argeo/cms/util/CmsLink.java
+++ b/org.argeo.cms.ui/src/org/argeo/cms/util/CmsLink.java
@@ -22,12 +22,11 @@ import org.eclipse.swt.widgets.Composite;
 import org.eclipse.swt.widgets.Control;
 import org.eclipse.swt.widgets.Label;
 import org.osgi.framework.BundleContext;
-import org.osgi.framework.FrameworkUtil;
 
 /** A link to an internal or external location. */
 public class CmsLink implements CmsUiProvider {
 	private final static Log log = LogFactory.getLog(CmsLink.class);
-	private BundleContext bundleContext ;
+	private BundleContext bundleContext;
 
 	private String label;
 	private String custom;
@@ -68,13 +67,17 @@ public class CmsLink implements CmsUiProvider {
 	/** @return {@link Composite} with a single {@link Label} child. */
 	@Override
 	public Control createUi(final Composite parent, Node context) {
+		if (image != null && (imageWidth == null || imageHeight == null)) {
+			throw new CmsException("Image is not properly configured."
+					+ " Make sure bundleContext property is set and init() method has been called.");
+		}
+
 		Composite comp = new Composite(parent, SWT.BOTTOM);
 		comp.setLayout(CmsUtils.noSpaceGridLayout());
 
 		Label link = new Label(comp, SWT.NONE);
 		link.setData(RWT.MARKUP_ENABLED, Boolean.TRUE);
-		GridData layoutData = new GridData(SWT.CENTER, verticalAlignment, true,
-				true);
+		GridData layoutData = new GridData(SWT.CENTER, verticalAlignment, true, true);
 		if (image != null) {
 			layoutData.heightHint = imageHeight;
 			if (label == null)
@@ -93,8 +96,7 @@ public class CmsLink implements CmsUiProvider {
 		// label
 		StringBuilder labelText = new StringBuilder();
 		if (target != null) {
-			labelText
-					.append("<a style='color:inherit;text-decoration:inherit;' href=\"");
+			labelText.append("<a style='color:inherit;text-decoration:inherit;' href=\"");
 			// if (!isUrl)
 			// labelText.append('#');
 			labelText.append(target);
@@ -103,8 +105,7 @@ public class CmsLink implements CmsUiProvider {
 		if (image != null) {
 			registerImageIfNeeded();
 			String imageLocation = RWT.getResourceManager().getLocation(image);
-			labelText.append("<img width='").append(imageWidth)
-					.append("' height='").append(imageHeight)
+			labelText.append("<img width='").append(imageWidth).append("' height='").append(imageHeight)
 					.append("' src=\"").append(imageLocation).append("\"/>");
 
 			// final Image img = loadImage(parent.getDisplay());
@@ -220,8 +221,7 @@ public class CmsLink implements CmsUiProvider {
 		} else if ("center".equals(vAlign)) {
 			verticalAlignment = SWT.CENTER;
 		} else {
-			throw new CmsException("Unsupported vertical allignment " + vAlign
-					+ " (must be: top, bottom or center)");
+			throw new CmsException("Unsupported vertical allignment " + vAlign + " (must be: top, bottom or center)");
 		}
 	}
 }