zdm/envoy
zdm/envoy
Archived
3
Fork 0
Code Issues 22 Pull Requests Releases 1 Wiki Activity

Fixed bug not updating GroupCreationTab after a new contact was added

Browse Source 
Fixes #35
This commit is contained in:
Leon Hofmeister 2020-09-22 17:51:33 +02:00
parent 41f07dc452
commit c6819e637b
Signed by: delvh
GPG Key ID: 3DECE05F6D9A647C
2 changed files with 19 additions and 3 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

7
client/src/main/java/envoy/client/ui/controller/ContactSearchTab.java
View File

@ -42,9 +42,10 @@ public class ContactSearchTab implements EventListener {
@FXML @FXML
private ListView<User> userList; private ListView<User> userList;
private Alert alert = new Alert(AlertType.CONFIRMATION);
private User currentlySelectedUser; private User currentlySelectedUser;
private final Alert alert = new Alert(AlertType.CONFIRMATION);
private static final EventBus eventBus = EventBus.getInstance(); private static final EventBus eventBus = EventBus.getInstance();
private static final Logger logger = EnvoyLog.getLogger(ChatScene.class); private static final Logger logger = EnvoyLog.getLogger(ChatScene.class);
@ -69,7 +70,7 @@ public class ContactSearchTab implements EventListener {
} }
/** /**
* Disables the clear and search button if no text is present in the search bar. * If text is present, sends a request to the server.
* *
* @since Envoy Client v0.1-beta * @since Envoy Client v0.1-beta
*/ */
@ -102,7 +103,7 @@ public class ContactSearchTab implements EventListener {
private void userListClicked() { private void userListClicked() {
final var user = userList.getSelectionModel().getSelectedItem(); final var user = userList.getSelectionModel().getSelectedItem();
if (user != null) { if (user != null) {
currentlySelectedUser = user; currentlySelectedUser = user;
final var event = new ContactOperation(currentlySelectedUser, ElementOperation.ADD); final var event = new ContactOperation(currentlySelectedUser, ElementOperation.ADD);
// Sends the event to the server // Sends the event to the server
eventBus.dispatch(new SendEvent(event)); eventBus.dispatch(new SendEvent(event));

15
client/src/main/java/envoy/client/ui/controller/GroupCreationTab.java
View File

@ -4,6 +4,7 @@ import static java.util.function.Predicate.not;
import java.util.stream.Collectors; import java.util.stream.Collectors;
import javafx.application.Platform;
import javafx.fxml.FXML; import javafx.fxml.FXML;
import javafx.scene.control.*; import javafx.scene.control.*;
import javafx.scene.layout.HBox; import javafx.scene.layout.HBox;
@ -13,6 +14,7 @@ import envoy.client.event.*;
import envoy.client.ui.listcell.*; import envoy.client.ui.listcell.*;
import envoy.data.*; import envoy.data.*;
import envoy.event.GroupCreation; import envoy.event.GroupCreation;
import envoy.event.contact.ContactOperation;
import envoy.util.Bounds; import envoy.util.Bounds;
import dev.kske.eventbus.*; import dev.kske.eventbus.*;
@ -70,6 +72,7 @@ public class GroupCreationTab implements EventListener {
userList.setCellFactory(new ListCellFactory<>(ContactControl::new)); userList.setCellFactory(new ListCellFactory<>(ContactControl::new));
userList.getSelectionModel().setSelectionMode(SelectionMode.MULTIPLE); userList.getSelectionModel().setSelectionMode(SelectionMode.MULTIPLE);
createButton.setDisable(true); createButton.setDisable(true);
eventBus.registerListener(this);
userList.getItems() userList.getItems()
.addAll(localDB.getChats() .addAll(localDB.getChats()
.stream() .stream()
@ -199,4 +202,16 @@ public class GroupCreationTab implements EventListener {
errorProceedBox.setMinHeight(value); errorProceedBox.setMinHeight(value);
errorProceedBox.setMaxHeight(value); errorProceedBox.setMaxHeight(value);
} }
@Event
private void onContactOperation(ContactOperation operation) {
switch (operation.getOperationType()) {
case ADD:
if (operation.get() instanceof User) Platform.runLater(() -> userList.getItems().add((User) operation.get()));
break;
case REMOVE:
Platform.runLater(() -> userList.getItems().removeIf(u -> u.equals(operation.get())));
break;
}
}
} }