steven/csep-2025
1
0
Fork 0
Code Issues Pull requests Projects Releases Packages Wiki Activity Actions

Merge branch 'bugfix/new-recipe-selection' into 'main'

Browse source 
Bugfix/new recipe selection

Closes #21 and #19

See merge request cse1105/2025-2026/teams/csep-team-76!22
This commit is contained in:
Oskar Rasieński 2025-12-17 13:44:52 +01:00
commit f5eed945a2
2 changed files with 10 additions and 9 deletions
Download patch file Download diff file Expand all files Collapse all files

13
client/src/main/java/client/scenes/FoodpalApplicationCtrl.java
View file

@ -223,14 +223,13 @@ public class FoodpalApplicationCtrl implements LocaleAware {
*/
@FXML
private void addRecipe() {
// a default factory provides the value
Recipe newRecipe = DefaultRecipeFactory.getDefaultRecipe();
Recipe newRecipe = DefaultRecipeFactory.getDefaultRecipe(); // Create default recipe
try {
server.addRecipe(newRecipe);
refresh();
// the list focuses on the new recipe
// otherwise strange issues occur when the autofocus on edit box is called
recipeList.getFocusModel().focus(recipeList.getItems().indexOf(newRecipe));
newRecipe = server.addRecipe(newRecipe); // get the new recipe id
refresh(); // refresh view with server recipes
// Select newly created recipe
recipeList.getSelectionModel().select(recipeList.getItems().indexOf(newRecipe));
} catch (IOException | InterruptedException e) {
printError("Error occurred when adding recipe!");
}

6
commons/src/main/java/commons/Recipe.java
View file

@ -157,7 +157,8 @@ public class Recipe {
@Override
public String toString() {
return "Recipe{" +
"name='" + name + '\'' +
"id=" + id +
", name='" + name + '\'' +
", ingredientsCount=" + ingredients.size() +
", preparationStepsCount=" + preparationSteps.size() +
"}";
@ -167,7 +168,8 @@ public class Recipe {
public String toDetailedString() {
// More detailed toString for debugging.
return "Recipe{" +
"name='" + name + '\'' +
"id=" + id +
", name='" + name + '\'' +
", ingredients=" + ingredients +
", preparationSteps=" + preparationSteps +
'}';