diff options
Diffstat (limited to 'plugins/org.eclipse.osee.display.view.web')
9 files changed, 82 insertions, 5 deletions
diff --git a/plugins/org.eclipse.osee.display.view.web/src/org/eclipse/osee/display/view/web/HasLogger.java b/plugins/org.eclipse.osee.display.view.web/src/org/eclipse/osee/display/view/web/HasLogger.java new file mode 100644 index 00000000000..f1010657ea8 --- /dev/null +++ b/plugins/org.eclipse.osee.display.view.web/src/org/eclipse/osee/display/view/web/HasLogger.java @@ -0,0 +1,24 @@ +/******************************************************************************* + * Copyright (c) 2011 Boeing. + * All rights reserved. This program and the accompanying materials + * are made available under the terms of the Eclipse Public License v1.0 + * which accompanies this distribution, and is available at + * http://www.eclipse.org/legal/epl-v10.html + * + * Contributors: + * Boeing - initial API and implementation + *******************************************************************************/ +package org.eclipse.osee.display.view.web; + +/** + * @author Shawn F. Cook + */ +public interface HasLogger { + public void logError(String format, Object... args); + + public void logWarn(String format, Object... args); + + public void logInfo(String format, Object... args); + + public void logDebug(String format, Object... args); +} diff --git a/plugins/org.eclipse.osee.display.view.web/src/org/eclipse/osee/display/view/web/components/ComponentUtility.java b/plugins/org.eclipse.osee.display.view.web/src/org/eclipse/osee/display/view/web/components/ComponentUtility.java index 6930e785632..3440d67d2dd 100644 --- a/plugins/org.eclipse.osee.display.view.web/src/org/eclipse/osee/display/view/web/components/ComponentUtility.java +++ b/plugins/org.eclipse.osee.display.view.web/src/org/eclipse/osee/display/view/web/components/ComponentUtility.java @@ -12,6 +12,7 @@ package org.eclipse.osee.display.view.web.components; import org.eclipse.osee.display.api.search.SearchNavigator; import org.eclipse.osee.display.api.search.SearchPresenter; +import org.eclipse.osee.display.view.web.HasLogger; import org.eclipse.osee.display.view.web.HasNavigator; import org.eclipse.osee.display.view.web.HasPresenter; import org.eclipse.osee.display.view.web.HasUrl; @@ -86,4 +87,32 @@ public final class ComponentUtility { } return presenter; } + + public static void logError(String format, Component component, Object... args) { + Application app = component.getApplication(); + if (app instanceof HasLogger) { + ((HasLogger) app).logError(format, args); + } + } + + public static void logWarn(String format, Component component, Object... args) { + Application app = component.getApplication(); + if (app instanceof HasLogger) { + ((HasLogger) app).logWarn(format, args); + } + } + + public static void logInfo(String format, Component component, Object... args) { + Application app = component.getApplication(); + if (app instanceof HasLogger) { + ((HasLogger) app).logInfo(format, args); + } + } + + public static void logDebug(String format, Component component, Object... args) { + Application app = component.getApplication(); + if (app instanceof HasLogger) { + ((HasLogger) app).logDebug(format, args); + } + } } diff --git a/plugins/org.eclipse.osee.display.view.web/src/org/eclipse/osee/display/view/web/components/OseeBreadcrumbComponent.java b/plugins/org.eclipse.osee.display.view.web/src/org/eclipse/osee/display/view/web/components/OseeBreadcrumbComponent.java index afa603f2da5..8af6c9b4ae7 100644 --- a/plugins/org.eclipse.osee.display.view.web/src/org/eclipse/osee/display/view/web/components/OseeBreadcrumbComponent.java +++ b/plugins/org.eclipse.osee.display.view.web/src/org/eclipse/osee/display/view/web/components/OseeBreadcrumbComponent.java @@ -51,6 +51,8 @@ public class OseeBreadcrumbComponent extends HorizontalLayout { } } } + } else { + ComponentUtility.logWarn("OseeBreadcrumbComponent.createLayout - WARNING: null value detected.", this); } } diff --git a/plugins/org.eclipse.osee.display.view.web/src/org/eclipse/osee/display/view/web/components/OseeDisplayOptionsComponentImpl.java b/plugins/org.eclipse.osee.display.view.web/src/org/eclipse/osee/display/view/web/components/OseeDisplayOptionsComponentImpl.java index 8dca72a402b..afbe47bf583 100644 --- a/plugins/org.eclipse.osee.display.view.web/src/org/eclipse/osee/display/view/web/components/OseeDisplayOptionsComponentImpl.java +++ b/plugins/org.eclipse.osee.display.view.web/src/org/eclipse/osee/display/view/web/components/OseeDisplayOptionsComponentImpl.java @@ -64,6 +64,9 @@ public class OseeDisplayOptionsComponentImpl extends VerticalLayout implements D if (showVerbose_fromUi != showVerbose_fromPres) { showVerboseCheckBox.setValue(showVerbose_fromPres); } + } else { + ComponentUtility.logWarn("OseeDisplayOptionsComponentImpl.setDisplayOptions - WARNING: null value detected.", + this); } } @@ -76,7 +79,7 @@ public class OseeDisplayOptionsComponentImpl extends VerticalLayout implements D if (presenter != null) { presenter.selectDisplayOptions(url, options, navigator); } else { - System.out.println("Presenter was null"); + ComponentUtility.logWarn("OseeDisplayOptionsComponentImpl.onBoxChecked - WARNING: null value detected.", this); } } diff --git a/plugins/org.eclipse.osee.display.view.web/src/org/eclipse/osee/display/view/web/components/OseeExceptionDialogComponent.java b/plugins/org.eclipse.osee.display.view.web/src/org/eclipse/osee/display/view/web/components/OseeExceptionDialogComponent.java index faa4def4794..802c7f07196 100644 --- a/plugins/org.eclipse.osee.display.view.web/src/org/eclipse/osee/display/view/web/components/OseeExceptionDialogComponent.java +++ b/plugins/org.eclipse.osee.display.view.web/src/org/eclipse/osee/display/view/web/components/OseeExceptionDialogComponent.java @@ -53,6 +53,8 @@ public class OseeExceptionDialogComponent extends Window { msgTypeLabel.setStyleName(CssConstants.OSEE_EXCEPTION_WARNING_TITLE_TEXT); break; default: + ComponentUtility.logWarn("OseeExceptionDialogComponent.[ctor] - WARNING: invalid or unhandled msgType.", + this); break; } @@ -63,6 +65,8 @@ public class OseeExceptionDialogComponent extends Window { mainWindow.addWindow(this); moveToCenter(); focus(); + } else { + ComponentUtility.logWarn("OseeExceptionDialogComponent.[ctor] - WARNING: null value detected.", this); } } diff --git a/plugins/org.eclipse.osee.display.view.web/src/org/eclipse/osee/display/view/web/components/OseeRelationsComponent.java b/plugins/org.eclipse.osee.display.view.web/src/org/eclipse/osee/display/view/web/components/OseeRelationsComponent.java index 091630dd452..b44454c97d8 100644 --- a/plugins/org.eclipse.osee.display.view.web/src/org/eclipse/osee/display/view/web/components/OseeRelationsComponent.java +++ b/plugins/org.eclipse.osee.display.view.web/src/org/eclipse/osee/display/view/web/components/OseeRelationsComponent.java @@ -82,7 +82,9 @@ public class OseeRelationsComponent extends VerticalLayout implements RelationCo presenter.selectRelationType(artifact, relationType, OseeRelationsComponent.this); } } catch (Exception e) { - System.out.println("OseeRelationsComponent.createLayout - CRITICAL ERROR: (WebArtifact) relationsListSelect.getValue() threw an exception."); + ComponentUtility.logError( + "OseeRelationsComponent.createLayout - CRITICAL ERROR: (WebArtifact) relationsListSelect.getValue() threw an exception.", + OseeRelationsComponent.this); } } } @@ -96,7 +98,9 @@ public class OseeRelationsComponent extends VerticalLayout implements RelationCo ViewArtifact artifact = (ViewArtifact) leftSelect.getValue(); handleValue(artifact); } catch (Exception e) { - System.out.println("OseeRelationsComponent.createLayout - CRITICAL ERROR: (WebArtifact) relationsListSelect.getValue() threw an exception."); + ComponentUtility.logError( + "OseeRelationsComponent.createLayout - CRITICAL ERROR: (WebArtifact) relationsListSelect.getValue() threw an exception.", + OseeRelationsComponent.this); } } } @@ -110,7 +114,9 @@ public class OseeRelationsComponent extends VerticalLayout implements RelationCo ViewArtifact artifact = (ViewArtifact) rightSelect.getValue(); handleValue(artifact); } catch (Exception e) { - System.out.println("OseeRelationsComponent.createLayout - CRITICAL ERROR: (WebArtifact) relationsListSelect.getValue() threw an exception."); + ComponentUtility.logError( + "OseeRelationsComponent.createLayout - CRITICAL ERROR: (WebArtifact) relationsListSelect.getValue() threw an exception.", + OseeRelationsComponent.this); } } } @@ -169,6 +175,8 @@ public class OseeRelationsComponent extends VerticalLayout implements RelationCo SearchPresenter<?, ?> presenter = ComponentUtility.getPresenter(OseeRelationsComponent.this); SearchNavigator navigator = ComponentUtility.getNavigator(OseeRelationsComponent.this); presenter.selectArtifact(url, artifact, navigator); + } else { + ComponentUtility.logWarn("OseeRelationsComponent.handleValue - WARNING: null value detected.", this); } } diff --git a/plugins/org.eclipse.osee.display.view.web/src/org/eclipse/osee/display/view/web/components/OseeSearchResultComponent.java b/plugins/org.eclipse.osee.display.view.web/src/org/eclipse/osee/display/view/web/components/OseeSearchResultComponent.java index d60a87e1f53..6c441434cfa 100644 --- a/plugins/org.eclipse.osee.display.view.web/src/org/eclipse/osee/display/view/web/components/OseeSearchResultComponent.java +++ b/plugins/org.eclipse.osee.display.view.web/src/org/eclipse/osee/display/view/web/components/OseeSearchResultComponent.java @@ -110,6 +110,8 @@ public class OseeSearchResultComponent extends VerticalLayout implements SearchR boolean showVerbose = options.getVerboseResults(); vLayout_Matches.setVisible(showVerbose); breadcrumbComp.setVisible(showVerbose); + } else { + ComponentUtility.logWarn("OseeSearchResultComponent.setDisplayOptions - WARNING: null value detected.", this); } } diff --git a/plugins/org.eclipse.osee.display.view.web/src/org/eclipse/osee/display/view/web/components/OseeSearchResultsListComponent.java b/plugins/org.eclipse.osee.display.view.web/src/org/eclipse/osee/display/view/web/components/OseeSearchResultsListComponent.java index b0aaee11563..8518a5ac6a6 100644 --- a/plugins/org.eclipse.osee.display.view.web/src/org/eclipse/osee/display/view/web/components/OseeSearchResultsListComponent.java +++ b/plugins/org.eclipse.osee.display.view.web/src/org/eclipse/osee/display/view/web/components/OseeSearchResultsListComponent.java @@ -185,7 +185,9 @@ public class OseeSearchResultsListComponent extends VerticalLayout implements Se mainLayout.addComponent(searchResultComp, 0); } } catch (IndexOutOfBoundsException e) { - System.out.println("OseeSearchResultsListComponent.updateSearchResultsLayout - CRITICAL ERROR: IndexOutOfBoundsException e"); + ComponentUtility.logError( + "OseeSearchResultsListComponent.updateSearchResultsLayout - CRITICAL ERROR: IndexOutOfBoundsException", + this); } } } diff --git a/plugins/org.eclipse.osee.display.view.web/src/org/eclipse/osee/display/view/web/search/OseeArtifactView.java b/plugins/org.eclipse.osee.display.view.web/src/org/eclipse/osee/display/view/web/search/OseeArtifactView.java index a15f615ecba..cc29bf4d541 100644 --- a/plugins/org.eclipse.osee.display.view.web/src/org/eclipse/osee/display/view/web/search/OseeArtifactView.java +++ b/plugins/org.eclipse.osee.display.view.web/src/org/eclipse/osee/display/view/web/search/OseeArtifactView.java @@ -13,6 +13,7 @@ package org.eclipse.osee.display.view.web.search; import org.eclipse.osee.display.api.components.ArtifactHeaderComponent; import org.eclipse.osee.display.api.data.ViewArtifact; import org.eclipse.osee.display.view.web.AbstractCommonView; +import org.eclipse.osee.display.view.web.components.ComponentUtility; import org.eclipse.osee.display.view.web.components.OseeArtifactNameLinkComponent; import org.eclipse.osee.display.view.web.components.OseeAttributeComponent; import org.eclipse.osee.display.view.web.components.OseeBreadcrumbComponent; @@ -124,6 +125,8 @@ public abstract class OseeArtifactView extends AbstractCommonView implements Art breadcrumbComp.setArtifact(artifact); artifactType.setCaption(String.format("[%s]", artifact.getArtifactType())); artifactName.setArtifact(artifact); + } else { + ComponentUtility.logWarn("OseeArtifactView.updateLayout - WARNING: null value detected", this); } } |