Fix of hiding of "Last 12 months" on peer portal page depending on

what's currently visible

Change-Id: I6b0b25817373bacc2af8e3413fc432955b7d9f4a
This commit is contained in:
mpeters 2011-07-28 15:45:11 +02:00
parent 115310561c
commit 532b93f008
2 changed files with 8 additions and 1 deletions
src/main/java/se/su/dsv/scipro/peer/panels

@ -89,7 +89,7 @@
<span class="box-title">Best rated reviewers</span> <span class="box-title">Best rated reviewers</span>
<div wicket:id="bestRatedPanel" class="append-bottom"></div> <div wicket:id="bestRatedPanel" class="append-bottom"></div>
</div> </div>
<div> <div wicket:id="statsTimespanContainer">
<i><span class="small right">Last 12 months</span></i> <i><span class="small right">Last 12 months</span></i>
</div> </div>
</div> </div>

@ -221,16 +221,23 @@ public class PeerPortalPanel extends Panel {
int displayedReviews = gsettings.getNumberOfLatestReviewsDisplayed(); int displayedReviews = gsettings.getNumberOfLatestReviewsDisplayed();
boolean showStatsTimespan = false;
if(gsettings.isPeerDisplayNumberOfReviewsPerformed()){ if(gsettings.isPeerDisplayNumberOfReviewsPerformed()){
add(new MostFrequentReviewersPanel(mostFrequentPanel)); add(new MostFrequentReviewersPanel(mostFrequentPanel));
showStatsTimespan = true;
} else { } else {
add(new InvisiblePanel(mostFrequentPanel)); add(new InvisiblePanel(mostFrequentPanel));
} }
if( gsettings.isPeerRatingsEnabled() ){ if( gsettings.isPeerRatingsEnabled() ){
add(new WebMarkupContainer(bestRatedContainer).add(new BestRatedReviewersPanel("bestRatedPanel") ) ); add(new WebMarkupContainer(bestRatedContainer).add(new BestRatedReviewersPanel("bestRatedPanel") ) );
showStatsTimespan = true;
} else { } else {
add(new InvisiblePanel(bestRatedContainer)); add(new InvisiblePanel(bestRatedContainer));
} }
WebMarkupContainer statsTimespanContainer = new WebMarkupContainer("statsTimespanContainer");
statsTimespanContainer.setVisible(showStatsTimespan);
add(statsTimespanContainer);
if(gsettings.isPeerDisplayLatestReviews()) { if(gsettings.isPeerDisplayLatestReviews()) {
// Checkbox on the AdminPeerSettingsPage is not visible since it's // Checkbox on the AdminPeerSettingsPage is not visible since it's
// not decided what kind of info that should be public for students. // not decided what kind of info that should be public for students.