From 43c2e13dc3c16eeb729ef9a26bc2ef40a47f7384 Mon Sep 17 00:00:00 2001
From: Fredrik Norberg <fnorbe@dsv.su.se>
Date: Wed, 27 Jul 2011 17:20:13 +0200
Subject: [PATCH 1/6] added the 4th option to traffic lights

---
 .../checklists/panels/TrafficLightPanel.java     |   4 ++++
 .../se/su/dsv/scipro/data/enums/StateOfMind.java |   3 ++-
 src/main/webapp/images/icons/grey_ball_32.png    | Bin 0 -> 1081 bytes
 3 files changed, 6 insertions(+), 1 deletion(-)
 create mode 100644 src/main/webapp/images/icons/grey_ball_32.png

diff --git a/src/main/java/se/su/dsv/scipro/checklists/panels/TrafficLightPanel.java b/src/main/java/se/su/dsv/scipro/checklists/panels/TrafficLightPanel.java
index 1aa44102c5..51082c7eb6 100644
--- a/src/main/java/se/su/dsv/scipro/checklists/panels/TrafficLightPanel.java
+++ b/src/main/java/se/su/dsv/scipro/checklists/panels/TrafficLightPanel.java
@@ -64,6 +64,10 @@ public class TrafficLightPanel extends Panel {
 					image = new StaticImage("image", new Model<String>(
 					"images/icons/yellow_ball_32.png"));
 				}
+				else if (stateOfMind.equals(StateOfMind.NOTAPLICABLE)) {
+					image = new StaticImage("image", new Model<String>(
+					"images/icons/grey_ball_32.png"));
+				}
 				item.add(image);
 			};
 		};
diff --git a/src/main/java/se/su/dsv/scipro/data/enums/StateOfMind.java b/src/main/java/se/su/dsv/scipro/data/enums/StateOfMind.java
index 85f0570407..67842eaf5b 100644
--- a/src/main/java/se/su/dsv/scipro/data/enums/StateOfMind.java
+++ b/src/main/java/se/su/dsv/scipro/data/enums/StateOfMind.java
@@ -10,7 +10,8 @@ package se.su.dsv.scipro.data.enums;
 public enum StateOfMind {
     NEEDHELP(0, "Need Help", "You think you need help to proceed or you are unsure of what to do next.", "som-needhelp"), 
     NEUTRAL(1, "Neutral", "This is the default option and signals that you don't have a clear standpoint on your status but no imminent help is needed.", "som-neutral"), 
-    FINE(2, "Everything is fine", "This means your work proceeds as planned and that no issues regarding the continuation are on the horizon.", "som-fine");
+    FINE(2, "Everything is fine", "This means your work proceeds as planned and that no issues regarding the continuation are on the horizon.", "som-fine"),
+    NOTAPLICABLE(3, "moo", "moo","som-moo");
 
     private int value;
     private String status;
diff --git a/src/main/webapp/images/icons/grey_ball_32.png b/src/main/webapp/images/icons/grey_ball_32.png
new file mode 100644
index 0000000000000000000000000000000000000000..a9017f0aa57e508ee1854248ce8ab4877107c403
GIT binary patch
literal 1081
zcmV-91jhS`P)<h;3K|Lk000e1NJLTq001BW001Be1^@s6b9#F800001b5ch_0Itp)
z=>Px#24YJ`L;(K){{a7>y{D4^000SaNLh0L01FcU01FcV0GgZ_00007bV*G`2ipf5
z4+;b6ue8qq00X&6L_t(o!_`;KYZOTo|8-4wM<<h=RmVvrhzHq=jE;<{5xj~2f=BTb
z7WV2vSlGKDd*6#kkNyvWcuI$`lMG%K7D9~UXf&u3GwH6b_E7b@s%vKQrJy!E>Z4!1
zkKe2Js>;LfIQF-&cJ>YcEMI>6__#8>!y2%#TECst1`j)1jSEOZhS*^U0R*vZ*1GG#
zR^|Tkl{Zhmi2=M@Ta3i~KiHk|kL23PPlEfKe?*O~N5?XNubityIMqXPx&o;u83R0K
zTKmf+Af;&mfB^{Jf8^Mnk6OLKp$%wvW*bZ1g${N?mMbp#hDbTnEh)?-kP1UIKOM_<
zw%%N7_PeewHb(jNC0ty~m6v=VJfX-B6$PQ0BML(EpXN$BYC9oY!o{_AXZHHO2=JA2
zmB>H;p`e@u+X&d9AOr|{8vzjmNv!%B2`)COb~p+fE0QxW$110mLg6vvoxKNAVR(d2
z6ORG$Q2pBpSrp8?1bV-Ygn)Nzi%9F$5#<EE3%flI9O&(Fpx2VYX}!w3wZ+j8{M$;f
zTlT?NQK{V?B?JV&-Nr-3bMmvm)s-MeU}3d>8@pvc_nv5+^25@kwE(bN_6Y?B0C8>b
zcOl&<#%}t#!fAXIBQ<Kn`rtus!0J}xe7<20E5zoda7^O0ozgXA?RE2v$$~wr;E*tT
zp*?oz0l?FH078k(B#>D~royz00V#mJzykorJF_<-U{Z!!&ndfK+lQk=0K7AM(}yn`
z#oB0jObhHWV`H*F2Qy<pD%E_(iY42X3jn@sFns6FhhW76LP|sSDd$O`1k-2QbDM|O
zNb6*DWfBhn2-am$v;Q<N$eE=ylf@(UYx{6?_(0U`KUJ!yVhItlLPevhR`QM4sov!f
zD!$o-N@ycM<5U#y3R6jz*MTdw?ndE2anE&^Y~i-xNxZg9jlku(&b!=z@ZI!ZxhLZm
zyJR1;sW7#EKck&X%856XKYyIV2mpAnaZl}RIqDIOMLa6lRbunF#z>VtJlME5{!X}Z
z=>zso|5CV$(V=5sNbGx|dm$m1$pPCy0GN9J5v>>1v3Ej_?RiYcDoCkNki|o_m|E(`
zL*uQ@XY&#dDW8rd$M*chOKH^V4cgiI4U~1NxbTyrgGe|b0)VnaJ6pfuUe8OoRzz}h
zZfBGC<7cme>RlwvQ=2Q*>60nS^F7ruPE=nCAWe(`GS$jO#_|8HsM&vhl!wV&gj4^0
zgNMn0WBa$L)qDMOJ<WcY$LT@n0Kh?B?)QBG3P9~LWaMsd00000NkvXXu0mjf3Bv$?

literal 0
HcmV?d00001


From 532b93f008d8bcaff78339d86b282fb8e219ef15 Mon Sep 17 00:00:00 2001
From: mpeters <mpeters@dsv.su.se>
Date: Thu, 28 Jul 2011 15:45:11 +0200
Subject: [PATCH 2/6] Fix of hiding of "Last 12 months" on peer portal page
 depending on what's currently visible

Change-Id: I6b0b25817373bacc2af8e3413fc432955b7d9f4a
---
 .../java/se/su/dsv/scipro/peer/panels/PeerPortalPanel.html | 2 +-
 .../java/se/su/dsv/scipro/peer/panels/PeerPortalPanel.java | 7 +++++++
 2 files changed, 8 insertions(+), 1 deletion(-)

diff --git a/src/main/java/se/su/dsv/scipro/peer/panels/PeerPortalPanel.html b/src/main/java/se/su/dsv/scipro/peer/panels/PeerPortalPanel.html
index 721f5125e9..21fac7dd7d 100644
--- a/src/main/java/se/su/dsv/scipro/peer/panels/PeerPortalPanel.html
+++ b/src/main/java/se/su/dsv/scipro/peer/panels/PeerPortalPanel.html
@@ -89,7 +89,7 @@
 			<span class="box-title">Best rated reviewers</span>
 			<div wicket:id="bestRatedPanel" class="append-bottom"></div>
 		</div>
-		<div>
+		<div wicket:id="statsTimespanContainer">
 			<i><span class="small right">Last 12 months</span></i>
 		</div>
 	</div>
diff --git a/src/main/java/se/su/dsv/scipro/peer/panels/PeerPortalPanel.java b/src/main/java/se/su/dsv/scipro/peer/panels/PeerPortalPanel.java
index 3d70815e37..d4db753c68 100644
--- a/src/main/java/se/su/dsv/scipro/peer/panels/PeerPortalPanel.java
+++ b/src/main/java/se/su/dsv/scipro/peer/panels/PeerPortalPanel.java
@@ -221,16 +221,23 @@ public class PeerPortalPanel extends Panel {
 		int displayedReviews = gsettings.getNumberOfLatestReviewsDisplayed();
 		
 		
+		boolean showStatsTimespan = false;
 		if(gsettings.isPeerDisplayNumberOfReviewsPerformed()){
 			add(new MostFrequentReviewersPanel(mostFrequentPanel));
+			showStatsTimespan = true;
 		} else {
 			add(new InvisiblePanel(mostFrequentPanel));
 		}
 		if( gsettings.isPeerRatingsEnabled() ){
 			add(new WebMarkupContainer(bestRatedContainer).add(new BestRatedReviewersPanel("bestRatedPanel") ) );
+			showStatsTimespan = true;
 		} else {
 			add(new InvisiblePanel(bestRatedContainer));
 		}
+		WebMarkupContainer statsTimespanContainer = new WebMarkupContainer("statsTimespanContainer");
+		statsTimespanContainer.setVisible(showStatsTimespan);
+		add(statsTimespanContainer);
+		
 		if(gsettings.isPeerDisplayLatestReviews()) {
 			// Checkbox on the AdminPeerSettingsPage is not visible since it's
 			// not decided what kind of info that should be public for students.

From 44c9b82b5aab7ab8c3058a0175574294e6d98080 Mon Sep 17 00:00:00 2001
From: Fredrik Norberg <fnorbe@dsv.su.se>
Date: Thu, 28 Jul 2011 15:53:56 +0200
Subject: [PATCH 3/6] changed trafficlightpanel to not use stateofmind

---
 .../admin/pages/AdminProjectPartnerPage.java  |  2 +-
 .../checklists/panels/TrafficLightPanel.html  | 45 ++++++++++++++----
 .../checklists/panels/TrafficLightPanel.java  | 46 +++++++++++--------
 .../su/dsv/scipro/data/enums/StateOfMind.java |  3 +-
 .../dsv/scipro/project/pages/ProjectPage.java |  1 +
 src/main/webapp/WEB-INF/web.xml               |  4 +-
 6 files changed, 68 insertions(+), 33 deletions(-)

diff --git a/src/main/java/se/su/dsv/scipro/admin/pages/AdminProjectPartnerPage.java b/src/main/java/se/su/dsv/scipro/admin/pages/AdminProjectPartnerPage.java
index cc47dc9a00..9913aa204e 100644
--- a/src/main/java/se/su/dsv/scipro/admin/pages/AdminProjectPartnerPage.java
+++ b/src/main/java/se/su/dsv/scipro/admin/pages/AdminProjectPartnerPage.java
@@ -19,7 +19,7 @@ import se.su.dsv.scipro.security.auth.roles.Roles;
  *
  */
 
-@Authorization(authorizedRoles={Roles.SYSADMIN})
+@Authorization(authorizedRoles={Roles.SYSADMIN, Roles.ADMIN})
 public class AdminProjectPartnerPage extends AbstractAdminPage {
 	
 	@SpringBean
diff --git a/src/main/java/se/su/dsv/scipro/checklists/panels/TrafficLightPanel.html b/src/main/java/se/su/dsv/scipro/checklists/panels/TrafficLightPanel.html
index a235198823..5bc9e200e2 100644
--- a/src/main/java/se/su/dsv/scipro/checklists/panels/TrafficLightPanel.html
+++ b/src/main/java/se/su/dsv/scipro/checklists/panels/TrafficLightPanel.html
@@ -6,16 +6,41 @@
 </head>
 <body>
 	<wicket:panel>
-		<div wicket:id="stateOfMindContainer">
-			<form wicket:id="form">
-				<table wicket:id="group">
-					<tr wicket:id="persons">
+		<div wicket:id="trafficLightContainer">
+			<div wicket:id="group">
+				<div wicket:id="lights">
+				<table >
+					<tr >
 						<td><input type="radio" wicket:id="radio" /></td>
-						<td> <img wicket:id="image"/></td>
-					</tr> 
-				</table>
-			</form>
-		</div>
+						<td><img wicket:id="image"/></td>
+						</tr> 
+						<tr>
+							
+						</tr>
+					</table>
+					</div>
+				</div>
+			</div>
 	</wicket:panel>
 </body>
-</html>
\ No newline at end of file
+</html>
+<!--
+<!DOCTYPE html>
+<html
+	xmlns:wicket="http://wicket.apache.org/dtds.data/wicket-xhtml1.4-strict.dtd">
+<head>
+<meta http-equiv="Content-type" content="text/html;charset=UTF-8" />
+</head>
+<body>
+	<wicket:panel>
+		<div wicket:id="trafficLightContainer">
+			<table wicket:id="group">
+				<tr wicket:id="lights">
+					<td><input type="radio" wicket:id="radio" /></td>
+					<td> <img wicket:id="image"/></td>
+					</tr> 
+				</table>
+			</div>
+	</wicket:panel>
+</body>
+</html> -->
\ No newline at end of file
diff --git a/src/main/java/se/su/dsv/scipro/checklists/panels/TrafficLightPanel.java b/src/main/java/se/su/dsv/scipro/checklists/panels/TrafficLightPanel.java
index 51082c7eb6..35bc079230 100644
--- a/src/main/java/se/su/dsv/scipro/checklists/panels/TrafficLightPanel.java
+++ b/src/main/java/se/su/dsv/scipro/checklists/panels/TrafficLightPanel.java
@@ -1,5 +1,8 @@
 package se.su.dsv.scipro.checklists.panels;
 
+import java.util.ArrayList;
+import java.util.List;
+
 import org.apache.wicket.AttributeModifier;
 import org.apache.wicket.ajax.AbstractDefaultAjaxBehavior;
 import org.apache.wicket.ajax.AjaxRequestTarget;
@@ -32,46 +35,53 @@ import edu.emory.mathcs.backport.java.util.Arrays;
  */
 
 public class TrafficLightPanel extends Panel {
-
+	
+	private List<TrafficLight> lights;
+	
+	enum TrafficLight{
+		GREEN, YELLOW, RED, NOT
+	}
 
 	private static final long serialVersionUID = 8862892008428526067L;
 
 
 	public TrafficLightPanel(String id, WebPage parent) {
 		super(id);
-		WebMarkupContainer stateOfMindContainer = new WebMarkupContainer(
-				"stateOfMindContainer");
-		add(stateOfMindContainer);
+		WebMarkupContainer trafficLightContainer = new WebMarkupContainer(
+				"trafficLightContainer");
+		add(trafficLightContainer);
+		lights = new ArrayList<TrafficLight>();
+		lights.add(TrafficLight.GREEN);lights.add(TrafficLight.YELLOW);lights.add(TrafficLight.RED);lights.add(TrafficLight.NOT);
+		RadioGroup<TrafficLight> group = new RadioGroup<TrafficLight>("group");
+		trafficLightContainer.add(group);
+		
 
-		Form<StateOfMind> f = new Form<StateOfMind>("form");
-		stateOfMindContainer.add(f);
-		RadioGroup<StateOfMind> group = new RadioGroup<StateOfMind>("group");
-		f.add(group);
+		ListView<TrafficLight> lightList = new ListView<TrafficLight>("lights", lights) {
+	
+			private static final long serialVersionUID = 1L;
 
-		ListView<StateOfMind> persons = new ListView<StateOfMind>("persons",
-				Arrays.asList(StateOfMind.values())) {
-			protected void populateItem(ListItem<StateOfMind> item) {
-				StateOfMind stateOfMind = item.getModelObject();
-				item.add(new Radio<StateOfMind>("radio", item.getModel()));
+			protected void populateItem(ListItem<TrafficLight> item) {
+				TrafficLight trafficLight = item.getModelObject();
+				item.add(new Radio<TrafficLight>("radio", item.getModel()));
 				StaticImage image = null;
-				if (stateOfMind.equals(StateOfMind.FINE)) {
+				if (trafficLight.equals(TrafficLight.GREEN)) {
 					image = new StaticImage("image", new Model<String>(
 							"images/icons/green_ball_32.png"));
-				} else if (stateOfMind.equals(StateOfMind.NEEDHELP)) {
+				} else if (trafficLight.equals(TrafficLight.YELLOW)) {
 					image = new StaticImage("image", new Model<String>(
 					"images/icons/red_ball_32.png"));
-				} else if (stateOfMind.equals(StateOfMind.NEUTRAL)) {
+				} else if (trafficLight.equals(TrafficLight.RED)) {
 					image = new StaticImage("image", new Model<String>(
 					"images/icons/yellow_ball_32.png"));
 				}
-				else if (stateOfMind.equals(StateOfMind.NOTAPLICABLE)) {
+				else if (trafficLight.equals(TrafficLight.NOT)) {
 					image = new StaticImage("image", new Model<String>(
 					"images/icons/grey_ball_32.png"));
 				}
 				item.add(image);
 			};
 		};
-		group.add(persons);
+		group.add(lightList);
 	}
 
 }
diff --git a/src/main/java/se/su/dsv/scipro/data/enums/StateOfMind.java b/src/main/java/se/su/dsv/scipro/data/enums/StateOfMind.java
index 67842eaf5b..85f0570407 100644
--- a/src/main/java/se/su/dsv/scipro/data/enums/StateOfMind.java
+++ b/src/main/java/se/su/dsv/scipro/data/enums/StateOfMind.java
@@ -10,8 +10,7 @@ package se.su.dsv.scipro.data.enums;
 public enum StateOfMind {
     NEEDHELP(0, "Need Help", "You think you need help to proceed or you are unsure of what to do next.", "som-needhelp"), 
     NEUTRAL(1, "Neutral", "This is the default option and signals that you don't have a clear standpoint on your status but no imminent help is needed.", "som-neutral"), 
-    FINE(2, "Everything is fine", "This means your work proceeds as planned and that no issues regarding the continuation are on the horizon.", "som-fine"),
-    NOTAPLICABLE(3, "moo", "moo","som-moo");
+    FINE(2, "Everything is fine", "This means your work proceeds as planned and that no issues regarding the continuation are on the horizon.", "som-fine");
 
     private int value;
     private String status;
diff --git a/src/main/java/se/su/dsv/scipro/project/pages/ProjectPage.java b/src/main/java/se/su/dsv/scipro/project/pages/ProjectPage.java
index 9dacb4dea1..68ee7374f8 100644
--- a/src/main/java/se/su/dsv/scipro/project/pages/ProjectPage.java
+++ b/src/main/java/se/su/dsv/scipro/project/pages/ProjectPage.java
@@ -27,6 +27,7 @@ public abstract class ProjectPage extends MenuPage implements FilePanelContainer
 	public ProjectPage(PageParameters pp)  {
 		
 		Project activeProject = SciProSession.get().getActiveProject();
+		User u = SciProSession.get().getUser();
 		
 		if(activeProject == null && (this.getClass() != NoActiveProjectPage.class && this.getClass() != ProjectPartnerPage.class)){
 			User user = SciProSession.get().getUser();
diff --git a/src/main/webapp/WEB-INF/web.xml b/src/main/webapp/WEB-INF/web.xml
index 7ea6274a27..54e9218c03 100644
--- a/src/main/webapp/WEB-INF/web.xml
+++ b/src/main/webapp/WEB-INF/web.xml
@@ -64,8 +64,8 @@
 		<!-- Use deployment for production, development for development -->
 	<context-param>
 		<param-name>configuration</param-name>
-<!-- 			<param-value>development</param-value> -->
-			<param-value>deployment</param-value>
+			<param-value>development</param-value> 
+			<!-- <param-value>deployment</param-value>-->
 	</context-param>
 	
 

From 8506215602c5295bd58fcc1ea205f4fb6aaebdc2 Mon Sep 17 00:00:00 2001
From: mpeters <mpeters@dsv.su.se>
Date: Thu, 28 Jul 2011 16:07:37 +0200
Subject: [PATCH 4/6] Spelling+comments

Change-Id: I51065466031338b116ab185722915bae1c45c5cc
---
 .../scipro/components/AbstractMenuPanel.java  | 19 ++++++++++---------
 1 file changed, 10 insertions(+), 9 deletions(-)

diff --git a/src/main/java/se/su/dsv/scipro/components/AbstractMenuPanel.java b/src/main/java/se/su/dsv/scipro/components/AbstractMenuPanel.java
index aa60f5e574..75f046c4e0 100644
--- a/src/main/java/se/su/dsv/scipro/components/AbstractMenuPanel.java
+++ b/src/main/java/se/su/dsv/scipro/components/AbstractMenuPanel.java
@@ -56,7 +56,7 @@ public abstract class AbstractMenuPanel extends Panel {
 			protected void populateItem( ListItem<MenuItem> item ) {
 				final MenuItem mi = item.getModelObject();
 				boolean highLightAsCurrentByInterface = false;
-				Class<?> menuItemInterface = mi.getMenuHilightInterface();
+				Class<?> menuItemInterface = mi.getMenuHighlightInterface();
 				if( menuItemInterface != null ){
 					for( Class<?> actualPageInterface : containerClass.getInterfaces() ){
 						if( actualPageInterface.equals(menuItemInterface) ){
@@ -72,7 +72,7 @@ public abstract class AbstractMenuPanel extends Panel {
 						&& 
 							mi.getTargetClass().getSuperclass() != menuContainerCommonSuperClass
 						)				
-					){
+				){
 					//item.setMarkupId("current"); 
 					//item.setOutputMarkupId(true);
 					//Replaced the above because it generates invalid markup if two menus are added to the same page (id no longer unique)
@@ -99,7 +99,8 @@ public abstract class AbstractMenuPanel extends Panel {
 	}
 	/**
 	 * May not rely on anything in the implementing class's constructor as this will not have been initialized yet.
-	 * Items are rendered from left to right as you might expect, first item in the list will be first tab item on the left.
+	 * Items are rendered from left to right (or top to bottom depending on type of menu) as you might expect,
+	 * the first item in the list will be the first menu item on the left/top.
 	 * @return
 	 */
 	protected abstract List<MenuItem> getItemList();
@@ -118,7 +119,7 @@ public abstract class AbstractMenuPanel extends Panel {
 		private Class<? extends Page> targetClass;
 		private PageParameters pp;
 		private String iconName;
-		private Class<? extends MenuHighlight> menuHilightInterface;
+		private Class<? extends MenuHighlight> menuHighlightInterface;
 		
 		public MenuItem(final String name, final Class<? extends Page> targetClass) {
 			this(name, targetClass, null, null, null);
@@ -148,7 +149,7 @@ public abstract class AbstractMenuPanel extends Panel {
 		public MenuItem(final String name, final Class<? extends Page> targetClass, final Class<? extends MenuHighlight> menuHilightInterface, PageParameters pp, String iconName) {
 			this.name = name;
 			this.targetClass = targetClass;
-			this.menuHilightInterface = menuHilightInterface;
+			this.menuHighlightInterface = menuHilightInterface;
 			this.pp = pp;
 			this.iconName = iconName == null ? ImageIcon.ICON_EMPTY : iconName;
 		}
@@ -169,13 +170,13 @@ public abstract class AbstractMenuPanel extends Panel {
 			return iconName;
 		}
 
-		public MenuItem setMenuHilightInterface(Class<? extends MenuHighlight> menuHilightInterface) {
-			this.menuHilightInterface = menuHilightInterface;
+		public MenuItem setMenuHighlightInterface(Class<? extends MenuHighlight> menuHighlightInterface) {
+			this.menuHighlightInterface = menuHighlightInterface;
 			return this;
 		}
 
-		public Class<? extends MenuHighlight> getMenuHilightInterface() {
-			return menuHilightInterface;
+		public Class<? extends MenuHighlight> getMenuHighlightInterface() {
+			return menuHighlightInterface;
 		}
 
 		

From deab4f9271521ce2332c4686cb20368b4e2d05f9 Mon Sep 17 00:00:00 2001
From: mpeters <mpeters@dsv.su.se>
Date: Thu, 28 Jul 2011 16:36:50 +0200
Subject: [PATCH 5/6] Fix of broken ui-logic, was impossible to create new
 final seminar (unless there was one already)

Change-Id: Ia190765d0df29c0e730653e724b80e6018522b89
---
 .../panels/ProjectFinalSeminarContainerPanel.java   | 13 +++----------
 1 file changed, 3 insertions(+), 10 deletions(-)

diff --git a/src/main/java/se/su/dsv/scipro/opponent/panels/ProjectFinalSeminarContainerPanel.java b/src/main/java/se/su/dsv/scipro/opponent/panels/ProjectFinalSeminarContainerPanel.java
index 8a7fa59e52..adcd3edf62 100644
--- a/src/main/java/se/su/dsv/scipro/opponent/panels/ProjectFinalSeminarContainerPanel.java
+++ b/src/main/java/se/su/dsv/scipro/opponent/panels/ProjectFinalSeminarContainerPanel.java
@@ -23,22 +23,15 @@ public class ProjectFinalSeminarContainerPanel extends Panel {
 	public ProjectFinalSeminarContainerPanel(final String id, final Project project, final boolean isSupervisorView){
 		super(id);
 		
-		final List<FinalSeminar> seminars = finalSeminarDao.findFinalSeminarsByProject(project);
+		List<FinalSeminar> seminars = finalSeminarDao.findFinalSeminarsByProject(project);
 		
-		if(isSupervisorView && !seminars.isEmpty() ){
+		if(isSupervisorView && seminars.isEmpty() ){
 			add(new OpponentPanel("addSeminar", project));
 		}
 		else {
 			add(new EmptyPanel("addSeminar"));
 		}
-		add(new Label("noSeminarsMessage", "No final seminar has been created for this project"){
-			private static final long serialVersionUID = 1L;
-
-			@Override
-			public boolean isVisible(){
-				return seminars.size() == 0;
-			}
-		});
+		add(new Label("noSeminarsMessage", "No final seminar has been created for this project").setVisible(seminars.isEmpty()));
 
 		add(new ListView<FinalSeminar>("seminarList", seminars){
 			private static final long serialVersionUID = 1L;

From 242f93fe2cf7ef86f0e6776062bad3120a9ae8be Mon Sep 17 00:00:00 2001
From: Fredrik Norberg <fnorbe@dsv.su.se>
Date: Thu, 28 Jul 2011 17:14:22 +0200
Subject: [PATCH 6/6] fixed test failure

---
 src/main/webapp/WEB-INF/web.xml | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

diff --git a/src/main/webapp/WEB-INF/web.xml b/src/main/webapp/WEB-INF/web.xml
index 54e9218c03..d1b02a7ae0 100644
--- a/src/main/webapp/WEB-INF/web.xml
+++ b/src/main/webapp/WEB-INF/web.xml
@@ -64,8 +64,8 @@
 		<!-- Use deployment for production, development for development -->
 	<context-param>
 		<param-name>configuration</param-name>
-			<param-value>development</param-value> 
-			<!-- <param-value>deployment</param-value>-->
+			<!--<param-value>development</param-value> -->
+			 <param-value>deployment</param-value>
 	</context-param>