RED

Issue-ref: 4149
Issue-URL: af3#4149

Signed-off-by: Andreas Bayha <bayha@fortiss.org>

org.fortiss.tooling.ext.reuse.ui/src/org/fortiss/tooling/ext/reuse/ui/commands/UpdateFromLibContextMenu.java

@@ -63,8 +63,9 @@ import org.fortiss.tooling.kernel.ui.extension.data.ContextMenuContextProvider;
 public class UpdateFromLibContextMenu implements IContextMenuContributor {
 
 	/** The title of the dialog shell(s). */
-	private final String shellTitle = "Update Element From Reuse Library";
+	private /*FIXME (AB) static?*/ final String shellTitle = "Update Element From Reuse Library";
 
+	// FIXME(AB) Constants like this should be named with ALL_CAPITAL_LETTERS 
 	/** The string of the context menu entry. */
 	public static final String updateFromContextMenuString = "Update from Reuse Library";
 

org.fortiss.tooling.ext.reuse.ui/src/org/fortiss/tooling/ext/reuse/ui/commands/UpdateInLibContextMenu.java

@@ -64,9 +64,10 @@ import org.fortiss.tooling.kernel.ui.extension.data.ContextMenuContextProvider;
 public class UpdateInLibContextMenu implements IContextMenuContributor {
 
 	/** The title of the dialog shell(s). */
-	private final String shellTitle = "Update Element In Reuse Library";
+	private /*FIXME (AB) static!?*/ final String shellTitle = "Update Element In Reuse Library";
 
 	/** The string of the context menu entry. */
+	// FIXME(AB) Constants like this should be named with ALL_CAPITAL_LETTERS 
 	public static final String updateInContextMenuString = "Update in Reuse Library";
 
 	/** {@inheritDoc} */

org.fortiss.tooling.ext.reuse/src/org/fortiss/tooling/ext/reuse/utils/ReuseLibraryUtils.java

@@ -62,6 +62,8 @@ import org.fortiss.tooling.kernel.model.INamedElement;
 /**
  * Utility methods for {@link ReuseLibrary}s and related reuse elements and
  * actions.
+ * 
+ * FIXME (AB): Decompose. Maybe the UUID stuff could be moved to a new class ReuseIDUtils or similar?
  *
  * @author bergemann
  */
@@ -292,7 +294,7 @@ public class ReuseLibraryUtils {
 	/**
 	 * Returns the UUID of the given reuse element that is stored in the
 	 * {@link ReuseElementSpec} which is connected to the given
-	 * {@link ReuseLibrary}. If such a {@link ReuseElementSpec} odes not exist, an
+	 * {@link ReuseLibrary}. If such a {@link ReuseElementSpec} does not exist, an
 	 * empty {@link String} will be returned.
 	 *
 	 * @param element The target reuse element