fix compile errors

matsim-vsp · Jan 19, 2024 · c67a16c · c67a16c
1 parent 5a53a53
commit c67a16c
Show file tree

Hide file tree

Showing 2 changed files with 7 additions and 9 deletions.
diff --git a/src/main/java/org/matsim/freight/logistics/LSPPlanImpl.java b/src/main/java/org/matsim/freight/logistics/LSPPlanImpl.java
@@ -58,7 +58,6 @@ public InitialShipmentAssigner getAssigner() {
   @Override
   public LSPPlan setAssigner(InitialShipmentAssigner assigner) {
     this.assigner = assigner;
-    this.assigner.setLSP(this.lsp);
     return this;
   }
 
@@ -101,13 +100,13 @@ public LSP getLSP() {
   @Override
   public void setLSP(LSP lsp) {
     this.lsp = lsp;
-    if (assigner != null) {
-      this.assigner.setLSP(lsp);
-      // yy vom Design her wäre es vlt. einfacher und logischer, wenn der assigner einen backpointer
-      // auf den LSPPlan hätte. Dann
-      // müsste man nicht (wie hier) hedgen gegen unterschiedliche Initialisierungssequenzen. kai,
-      // may'22
-    }
+//    if (assigner != null) {
+//      this.assigner.setLSP(lsp);
+//      // yy vom Design her wäre es vlt. einfacher und logischer, wenn der assigner einen backpointer
+//      // auf den LSPPlan hätte. Dann
+//      // müsste man nicht (wie hier) hedgen gegen unterschiedliche Initialisierungssequenzen. kai,
+//      // may'22
+//    }
     for (LogisticChain solution : logisticChains) {
       solution.setLSP(lsp);
     }

diff --git a/...a/org/matsim/freight/logistics/example/lsp/lspReplanning/CollectionLSPReplanningTest.java b/...a/org/matsim/freight/logistics/example/lsp/lspReplanning/CollectionLSPReplanningTest.java
@@ -190,7 +190,6 @@ public void initialize() {
 					LSPStrategyManager manager = new LSPStrategyManagerImpl();
 					{
 						InitialShipmentAssigner maybeTodayAssigner = new MaybeTodayAssigner();
-						maybeTodayAssigner.setLSP( collectionLSP );
 						final GenericPlanStrategy<LSPPlan, LSP> strategy = new TomorrowShipmentAssignerStrategyFactory( maybeTodayAssigner ).createStrategy();
 						// (a factory makes sense if it is passed around; in this case it feels like overkill.  kai, jul'22)
 						manager.addStrategy( strategy, null, 1 );