fix: other major changes

This commit is contained in:
filippo-ferrari 2024-09-08 16:56:55 +02:00
parent ed8d4290b9
commit c716570bc2
5 changed files with 83 additions and 3 deletions

View file

@ -0,0 +1,16 @@
package com.application.munera;
import org.springframework.security.core.Authentication;
import org.springframework.security.core.context.SecurityContextHolder;
import org.springframework.security.core.userdetails.UserDetails;
public class SecurityUtils {
public static UserDetails getLoggedInUserDetails() {
Authentication authentication = SecurityContextHolder.getContext().getAuthentication();
if (authentication != null && authentication.getPrincipal() instanceof UserDetails) {
return (UserDetails) authentication.getPrincipal();
}
return null;
}
}

View file

@ -50,7 +50,7 @@ public class Person extends AbstractEntity {
@ManyToMany(mappedBy = "participants") @ManyToMany(mappedBy = "participants")
private Set<Event> events; private Set<Event> events;
@Column(name = "Username", unique = true, nullable = false) @Column(name = "Username", unique = true)
private String username; // This field will link to the User entity private String username; // This field will link to the User entity
@Column(name = "UserId", unique = true) @Column(name = "UserId", unique = true)

View file

@ -40,6 +40,9 @@ public class PersonService {
return this.personRepository.findAll(); return this.personRepository.findAll();
} }
public Optional<Person> findByUserId(Long id) {
return this.personRepository.findByUserId(id);
}
/** /**
* Finds all people excluding the users'ones. * Finds all people excluding the users'ones.

View file

@ -0,0 +1,19 @@
package com.application.munera.services;
import com.application.munera.data.User;
import com.application.munera.repositories.UserRepository;
import org.springframework.stereotype.Service;
@Service
public class UserService {
private final UserRepository userRepository;
public UserService(UserRepository userRepository) {
this.userRepository = userRepository;
}
public User findByUsername (String username) {
return this.userRepository.findByUsername(username);
}
}

View file

@ -1,5 +1,6 @@
package com.application.munera.views.expenses; package com.application.munera.views.expenses;
import com.application.munera.SecurityUtils;
import com.application.munera.data.*; import com.application.munera.data.*;
import com.application.munera.services.*; import com.application.munera.services.*;
import com.application.munera.views.MainLayout; import com.application.munera.views.MainLayout;
@ -27,7 +28,9 @@ import com.vaadin.flow.data.converter.StringToBigDecimalConverter;
import com.vaadin.flow.data.renderer.ComponentRenderer; import com.vaadin.flow.data.renderer.ComponentRenderer;
import com.vaadin.flow.router.*; import com.vaadin.flow.router.*;
import jakarta.annotation.security.PermitAll; import jakarta.annotation.security.PermitAll;
import org.springframework.beans.factory.annotation.Autowired;
import org.springframework.orm.ObjectOptimisticLockingFailureException; import org.springframework.orm.ObjectOptimisticLockingFailureException;
import org.springframework.security.core.userdetails.UserDetails;
import org.vaadin.klaudeta.PaginatedGrid; import org.vaadin.klaudeta.PaginatedGrid;
import java.util.Objects; import java.util.Objects;
@ -57,6 +60,7 @@ public class ExpensesView extends Div implements BeforeEnterObserver {
private final PersonService personService; private final PersonService personService;
private final EventService eventService; private final EventService eventService;
private final ViewService viewService; private final ViewService viewService;
private final UserService userService;
private TextField name; private TextField name;
private TextField cost; private TextField cost;
private ComboBox<Category> category; private ComboBox<Category> category;
@ -70,12 +74,14 @@ public class ExpensesView extends Div implements BeforeEnterObserver {
private ComboBox<Person> beneficiary; private ComboBox<Person> beneficiary;
private ComboBox<Event> event; private ComboBox<Event> event;
public ExpensesView(ExpenseService expenseService, CategoryService categoryService, PersonService personService, EventService eventService, ViewService viewService) { @Autowired
public ExpensesView(ExpenseService expenseService, CategoryService categoryService, PersonService personService, EventService eventService, ViewService viewService, UserService userService) {
this.expenseService = expenseService; this.expenseService = expenseService;
this.categoryService = categoryService; this.categoryService = categoryService;
this.personService = personService; this.personService = personService;
this.eventService = eventService; this.eventService = eventService;
this.viewService = viewService; this.viewService = viewService;
this.userService = userService;
addClassNames("expenses-view"); addClassNames("expenses-view");
// Create UI // Create UI
@ -152,7 +158,21 @@ public class ExpensesView extends Div implements BeforeEnterObserver {
} }
}); });
// // Event listeners that will remove the selected creditors from the debtors list and vice versa
// // Done so that the user cant create an expense with the same person as creditor and debtor
// payer.addValueChangeListener(event -> {
// Person selectedDebtors = event.getValue();
// final var creditorsSet = new HashSet<>(personService.findAllWithoutUser());
// creditorsSet.removeIf(creditorsSet::contains);
// payer.setItems(creditorsSet);
// });
//
// beneficiary.addValueChangeListener(event -> {
// Person selectedCreditors = event.getValue();
// final var debtorsSet = new HashSet<>(personService.findAllWithoutUser());
// debtorsSet.removeIf(debtorsSet::contains);
// beneficiary.setItems(debtorsSet);
// });
cancel.addClickListener(e -> { cancel.addClickListener(e -> {
clearForm(); clearForm();
@ -193,10 +213,14 @@ public class ExpensesView extends Div implements BeforeEnterObserver {
n.addThemeVariants(NotificationVariant.LUMO_ERROR); n.addThemeVariants(NotificationVariant.LUMO_ERROR);
} }
}); });
// Initialize ComboBox with the logged-in user's Person entity as default
initializeComboBoxes();
} }
@Override @Override
public void beforeEnter(BeforeEnterEvent event) { public void beforeEnter(BeforeEnterEvent event) {
initializeComboBoxes();
Optional<Long> expenseId = event.getRouteParameters().get(EXPENSE_ID).map(Long::parseLong); Optional<Long> expenseId = event.getRouteParameters().get(EXPENSE_ID).map(Long::parseLong);
if (expenseId.isPresent()) { if (expenseId.isPresent()) {
Optional<Expense> expenseFromBackend = expenseService.get(expenseId.get()); Optional<Expense> expenseFromBackend = expenseService.get(expenseId.get());
@ -289,4 +313,22 @@ public class ExpensesView extends Div implements BeforeEnterObserver {
periodUnit.setVisible(isPeriodicChecked); periodUnit.setVisible(isPeriodicChecked);
periodInterval.setVisible(isPeriodicChecked); periodInterval.setVisible(isPeriodicChecked);
} }
private void initializeComboBoxes() {
// Fetch the logged-in user's Person entity
UserDetails userDetails = SecurityUtils.getLoggedInUserDetails();
if (userDetails != null) {
String username = userDetails.getUsername();
final var user = this.userService.findByUsername(username);
if (user != null) {
Optional<Person> loggedInPerson = personService.findByUserId(user.getId());
if (loggedInPerson.isPresent()) {
Person person = loggedInPerson.get();
// Set default values for payer and beneficiary ComboBoxes
payer.setValue(person);
beneficiary.setValue(person);
}
}
}
}
} }