Set and check if it's first time opening

Issue: BMS-5074 Reviewer: Clary
IntegratedBreedingPlatform · Aug 29, 2018 · 1a62576 · 1a62576 · nahuel-soldevilla · Aug 30, 2018
1 parent 10f5d05
commit 1a62576
Show file tree

Hide file tree

Showing 4 changed files with 29 additions and 10 deletions.
diff --git a/src/main/java/org/generationcp/ibpworkbench/study/StudyBrowserMain.java b/src/main/java/org/generationcp/ibpworkbench/study/StudyBrowserMain.java
@@ -131,7 +131,7 @@ public void addListeners() {
 			private static final long serialVersionUID = 1L;
 
 			@Override
-			public void buttonClick(ClickEvent event) {
+			public void buttonClick(final ClickEvent event) {
 				StudyBrowserMain.this.openBrowseForStudyWindow();
 			}
 		});
@@ -141,7 +141,7 @@ public void buttonClick(ClickEvent event) {
 			private static final long serialVersionUID = 1L;
 
 			@Override
-			public void buttonClick(ClickEvent event) {
+			public void buttonClick(final ClickEvent event) {
 				StudyBrowserMain.this.openSearchForStudyWindow();
 			}
 		});
@@ -154,7 +154,7 @@ public void layoutComponents() {
 
 		final HeaderLabelLayout headingLayout = new HeaderLabelLayout(StudyBrowserMain.STUDY_DETAILS_ICON, this.headingLabel);
 
-		HorizontalLayout directionLayout = new HorizontalLayout();
+		final HorizontalLayout directionLayout = new HorizontalLayout();
 		directionLayout.addStyleName("study-browser-main");
 		directionLayout.setHeight("16px");
 		directionLayout.setSpacing(true);
@@ -183,6 +183,7 @@ public void updateLabels() {
 	}
 
 	public void openBrowseForStudyWindow() {
+		this.browseTreeComponent.setFirstTimeOpening(true);
 		this.browseTreeComponent.refreshTree();
 		this.launchListSelectionWindow(this.getWindow(), this.browseTreeComponent, this.messageSource.getMessage(Message.BROWSE_STUDIES))
 				.addListener(this.browseTreeComponent.getSaveTreeStateListener());

diff --git a/src/main/java/org/generationcp/ibpworkbench/study/tree/BrowseStudyTreeComponent.java b/src/main/java/org/generationcp/ibpworkbench/study/tree/BrowseStudyTreeComponent.java
@@ -70,6 +70,7 @@ public class BrowseStudyTreeComponent extends VerticalLayout
 	private Map<Integer, Integer> parentChildItemIdMap;
 	private StudyTreeButtonsPanel buttonsPanel;
 	private StudyTypeFilterComponent studyTypeFilterComponent;
+	private boolean isFirstTimeOpening;
 
 	public BrowseStudyTreeComponent(final StudyBrowserMain studyBrowserMain) {
 		this.studyBrowserMain = studyBrowserMain;
@@ -261,7 +262,7 @@ private void buildChildMap(final Integer studyId, final Boolean endNode) {
 	}
 
 	public void updateButtons(final Object itemId) {
-		this.buttonsPanel.updateButtons(itemId);
+		this.buttonsPanel.updateButtons(itemId, this.isFirstTimeOpening);
 
 	}
 
@@ -350,4 +351,11 @@ protected void setButtonsPanel(final StudyTreeButtonsPanel buttonsPanel) {
 		this.buttonsPanel = buttonsPanel;
 	}
 
+	public boolean isFirstTimeOpening() {
+		return this.isFirstTimeOpening;
+	}
+
+	public void setFirstTimeOpening(final boolean firstTimeOpening) {
+		this.isFirstTimeOpening = firstTimeOpening;
+	}
 }
diff --git a/src/main/java/org/generationcp/ibpworkbench/study/tree/StudyTreeButtonsPanel.java b/src/main/java/org/generationcp/ibpworkbench/study/tree/StudyTreeButtonsPanel.java
@@ -175,14 +175,10 @@ public void afterPropertiesSet() throws Exception {
 	public void updateButtons(final Object itemId) {
 		if (itemId instanceof String) {
 			// this means its the ROOT Folder
-			this.addFolderBtn.setEnabled(true);
-			this.renameFolderBtn.setEnabled(false);
-			this.deleteFolderBtn.setEnabled(false);
+			this.setButtonStatus(false);
 
 		} else if (this.studyTree.isFolder((Integer) itemId)) {
-			this.addFolderBtn.setEnabled(true);
-			this.renameFolderBtn.setEnabled(true);
-			this.deleteFolderBtn.setEnabled(true);
+			this.setButtonStatus(true);
 			// The rest of the local lists
 		} else {
 			this.addFolderBtn.setEnabled(true);
@@ -192,6 +188,12 @@ public void updateButtons(final Object itemId) {
 
 	}
 
+	public void setButtonStatus(final boolean b) {
+		this.addFolderBtn.setEnabled(true);
+		this.renameFolderBtn.setEnabled(b);
+		this.deleteFolderBtn.setEnabled(b);
+	}
+
 	protected Button getAddFolderBtn() {
 		return this.addFolderBtn;
 	}
@@ -232,4 +234,11 @@ protected void setStudyTree(final StudyTree studyTree) {
 		this.studyTree = studyTree;
 	}
 
+	public void updateButtons(final Object itemId, final boolean isFirstTimeOpening) {
+		if (isFirstTimeOpening) {
+			this.setButtonStatus(false);
+		} else {
+			this.updateButtons(itemId);
+		}
+	}
 }
diff --git a/...n/java/org/generationcp/ibpworkbench/study/tree/listeners/StudyTreeItemClickListener.java b/...n/java/org/generationcp/ibpworkbench/study/tree/listeners/StudyTreeItemClickListener.java
@@ -28,6 +28,7 @@ public void itemClick(final ItemClickEvent event) {
 	}
 
 	public void studyTreeItemClickAction(final Object itemId) {
+		this.browseStudyTreeComponent.setFirstTimeOpening(false);
 		this.studyTree.expandOrCollapseStudyTreeNode(itemId);
 		if (!StudyTree.STUDY_ROOT_NODE.equals(itemId)) {
 			final int studyId = Integer.valueOf(itemId.toString());