[client] Allow user to define container with Repository Image (directly from hub.docker)

3 files changed, 45 insertions, 11 deletions

diff --git a/dozentenmodul/src/main/java/org/openslx/dozmod/gui/panel/ContainerPanel.java b/dozentenmodul/src/main/java/org/openslx/dozmod/gui/panel/ContainerPanel.java
index 37008e78..028b9878 100644
--- a/dozentenmodul/src/main/java/org/openslx/dozmod/gui/panel/ContainerPanel.java
+++ b/dozentenmodul/src/main/java/org/openslx/dozmod/gui/panel/ContainerPanel.java
@@ -103,12 +103,21 @@ public class ContainerPanel extends JPanel {
 
 		txtContainerImageName.setText(image.imageName);
 		txtContainerImageName.setEnabled(false);
-		// TODO simplify this mess. ContainerBuildContextMethod is to complex
-		if (containerDefinition.getBuildContextMethod() == ContainerBuildContextMethod.FILE) {
+		// TODO simplify this mess. ContainerBuildContextMethod is to complex or useless
+		switch (containerDefinition.getBuildContextMethod())
+		{
+		case FILE:
 			txtContainerRecipe.setText(containerDefinition.getContainerRecipe());
-		} else if (containerDefinition.getBuildContextMethod()
-				== ContainerBuildContextMethod.GIT_REPOSITORY) {
+			break;
+		case IMAGE_REPO:
+			txtContainerRecipe.setText(containerDefinition.getContainerMeta().getImageRepo());
+			break;
+		case GIT_REPOSITORY:
 			txtContainerRecipe.setText(containerDefinition.getContainerMeta().getBuildContextUrl());
+			break;
+		default:
+			LOGGER.error("Unknown Build Context");
+			break;
 		}
 
 		if (context.equals(IMAGE_CONTEXT)) {
@@ -118,12 +127,11 @@ public class ContainerPanel extends JPanel {
 		} else {
 			LOGGER.error("Container Panel init failed: Please use proper context");
 		}
-
 	}
 
 	private void initImageDetails() {
 
-		// currently do not allow user to change the Dockerfile in the suite.
+		// currently do not allow user to change the Image Repository or Dockerfile in the suite.
 		txtContainerRecipe.setEnabled(false);
 		lblContainerRunOpt.setVisible(false);
 		txtContainerRun.setVisible(false);
diff --git a/dozentenmodul/src/main/java/org/openslx/dozmod/gui/wizard/layout/ContainerUploadPageLayout.java b/dozentenmodul/src/main/java/org/openslx/dozmod/gui/wizard/layout/ContainerUploadPageLayout.java
index 5fcc6dcd..0496261f 100644
--- a/dozentenmodul/src/main/java/org/openslx/dozmod/gui/wizard/layout/ContainerUploadPageLayout.java
+++ b/dozentenmodul/src/main/java/org/openslx/dozmod/gui/wizard/layout/ContainerUploadPageLayout.java
@@ -24,6 +24,8 @@ public class ContainerUploadPageLayout extends WizardPage {
 	protected final QLabel lblImageName;
 	protected final JTextArea txtInfoText;
 
+	protected final JTextField txtImageRepo;
+
 	protected final JTabbedPane tpInput;
 	protected final JTextField txtGitRepo;
 
@@ -38,8 +40,24 @@ public class ContainerUploadPageLayout extends WizardPage {
 		setDescription(I18n.PAGE_LAYOUT.getString("ContainerUploadPage.description"));
 		GridManager grid = new GridManager(this, 3, false);
 
+		JPanel imageRepoPanel = new JPanel();
+		imageRepoPanel.setVisible(true);
+		GridManager tmpGrid = new GridManager(imageRepoPanel, 2, true, new Insets(5, 0, 5, 0));
+		QLabel lblImageRepo = new QLabel(
+				I18n.PAGE_LAYOUT.getString("ContainerUploadPage.DockerFile.label"));
+		lblImageRepo.setToolTipText(
+				I18n.PAGE_LAYOUT.getString("ContainerUploadPage.ImageRepository.ToolTipText"));
+		txtImageRepo = new JTextField();
+		txtImageRepo.setEditable(true);
+		txtImageRepo.setToolTipText(
+				I18n.PAGE_LAYOUT.getString("ContainerUploadPage.ImageRepository.ToolTipText"));
+		tmpGrid.add(lblImageRepo);
+		tmpGrid.add(txtImageRepo).fill(true, false).expand(true, false);
+		tmpGrid.finish(false);
+
+
 		JPanel p1 = new JPanel();
-		p1.setVisible(true);
+		p1.setVisible(false);
 		GridManager g1 = new GridManager(p1, 3, true, new Insets(5, 0, 5, 0));
 		QLabel imageFileCaption = new QLabel(
 				I18n.PAGE_LAYOUT.getString("ContainerUploadPage.DockerFile.label"));
@@ -68,13 +86,12 @@ public class ContainerUploadPageLayout extends WizardPage {
 		tpInput = new JTabbedPane();
 		tpInput.addTab("Dockerfile", p1);
 		tpInput.addTab("Git Repository", p2);
-		tpInput.setSelectedIndex(ContainerBuildContextMethod.FILE.ordinal());
+		tpInput.addTab("Image Repository", imageRepoPanel);
+		// set "Image Repository" as selected
+		tpInput.setSelectedIndex(2);
 
 		grid.add(tpInput, 3).fill(true, false);
 
-		// Start as with Dockerfile as input!
-		tpInput.setSelectedIndex(0);
-
 		lblImageName = new QLabel(I18n.PANEL.getString("ContainerPanel.Label.ImageName.text"));
 		txtImageName = new JTextField();
 		grid.add(lblImageName);
diff --git a/dozentenmodul/src/main/java/org/openslx/dozmod/gui/wizard/page/ContainerUploadPage.java b/dozentenmodul/src/main/java/org/openslx/dozmod/gui/wizard/page/ContainerUploadPage.java
index b16a9785..7ff19882 100644
--- a/dozentenmodul/src/main/java/org/openslx/dozmod/gui/wizard/page/ContainerUploadPage.java
+++ b/dozentenmodul/src/main/java/org/openslx/dozmod/gui/wizard/page/ContainerUploadPage.java
@@ -232,6 +232,12 @@ public class ContainerUploadPage extends ContainerUploadPageLayout {
 		ContainerBuildContextMethod method = getBuildContextMethod();
 
 		switch (method) {
+		case IMAGE_REPO:
+			if (txtImageRepo.getText() == null || txtImageRepo.getText().isEmpty()) {
+				setWarningMessage(I18n.PAGE.getString("ContainerUploadPage.Warning.NoImageRepo"));
+				return false;
+			}
+			break;
 		case FILE:
 			if (txtImageFile.getText() == null || txtImageFile.getText().isEmpty()) {
 				setWarningMessage(I18n.PAGE.getString("ContainerUploadPage.Warning.NoReceipt"));
@@ -270,6 +276,9 @@ public class ContainerUploadPage extends ContainerUploadPageLayout {
 		case FILE:
 			containerDefinition.setContainerRecipe(state.descriptionFile);
 			break;
+		case IMAGE_REPO:
+			containerMeta.setImageRepo(txtImageRepo.getText());
+			state.descriptionFile = getDummyFile();
 		case GIT_REPOSITORY:
 			containerMeta.setBuildContextUrl(txtGitRepo.getText());
 			state.descriptionFile = getDummyFile();