# HG changeset patch # User Tom Gottfried # Date 1464369647 -7200 # Node ID 829092ba09f776d1d1d5bbb1991221c66e081f8f # Parent 091b955c12e74bd68d6b7eda733677fef1cf98fb Improve readability. diff -r 091b955c12e7 -r 829092ba09f7 src/main/java/de/intevation/lada/util/auth/MessungAuthorizer.java --- a/src/main/java/de/intevation/lada/util/auth/MessungAuthorizer.java Fri May 27 16:05:06 2016 +0200 +++ b/src/main/java/de/intevation/lada/util/auth/MessungAuthorizer.java Fri May 27 19:20:47 2016 +0200 @@ -103,8 +103,10 @@ LStatusProtokoll.class, messung.getStatus(), "land"); - messung.setReadonly( - status.getStatusWert() != 0 && status.getStatusWert() != 4); + int stufe = status.getStatusStufe(); + int wert = status.getStatusWert(); + + messung.setReadonly(wert != 0 && wert != 4); boolean statusEdit = false; if (userInfo.getFunktionen().contains(3)) { @@ -116,23 +118,25 @@ repository.filterPlain(lstFilter.getQuery(), "stamm"); for (int i = 0; i < lsts.size(); i++) { if (lsts.get(i).getUmwId().equals(probe.getUmwId()) - && (status.getStatusStufe() == 2 - || status.getStatusStufe() == 3) + && (stufe == 2 || stufe == 3) ) { statusEdit = true; } } } - if (userInfo.getFunktionenForNetzbetreiber(probe.getNetzbetreiberId()).contains(2) && - userInfo.getNetzbetreiber().contains(probe.getNetzbetreiberId()) && - (status.getStatusStufe() == 1 || status.getStatusStufe() == 2) && - status.getStatusWert() >= 1 + if (userInfo.getFunktionenForNetzbetreiber( + probe.getNetzbetreiberId()).contains(2) + && userInfo.getNetzbetreiber().contains( + probe.getNetzbetreiberId()) + && (stufe == 1 || stufe == 2) + && wert >= 1 ) { statusEdit = true; } - if (userInfo.getFunktionenForMst(probe.getMstId()).contains(1) && - userInfo.belongsTo(probe.getMstId(), probe.getLaborMstId()) && - (status.getStatusStufe() <= 1 || status.getStatusWert() == 4) + if (userInfo.getFunktionenForMst(probe.getMstId()).contains(1) + && userInfo.belongsTo(probe.getMstId(), + probe.getLaborMstId()) + && (stufe <= 1 || wert == 4) ) { statusEdit = true; }