From c29b0df0891a488f6680871e95111727d2381424 Mon Sep 17 00:00:00 2001 From: Bruno Nocera Zanette Date: Mon, 9 Dec 2013 17:35:11 -0200 Subject: [PATCH] Added null verification to avoid NullPointer Added null verification to avoid NullPointer. Signed-off-by: Bruno Nocera Zanette --- .../c3sl/sapos/models/util/PermissionProvider.java | 12 +++++++++--- 1 file changed, 9 insertions(+), 3 deletions(-) diff --git a/src/main/java/br/ufpr/c3sl/sapos/models/util/PermissionProvider.java b/src/main/java/br/ufpr/c3sl/sapos/models/util/PermissionProvider.java index 64d4a9c..d9b9082 100644 --- a/src/main/java/br/ufpr/c3sl/sapos/models/util/PermissionProvider.java +++ b/src/main/java/br/ufpr/c3sl/sapos/models/util/PermissionProvider.java @@ -223,10 +223,16 @@ public class PermissionProvider { SectionPos sp = SectionPos.findSectioPosOfTranscriptId(id); + if ( sp == null + || currentYear == null + || currentPeriod == null + || modificationPeriodIsOpen == null) + return false; + if ( modificationPeriodIsOpen.equals("true") - & sp.getPeriod().equals(currentPeriod) - & sp.getSectionYear().equals(currentYear) - ) return true; + && sp.getPeriod().equals(currentPeriod) + && sp.getSectionYear().equals(currentYear)) + return true; return false; } -- GitLab