Skip to content

Commit ee549c6

Browse files
authored
Fixes #632 and #634 - update existing/add missing localized messages (#636)
1 parent 2f983b7 commit ee549c6

File tree

3 files changed

+27
-7
lines changed

3 files changed

+27
-7
lines changed

plugins/com.google.cloud.tools.eclipse.appengine.deploy.ui/src/com/google/cloud/tools/eclipse/appengine/deploy/ui/messages.properties

Lines changed: 7 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -5,11 +5,16 @@ google.preferences=Google Cloud Platform-related settings
55
auto.promote=Promote the deployed version to receive all traffic
66
bucket.name=Bucket:
77
cannot.open.browser=Cannot open browser: {0}
8+
deploy.login.dialog.message=Browser opened to authorize deployment.
89
deploy.manual.link=To manually promote a version use the <a href="{0}">Google Cloud Console.</a>
910
deploy.preferences.save.error.title=Could not save preferences
1011
deploy.preferences.save.error.message=Error while saving the preferences: {0}
11-
deploy.prompt.projectid=Always prompt for project ID on deploy
12-
deploy.prompt.projectid.long=During deploy you will have the opportunity to provide another value for project ID that will be used instead of the project ID set on this page.
12+
deploy.prompt.projectid=Prompt for project ID on deploy
13+
deploy.prompt.projectid.long=It will override the project ID set on this page.
14+
dialog.prompt.projectId.title=Enter project ID
15+
dialog.prompt.projectId.message=Project ID to use for deployment
16+
error.projectId.missing=Project ID must be set for deploy.
17+
error.projectId.prefix=Error: {0}
1318
project.id=Project ID:
1419
project.version=Version:
1520
settings.advanced=Advanced

plugins/com.google.cloud.tools.eclipse.appengine.deploy.ui/src/com/google/cloud/tools/eclipse/appengine/deploy/ui/standard/StandardDeployCommandHandler.java

Lines changed: 19 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -39,13 +39,13 @@
3939
import com.google.cloud.tools.appengine.api.deploy.DefaultDeployConfiguration;
4040
import com.google.cloud.tools.eclipse.appengine.deploy.AppEngineProjectDeployer;
4141
import com.google.cloud.tools.eclipse.appengine.deploy.CleanupOldDeploysJob;
42-
import com.google.cloud.tools.eclipse.appengine.deploy.Messages;
4342
import com.google.cloud.tools.eclipse.appengine.deploy.standard.ExplodedWarPublisher;
4443
import com.google.cloud.tools.eclipse.appengine.deploy.standard.StandardDeployJob;
4544
import com.google.cloud.tools.eclipse.appengine.deploy.standard.StandardDeployJobConfig;
4645
import com.google.cloud.tools.eclipse.appengine.deploy.standard.StandardDeployPreferences;
4746
import com.google.cloud.tools.eclipse.appengine.deploy.standard.StandardDeployPreferencesConverter;
4847
import com.google.cloud.tools.eclipse.appengine.deploy.standard.StandardProjectStaging;
48+
import com.google.cloud.tools.eclipse.appengine.deploy.ui.Messages;
4949
import com.google.cloud.tools.eclipse.appengine.login.IGoogleLoginService;
5050
import com.google.cloud.tools.eclipse.sdk.ui.MessageConsoleWriterOutputLineListener;
5151
import com.google.cloud.tools.eclipse.ui.util.MessageConsoleUtilities;
@@ -156,7 +156,24 @@ private String promptForProjectId(Shell shell, String initialValue) throws Deplo
156156
Messages.getString("dialog.prompt.projectId.title"),
157157
Messages.getString("dialog.prompt.projectId.message"),
158158
initialValue,
159-
new ProjectIdValidator());
159+
new ProjectIdValidator() {
160+
@Override
161+
// The input dialog does not provide decoration for the error message,
162+
// make errors more recognizable with modified validation messages
163+
// FIXME https://github.com/GoogleCloudPlatform/cloud-tools-for-eclipse/issues/635
164+
public String isValid(String newText) {
165+
String error = super.isValid(newText);
166+
if (error != null) {
167+
if (error.isEmpty()) {
168+
return Messages.getString("error.projectId.invalid");
169+
} else {
170+
return Messages.getString("error.projectId.prefix", error);
171+
}
172+
} else {
173+
return null;
174+
}
175+
}
176+
});
160177
int result = dialog.open();
161178
if (result == Window.OK) {
162179
return dialog.getValue();

plugins/com.google.cloud.tools.eclipse.appengine.deploy/src/com/google/cloud/tools/eclipse/appengine/deploy/messages.properties

Lines changed: 1 addition & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -2,7 +2,6 @@ missing.appengine.xml.element=Cannot read element in appengine-web.xml:
22
cannot.parse.appengine.xml=Cannot parse appengine-web.xml
33
cleanup.deploy.job.name=Delete old deploy work directories
44
cleanup.deploy.job.error=Cannot delete old deploy work directories
5-
error.projectId.missing=Project Id must be set for deploy.
65
task.name.deploy.project=Deploying project
76
task.name.stage.project=Staging exploded WAR
87
task.name.publish.war=Publish exploded WAR
@@ -11,5 +10,4 @@ deploy.standard.runnable.name=Deploying to App Engine Standard
1110
deploy.job.stagingdir.create.failed=Cannot create staging directory.
1211
deploy.job.sourcedir.missing=Staging source directory does not exist.
1312
deploy.failed.error.message=Deploy failed.
14-
save.credential.failed=Saving credential to file failed.
15-
deploy.login.dialog.message=Browser opened to authorize deployment.
13+
save.credential.failed=Saving credential to file failed.

0 commit comments

Comments
 (0)