CiviCRM
  1. CiviCRM
  2. CRM-10432

duplicate email fields in Cart checkout

    Details

    • Type: Bug Bug
    • Status: Open
    • Priority: Minor Minor
    • Resolution: Unresolved
    • Affects Version/s: 4.2.0
    • Fix Version/s: 4.6
    • Component/s: CiviEvent
    • Labels:
    • Is MIH?:
      No
    • Code Sprint:
      No

      Description

      Reported by kurund: the ParticipantsAndPrices page in the cart workflow shows two email fields for each participant profile.

      This probably has to do with changes in CRM-9587, and the solution is to remove hardcoded "email" fields in the Cart participant profile. We'll have to identify contacts by another unique key, hopefully contact_id is now consistent when a temporary participant in the "Deleted" status is converted into a live participant.
      1. cart.patch
        12 kB
        Peter Haight
      2. cart-fixes-v2.patch
        15 kB
        Peter Haight
      3. Screen Shot 2012-06-22 at 11.47.18 PM.JPG
        59 kB
        Adam Wight

        Activity

        Hide
        Adam Wight added a comment -
        That's bad. Worse, I'm seeing just one field, but can't proceed past the ParticipantsAndPrices page because of failing validation on the invisible second email field.
        Show
        Adam Wight added a comment - That's bad. Worse, I'm seeing just one field, but can't proceed past the ParticipantsAndPrices page because of failing validation on the invisible second email field.
        Hide
        Peter Haight added a comment -
        Fixes issue CRM-10432.
        Show
        Peter Haight added a comment - Fixes issue CRM-10432 .
        Hide
        Peter Haight added a comment -
        On my way home from the sprint I realized there was a problem with the way I fixed a problem with going back during the checkout process. I finally figured out what the real problem was. The logic that deletes temporary contacts if they were removed from the participants was deleting them before going through the whole list of participants and sometimes a later participant might be referencing that contact that was deleted earlier in the list. Now we go through the whole list and just delete any temporary contacts that are completely unreferenced.

        Please apply cart-fixes-v2.patch instead of cart.patch.
        Show
        Peter Haight added a comment - On my way home from the sprint I realized there was a problem with the way I fixed a problem with going back during the checkout process. I finally figured out what the real problem was. The logic that deletes temporary contacts if they were removed from the participants was deleting them before going through the whole list of participants and sometimes a later participant might be referencing that contact that was deleted earlier in the list. Now we go through the whole list and just delete any temporary contacts that are completely unreferenced. Please apply cart-fixes-v2.patch instead of cart.patch.

          People

          • Assignee:
            Adam Wight
            Reporter:
            Adam Wight
          • Votes:
            0 Vote for this issue
            Watchers:
            2 Start watching this issue

            Dates

            • Created:
              Updated:

              Development