Skip to content

Commit 0a25b39

Browse files
committed
escape time variable name
1 parent 2bf514b commit 0a25b39

File tree

2 files changed

+5
-2
lines changed

2 files changed

+5
-2
lines changed

post-processing-tool/src/main/java/com/bc/fiduceo/post/plugin/era5/SatelliteFieldsConfiguration.java

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -208,7 +208,7 @@ String get_nwp_time_variable_name() {
208208
}
209209

210210
void set_nwp_time_variable_name(String nwp_time_variable_name) {
211-
this.nwp_time_variable_name = nwp_time_variable_name;
211+
this.nwp_time_variable_name = NetCDFUtils.escapeVariableName(nwp_time_variable_name);
212212
}
213213

214214
String get_time_variable_name() {

post-processing-tool/src/test/java/com/bc/fiduceo/post/plugin/era5/SatelliteFieldsConfigurationTest.java

Lines changed: 4 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -289,7 +289,7 @@ public void testVerify_z_dim_name() {
289289
@Test
290290
public void testVerify_nwp_time_variable_name() {
291291
prepareConfig();
292-
config.set_nwp_time_variable_name(null);
292+
config.set_nwp_time_variable_name("");
293293

294294
try {
295295
config.verify();
@@ -338,6 +338,9 @@ public void testVerify_lat_variable_name() {
338338
public void testSetGet_nwp_time_variable_name() {
339339
config.set_nwp_time_variable_name("tickTock");
340340
assertEquals("tickTock", config.get_nwp_time_variable_name());
341+
342+
config.set_nwp_time_variable_name("tick.Tock");
343+
assertEquals("tick\\.Tock", config.get_nwp_time_variable_name());
341344
}
342345

343346
@Test

0 commit comments

Comments
 (0)