Merge pull request #14 from Travelport-Ukraine/fix-pc-import

Fix pc import

package.json

@@ -1,6 +1,6 @@
 {
   "name": "uapi-json",
-  "version": "0.2.1",
+  "version": "0.2.2",
   "description": "Travelport Universal API",
   "main": "build/",
   "files": [

src/Air/AirParser.js

@@ -17,6 +17,19 @@ const groupSegmentsByLegs = (segments) => {
   return result;
 };
 
+const getPlatingCarrier = (booking) => {
+  let platingCarriers = _.pluck(booking['air:AirPricingInfo'], 'PlatingCarrier').filter(pc => pc);
+
+  if (platingCarriers.length === 0) {
+    // FIXME: use a smart collapse algorithm?
+    platingCarriers = _.pluck(booking['air:TicketingModifiers'], 'PlatingCarrier');
+  }
+
+  const singlePlatingCarrier = _.uniq(platingCarriers);
+
+  return singlePlatingCarrier[0];
+};
+
 const searchLowFaresValidate = (obj) => {
   // +List, e.g. AirPricePointList, see below
   const rootArrays = ['AirPricePoint', 'AirSegment', 'FareInfo', 'FlightDetails', 'Route'];
@@ -409,12 +422,7 @@ function extractBookings(obj) {
     }
 
     // we usually have one plating carrier across all per-passenger reservations
-    let platingCarriers = _.pluck(booking['air:AirPricingInfo'], 'PlatingCarrier');
-    const singlePlatingCarrier = _.uniq(platingCarriers);
-    if (singlePlatingCarrier.length === 1) {
-      // FIXME: use a smart collapse algorithm?
-      platingCarriers = singlePlatingCarrier[0];
-    }
+    const platingCarriers = getPlatingCarrier(booking);
 
     const passengers = getPassengers.call(
       this,

test/Air/AirParser.test.js

@@ -130,6 +130,7 @@ describe('#AirParser', function () {
     assert(_.isArray(jsonResult), 'result not array');
     jsonResult.forEach(result => {
       assert(result.bookingPCC, 'no booking pcc');
+      assert(result.platingCarrier, 'no PC');
       assert(_.isArray(result.passengers), 'passengers is not array');
       assert(result.passengers.length);
       assert(result.pnr, 'no pnr');