Revert "Add feedback to SSH Key Form. gitblit-org#1226"

This reverts commit c3e317a. Was added by mistake here. Should go into own branch to be able to merge with upstream using separate PR
Janix520 · May 21, 2017 · 279a5b6 · 279a5b6
1 parent c3e317a
commit 279a5b6
Show file tree

Hide file tree

Showing 4 changed files with 1 addition and 13 deletions.
diff --git a/src/main/java/com/gitblit/wicket/GitBlitWebApp.properties b/src/main/java/com/gitblit/wicket/GitBlitWebApp.properties
@@ -739,8 +739,6 @@ gb.emailAddressDescription = The primary email address for receiving notificatio
 gb.sshKeys = SSH Keys
 gb.sshKeysDescription = SSH public key authentication is a secure alternative to password authentication
 gb.addSshKey = Add SSH Key
-gb.addSshKeyErrorEmpty = SSH public key empty. Please provide a valid SSH public key
-gb.addSshKeyErrorFormat = Not a valid SSH public key format. Please provide a valid SSH public key
 gb.key = Key
 gb.comment = Comment
 gb.sshKeyCommentDescription = Enter an optional comment. If blank, the comment will be extracted from the key data.

diff --git a/src/main/java/com/gitblit/wicket/GitBlitWebApp_de.properties b/src/main/java/com/gitblit/wicket/GitBlitWebApp_de.properties
@@ -736,8 +736,6 @@ gb.emailAddressDescription = Die prim\u00e4re Emailadresse f\u00fcr den Empfang
 gb.sshKeys = SSH Keys
 gb.sshKeysDescription = SSH Public Key Authentifizierung ist eine sichere Alternative zur Authentifizierung mit Passwort
 gb.addSshKey = SSH Key hinzuf\u00fcgen
-gb.addSshKeyErrorEmpty = SSH Public Key leer. Bitte geben Sie einen g\u00fltigen SSH Public Key an
-gb.addSshKeyErrorFormat = SSH Public Key Format ungültig. Bitte geben Sie einen g\u00fltigen SSH Public Key an
 gb.key = Key
 gb.comment = Kommentar
 gb.sshKeyCommentDescription = Geben Sie optional einen Kommentar ein. Falls Sie dies nicht tun, wird der Kommentar aus dem Key extrahiert.

diff --git a/src/main/java/com/gitblit/wicket/pages/BasePage.java b/src/main/java/com/gitblit/wicket/pages/BasePage.java
@@ -251,7 +251,7 @@ protected void setupPage(String repositoryName, String pageName) {
 		add(rootLink);
 
 		// Feedback panel for info, warning, and non-fatal error messages
-		add(new FeedbackPanel("feedback").setOutputMarkupId(true));
+		add(new FeedbackPanel("feedback"));
 
 		add(new Label("gbVersion", "v" + Constants.getVersion()));
 		if (app().settings().getBoolean(Keys.web.aggressiveHeapManagement, false)) {

diff --git a/src/main/java/com/gitblit/wicket/panels/SshKeysPanel.java b/src/main/java/com/gitblit/wicket/panels/SshKeysPanel.java
@@ -24,7 +24,6 @@
 import org.apache.wicket.ajax.markup.html.form.AjaxButton;
 import org.apache.wicket.markup.html.basic.Label;
 import org.apache.wicket.markup.html.form.Form;
-import org.apache.wicket.markup.html.panel.FeedbackPanel;
 import org.apache.wicket.markup.repeater.Item;
 import org.apache.wicket.markup.repeater.data.DataView;
 import org.apache.wicket.markup.repeater.data.ListDataProvider;
@@ -124,9 +123,6 @@ public void onClick(AjaxRequestTarget target) {
 				"span5",
 				keyComment));
 
-//		final FeedbackPanel feedback = new FeedbackPanel("feedback");
-//		feedback.setOutputMarkupId(true);
-//		addKeyForm.add(feedback);
 		addKeyForm.add(new AjaxButton("addKeyButton") {
 
 			private static final long serialVersionUID = 1L;
@@ -138,8 +134,6 @@ protected void onSubmit(AjaxRequestTarget target, Form<?> form) {
 				String data = keyData.getObject();
 				if (StringUtils.isEmpty(data)) {
 					// do not submit empty key
-					error(getString("gb.addSshKeyErrorEmpty"));
-					target.addComponent(getPage().get("feedback"));
 					return;
 				}
 
@@ -148,8 +142,6 @@ protected void onSubmit(AjaxRequestTarget target, Form<?> form) {
 					key.getPublicKey();
 				} catch (Exception e) {
 					// failed to parse the key
-					error(getString("gb.addSshKeyErrorFormat"));
-					target.addComponent(getPage().get("feedback"));
 					return;
 				}