Array of objects parsing

[ljharb]

Array of objects parsing

[nlf]

The problem here is that paypal uses bracket notation [0] for arrays, but dot notation object.property for object properties, while qs only supports bracket notation.

In order to fully support payloads like paypal sends, qs needs to be modified to allow for dot notation in object properties. It's on my list of possible new features.

[longuniquename]

@nlf, thank you for quick response. OK, I'll be waiting for this feature )

[longuniquename]

Native querystring module gives this response:

{
    "transaction[0].is_primary_receiver":          "true",
    "transaction[0].id_for_sender_txn":            "1EN35283R74763703",
    "log_default_shipping_address_in_transaction": "false",
    "transaction[0].receiver":                     "[email protected]",
    "action_type":                                 "PAY",
    "ipn_notification_url":                        "http://staging.mercherdev.com/ipn",
    "transaction[1].paymentType":                  "SERVICE",
    "transaction[0].amount":                       "USD 107.00",
    "charset":                                     "windows-1252",
    "transaction_type":                            "Adaptive Payment PAY",
    "transaction[1].id_for_sender_txn":            "7XF42668MS476925V",
    "transaction[1].is_primary_receiver":          "false",
    "transaction[0].status":                       "Completed",
    "notify_version":                              "UNVERSIONED",
    "transaction[0].id":                           "90E58532R7615693H",
    "cancel_url":                                  "http://staging.mercherdev.com/test",
    "transaction[1].status_for_sender_txn":        "Completed",
    "transaction[1].receiver":                     "[email protected]",
    "verify_sign":                                 "AFcWxV21C7fd0v3bYYYRCpSSRl31AW50ZrxhpTVKQjEObfEFfTcdVouU",
    "sender_email":                                "[email protected]",
    "fees_payer":                                  "PRIMARYRECEIVER",
    "transaction[0].status_for_sender_txn":        "Completed",
    "return_url":                                  "http://staging.mercherdev.com/test",
    "transaction[0].paymentType":                  "GOODS",
    "transaction[1].amount":                       "USD 2.00",
    "reverse_all_parallel_payments_on_error":      "true",
    "tracking_id":                                 "31",
    "transaction[1].pending_reason":               "NONE",
    "pay_key":                                     "AP-10U799054T2106106",
    "transaction[1].id":                           "6XR12273235492049",
    "transaction[0].pending_reason":               "NONE",
    "status":                                      "COMPLETED",
    "transaction[1].status":                       "Completed",
    "test_ipn":                                    "1",
    "payment_request_date":                        "Mon Oct 06 09:00:28 PDT 2014"
}

For now I'll try to parse these keys into objects. But it would be great to have it supported by your module.

[nlf]

Closed via #47

[hueniverse]

So... this is not marked with a release, is actually a breaking change (see hapijs/hapi#2718) and the comment above is referencing back this issue.

This is a pretty big change. I am not sure what to do about this now as it is breaking code for hapi users.