docs: correct documentation

fixes #1202

docs/src/modules/ROOT/pages/quickstart/quarkus/quarkus-quickstart.adoc

@@ -64,9 +64,14 @@ To complete this guide, you need:
 
 == The build file and the dependencies
 
-Use https://code.quarkus.io/[code.quarkus.io] to generate an application
+Use https://code.quarkus.io/?a=timefold-solver-quickstart&j=17&e=resteasy&e=resteasy-jackson&e=ai.timefold.solver%3Atimefold-solver-quarkus-jackson&e=ai.timefold.solver%3Atimefold-solver-quarkus[code.quarkus.io] to generate an application
 with the following extensions, for Maven or Gradle:
 
+[NOTE]
+====
+Clicking the link above will automatically select the dependencies for you on *code.quarkus.io*.
+====
+
 * RESTEasy JAX-RS (`quarkus-resteasy`)
 * RESTEasy Jackson (`quarkus-resteasy-jackson`)
 * Timefold Solver (`timefold-solver-quarkus`)
@@ -250,9 +255,9 @@ package org.acme.schooltimetabling.rest;
 
 import java.util.UUID;
 import java.util.concurrent.ExecutionException;
-import javax.inject.Inject;
-import javax.ws.rs.POST;
-import javax.ws.rs.Path;
+import jakarta.inject.Inject;
+import jakarta.ws.rs.POST;
+import jakarta.ws.rs.Path;
 
 import org.acme.schooltimetabling.domain.Timetable;
 import ai.timefold.solver.core.api.solver.SolverJob;
@@ -602,7 +607,7 @@ package org.acme.schooltimetabling.solver;
 import java.time.DayOfWeek;
 import java.time.LocalTime;
 
-import javax.inject.Inject;
+import jakarta.inject.Inject;
 
 import io.quarkus.test.junit.QuarkusTest;
 import org.acme.schooltimetabling.domain.Lesson;
@@ -615,7 +620,7 @@ import ai.timefold.solver.test.api.score.stream.ConstraintVerifier;
 @QuarkusTest
 class TimetableConstraintProviderTest {
 
-    private static final Room ROOM = new Room("Room1");
+    private static final Room ROOM1 = new Room("Room1");
     private static final Timeslot TIMESLOT1 = new Timeslot(DayOfWeek.MONDAY, LocalTime.of(9,0), LocalTime.NOON);
     private static final Timeslot TIMESLOT2 = new Timeslot(DayOfWeek.TUESDAY, LocalTime.of(9,0), LocalTime.NOON);
 
@@ -624,14 +629,23 @@ class TimetableConstraintProviderTest {
 
     @Test
     void roomConflict() {
-        Lesson firstLesson = new Lesson("1", "Subject1", "Teacher1", "Group1", TIMESLOT1, ROOM1);
-        Lesson conflictingLesson = new Lesson("2", "Subject2", "Teacher2", "Group2", TIMESLOT1, ROOM1);
-        Lesson nonConflictingLesson = new Lesson("3", "Subject3", "Teacher3", "Group3", TIMESLOT2, ROOM1);
+        Lesson firstLesson = createTestLesson("1", "Subject1", "Teacher1", "Group1", TIMESLOT1, ROOM1);
+        Lesson conflictingLesson = createTestLesson("2", "Subject2", "Teacher2", "Group2", TIMESLOT1, ROOM1);
+        Lesson nonConflictingLesson = createTestLesson("3", "Subject3", "Teacher3", "Group3", TIMESLOT2, ROOM1);
+
         constraintVerifier.verifyThat(TimetableConstraintProvider::roomConflict)
                 .given(firstLesson, conflictingLesson, nonConflictingLesson)
                 .penalizesBy(1);
     }
 
+    // This methods avoids the need to create a constructor just for testing and uses the setter like the solver does.
+    Lesson createTestLesson(String id, String subject, String teacher, String studentGroup, Timeslot timeslot, Room room) {
+        Lesson lesson = new Lesson(id, subject, teacher, group);
+        lesson.setTimeslot(timeslot);
+        lesson.setRoom(room);
+        return lesson;
+    }
+
 }
 ----
 --
@@ -670,11 +684,20 @@ class TimetableConstraintProviderTest {
         val firstLesson = Lesson("1", "Subject1", "Teacher1", "Group1", TIMESLOT1, ROOM1)
         val conflictingLesson = Lesson("2", "Subject2", "Teacher2", "Group2", TIMESLOT1, ROOM1)
         val nonConflictingLesson = Lesson("3", "Subject3", "Teacher3", "Group3", TIMESLOT2, ROOM1)
+
         constraintVerifier.verifyThat(TimeTableConstraintProvider::roomConflict)
             .given(firstLesson, conflictingLesson, nonConflictingLesson)
             .penalizesBy(1)
     }
 
+    // This methods avoids the need to create a constructor just for testing and uses the setter like the solver does.
+    fun createTestLesson(String id, String subject, String teacher, String studentGroup, Timeslot timeslot, Room room) : Lesson {
+        Lesson lesson = Lesson(id, subject, teacher, group);
+        lesson.setTimeslot(timeslot);
+        lesson.setRoom(room);
+        return lesson;
+    }
+
 }
 ----
 --

docs/src/modules/ROOT/pages/quickstart/school-timetabling/school-timetabling-constraints.adoc

@@ -170,7 +170,7 @@ public class TimetableConstraintProvider implements ConstraintProvider {
         };
     }
 
-    private Constraint roomConflict(ConstraintFactory constraintFactory) {
+    Constraint roomConflict(ConstraintFactory constraintFactory) {
         // A room can accommodate at most one lesson at the same time.
         return constraintFactory
                 // Select each pair of 2 different lessons ...
@@ -184,7 +184,7 @@ public class TimetableConstraintProvider implements ConstraintProvider {
                 .asConstraint("Room conflict");
     }
 
-    private Constraint teacherConflict(ConstraintFactory constraintFactory) {
+    Constraint teacherConflict(ConstraintFactory constraintFactory) {
         // A teacher can teach at most one lesson at the same time.
         return constraintFactory
                 .forEachUniquePair(Lesson.class,
@@ -194,7 +194,7 @@ public class TimetableConstraintProvider implements ConstraintProvider {
                 .asConstraint("Teacher conflict");
     }
 
-    private Constraint studentGroupConflict(ConstraintFactory constraintFactory) {
+    Constraint studentGroupConflict(ConstraintFactory constraintFactory) {
         // A student can attend at most one lesson at the same time.
         return constraintFactory
                 .forEachUniquePair(Lesson.class,

docs/src/modules/ROOT/pages/using-timefold-solver/modeling-planning-problems.adoc

@@ -1066,7 +1066,7 @@ This does not apply to Timefold Solver for Python, since Python does not have pr
 ====
 
 A planning value range is the set of possible planning values for a planning variable.
-Planning value ranges need to be countable, meaning there needs to be a finite number of possible planning values.
+Planning value ranges need to come from a finite collection.
 
 
 [#planningValueRangeProvider]