Skip to content

Commit 852f69f

Browse files
committed
fixed display groups bug
1 parent 44dfa4e commit 852f69f

File tree

4 files changed

+11
-11
lines changed

4 files changed

+11
-11
lines changed

src/main/java/ie/tcd/pavel/ExerciseChartsPage.java

Lines changed: 8 additions & 8 deletions
Original file line numberDiff line numberDiff line change
@@ -31,6 +31,8 @@
3131
import ie.tcd.pavel.security.SecurityUtils;
3232
import ie.tcd.pavel.utility.ExerciseTypes;
3333

34+
import javax.management.Notification;
35+
3436
@Route("charts")
3537
public class ExerciseChartsPage extends VerticalLayout {
3638

@@ -42,7 +44,7 @@ public class ExerciseChartsPage extends VerticalLayout {
4244
public ExerciseChartsPage() {
4345
database = BeanUtil.getBean(MongoDBOperations.class);
4446
exerciseTypes = BeanUtil.getBean(ExerciseTypes.class);
45-
47+
4648
String[] exercises = exerciseTypes.getExerciseTypes();
4749
List<Group> groups= database.getGroupsByUser(SecurityUtils.getUsername());
4850
User currentUser= database.getUserByLogin(SecurityUtils.getUsername());
@@ -63,9 +65,12 @@ public ExerciseChartsPage() {
6365
comboBoxExercise.setLabel("Exercise");
6466
comboBoxExercise.setItems(exercises);
6567
comboBoxExercise.addValueChangeListener(exerciseEvent ->{
66-
HashMap<User, Double> data= database.inGroupGetCumulativeValuesByUserAndType(comboBoxGroup.getValue(), comboBoxExercise.getValue());
68+
HashMap<User, Double> data= database.inGroupGetCumulativeValuesByUserAndType(comboBoxGroup.getValue(),
69+
comboBoxExercise.getValue());
70+
6771

6872
Chart pieChart= new Chart(ChartType.PIE);
73+
horizontalLayout.remove(pieChart);
6974
Configuration pieChartConfig= pieChart.getConfiguration();
7075
pieChartConfig.setTitle("Member Contribution");
7176
pieChartConfig.setSubTitle(comboBoxExercise.getValue());
@@ -81,13 +86,8 @@ public ExerciseChartsPage() {
8186
pieChartConfig.setPlotOptions(plotOptions);
8287

8388
DataSeries dataSeries= new DataSeries();
84-
dataSeries.clear();
85-
DataSeriesItem currentUserSeries= new DataSeriesItem(currentUser.getLogin(), data.get(currentUser));
86-
currentUserSeries.setSliced(true);
87-
currentUserSeries.setSelected(true);
88-
dataSeries.add(currentUserSeries);
8989
for(int i= 0; i<database.getUsersByGroup(comboBoxGroup.getValue()).size(); i++)
90-
{
90+
{
9191
User user= database.getUsersByGroup(comboBoxGroup.getValue()).get(i);
9292
dataSeries.add(new DataSeriesItem(user.getLogin(), data.get(user)));
9393
}

src/main/java/ie/tcd/pavel/MainPage.java

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -35,7 +35,7 @@
3535
public class MainPage extends AppLayout implements BeforeEnterObserver {
3636

3737
private HashMap<Tab, Component> tabMap = new HashMap<>();
38-
private Image background = new Image("https://i.imgur.com/2ikSvg1.jpg", "Background");
38+
private Image background = new Image("https://i.imgur.com/4AhkxVz.jpg", "Background");
3939

4040
public MainPage() {
4141
this.setContent(background);

src/main/java/ie/tcd/pavel/MongoDBOperations.java

Lines changed: 2 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -220,7 +220,7 @@ public List<User> getUsersByGroup(String group)
220220
List<Group> resultGroup = mongoTemplate.query(Group.class).matching(searchGroup).all();
221221
List<User> users = new ArrayList<>();
222222
for (Group gr: resultGroup) {
223-
users.add(getUserByLogin(gr.getName()));
223+
users.add(getUserByLogin(gr.getUser()));
224224
}
225225
return users;
226226
}
@@ -250,6 +250,7 @@ public HashMap<User,Double> inGroupGetCumulativeValuesByUserAndType(String group
250250
List<Exercise> exercises = allUsersExercises.get(user);
251251
for(Exercise exercise: exercises)
252252
{
253+
System.out.println(ExerciseAdaptor.getDistanceValue(exercise.getInformation()));
253254
currentValue+= ExerciseAdaptor.getDistanceValue(exercise.getInformation());
254255
}
255256
cumulativeData.put(user,currentValue);

src/main/java/ie/tcd/pavel/MyGroupsPage.java

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -35,7 +35,6 @@ public MyGroupsPage()
3535
ArrayList<User> users = (ArrayList<User>) database.getUsersByGroup(g.getName());
3636
if(!users.isEmpty()) {
3737
for (User u : users) {
38-
System.out.printf(u.getLogin() + "%n");
3938
group.addContent(new H3(u.getLogin()));
4039
}
4140
}

0 commit comments

Comments
 (0)