Added support for empty transfer allowance in R5RTransferAllowance;

Setting fare type for first leg of trip.

java-r5rcore/src/org/ipea/r5r/Fares/R5RTransferAllowance.java

@@ -10,6 +10,11 @@ public R5RTransferAllowance(int fareType, int value, int number, int expirationT
         this.fareType = fareType;
     }
 
+    public R5RTransferAllowance() {
+        super();
+        this.fareType = -1;
+    }
+
     public R5RTransferAllowance tightenExpiration(int maxClockTime){
         // cap expiration time of transfer at max clock time of search, so that transfer slips that technically have more time
         // remaining, but that time cannot be used within the constraints of this search, can be pruned.
@@ -34,7 +39,11 @@ public R5RTransferAllowance tightenExpiration(int maxClockTime){
      * comparable.
      */
     public boolean atLeastAsGoodForAllFutureRedemptions(R5RTransferAllowance other){
+        // for empty transfer allowances, this is always true
+        if (other.fareType == -1) return true;
+        // if this transfer allowance is for a different fare type, it is not comparable
         if (fareType != other.fareType) return false;
+        // otherwise, compare value, expiration time, and number of transfers remaining
         return value >= other.value && expirationTime >= other.expirationTime && number >= other.number;
     }
 

java-r5rcore/src/org/ipea/r5r/Fares/RuleBasedInRoutingFareCalculator.java

@@ -120,6 +120,7 @@ public FareBounds calculateFare(McRaptorSuboptimalPathProfileRouter.McRaptorStat
             fareForState = getFullFareForRoute(currentPatternIndex);
 
             previousPatternIndex = currentPatternIndex;
+            lastFareType = faresPerRoute[currentPatternIndex].getTypeIndex();
         }
 
         // subsequent legs
@@ -162,7 +163,7 @@ public FareBounds calculateFare(McRaptorSuboptimalPathProfileRouter.McRaptorStat
 
         // fares are limited by the maxFare parameter
         if (fareStructure.getFareCap() > 0) {
-            fareForState = Math.min(fareForState, Math.round(fareStructure.getIntegerFareCap()));
+            fareForState = Math.min(fareForState, fareStructure.getIntegerFareCap());
         }
 
         // initialize transfer allowance
@@ -173,20 +174,20 @@ public FareBounds calculateFare(McRaptorSuboptimalPathProfileRouter.McRaptorStat
         // if transfer allowances are inactive (for debugging purposes), just use and empty transfer allowance and
         // quit the function
         if (!ParetoItineraryPlanner.travelAllowanceActive) {
-            return new FareBounds(fareForState, new TransferAllowance());
+            return new FareBounds(fareForState, new R5RTransferAllowance());
         }
 
         // pattern is valid?
         if (currentPatternIndex == -1) {
             // no public transport patterns - return empty transfer allowance
-            return new FareBounds(fareForState, new TransferAllowance());
+            return new FareBounds(fareForState, new R5RTransferAllowance());
         }
 
         // remaining transfers
         int numberOfRemainingTransfers = fareStructure.getMaxDiscountedTransfers() - discountsApplied;
         if (numberOfRemainingTransfers <= 0) {
             // no remaining available transfers - return empty transfer allowance
-            return new FareBounds(fareForState, new TransferAllowance());
+            return new FareBounds(fareForState, new R5RTransferAllowance());
         }
 
         // get max benefit from possible transfers
@@ -210,7 +211,7 @@ public FareBounds calculateFare(McRaptorSuboptimalPathProfileRouter.McRaptorStat
         int expirationTime = currentBoardTime + fareStructure.getTransferTimeAllowanceSeconds();
 
         // build transfer allowance considering constraints above
-        TransferAllowance transferAllowance = new R5RTransferAllowance(lastFareType, maxAllowanceValue, numberOfRemainingTransfers, expirationTime);
+        R5RTransferAllowance transferAllowance = new R5RTransferAllowance(lastFareType, maxAllowanceValue, numberOfRemainingTransfers, expirationTime);
         return new FareBounds(fareForState, transferAllowance);
 
     }