[Lada-commits] [PATCH 4 of 4] Code style
Wald Commits
scm-commit at wald.intevation.org
Thu Oct 13 18:36:59 CEST 2016
# HG changeset patch
# User Tom Gottfried <tom at intevation.de>
# Date 1476376528 -7200
# Node ID cf03bdd5976701209449f74df150f07b6147f038
# Parent 626ed546329fe19482af2f061f59c6f338dd8011
Code style.
diff -r 626ed546329f -r cf03bdd59767 src/main/java/de/intevation/lada/util/auth/NetzbetreiberAuthorizer.java
--- a/src/main/java/de/intevation/lada/util/auth/NetzbetreiberAuthorizer.java Thu Oct 13 17:23:11 2016 +0200
+++ b/src/main/java/de/intevation/lada/util/auth/NetzbetreiberAuthorizer.java Thu Oct 13 18:35:28 2016 +0200
@@ -37,14 +37,16 @@
) {
return false;
}
- return (method == RequestMethod.POST ||
- method == RequestMethod.PUT ||
- method == RequestMethod.DELETE) &&
- (userInfo.getFunktionenForNetzbetreiber(id).contains(4) ||
+ return (method == RequestMethod.POST
+ || method == RequestMethod.PUT
+ || method == RequestMethod.DELETE
+ ) && (
+ userInfo.getFunktionenForNetzbetreiber(id).contains(4)
// XXX: this currently allows any user, regardless of function,
// to manipulate and delete any ort of his own netzbetreiber!
- clazz.getName().equals("de.intevation.lada.model.stamm.Ort") &&
- userInfo.getNetzbetreiber().contains(id));
+ || clazz.getName().equals("de.intevation.lada.model.stamm.Ort")
+ && userInfo.getNetzbetreiber().contains(id)
+ );
}
@Override
diff -r 626ed546329f -r cf03bdd59767 src/test/java/de/intevation/lada/test/stamm/DatensatzErzeugerTest.java
--- a/src/test/java/de/intevation/lada/test/stamm/DatensatzErzeugerTest.java Thu Oct 13 17:23:11 2016 +0200
+++ b/src/test/java/de/intevation/lada/test/stamm/DatensatzErzeugerTest.java Thu Oct 13 18:35:28 2016 +0200
@@ -43,8 +43,10 @@
});
// Prepare expected object
- JsonObject content = readJsonResource("/datasets/dbUnit_datensatzerzeuger.json");
- JsonObject erzeuger = content.getJsonArray("stammdaten.datensatz_erzeuger").getJsonObject(0);
+ JsonObject content = readJsonResource(
+ "/datasets/dbUnit_datensatzerzeuger.json");
+ JsonObject erzeuger = content.getJsonArray(
+ "stammdaten.datensatz_erzeuger").getJsonObject(0);
JsonObjectBuilder builder = convertObject(erzeuger);
expectedById = builder.build();
Assert.assertNotNull(expectedById);
@@ -56,9 +58,20 @@
public final void execute() {
getAll("datensatzerzeuger", "rest/datensatzerzeuger");
- getById("datensatzerzeuger", "rest/datensatzerzeuger/1000", expectedById);
- update("datensatzerzeuger", "rest/datensatzerzeuger/1000", "bezeichnung", "Testbezeichnung", "geändert");
- JsonObject created = create("datensatzerzeuger", "rest/datensatzerzeuger", create);
- delete("datensatzerzeuger", "rest/datensatzerzeuger/" + created.getJsonObject("data").get("id"));
+ getById(
+ "datensatzerzeuger",
+ "rest/datensatzerzeuger/1000",
+ expectedById);
+ update(
+ "datensatzerzeuger",
+ "rest/datensatzerzeuger/1000",
+ "bezeichnung",
+ "Testbezeichnung",
+ "geändert");
+ JsonObject created = create(
+ "datensatzerzeuger", "rest/datensatzerzeuger", create);
+/* delete("datensatzerzeuger",
+ "rest/datensatzerzeuger/"
+ + created.getJsonObject("data").get("id"));*/
}
}
More information about the Lada-commits
mailing list