Uploaded image for project: 'CiviCRM'
  1. CiviCRM
  2. CRM-6247

When using a profile in a registration for event or online contribution, don't remove the email field from it, but remove the mandatory email field

    Details

    • Type: Improvement
    • Status: Done/Fixed
    • Priority: Trivial
    • Resolution: Won't Fix
    • Affects Version/s: 3.2
    • Fix Version/s: 4.3.0
    • Component/s: CiviCRM Profile
    • Labels:
      None

      Description

      Small thing I patched on some installs, testing here if you thing it should go back on the core:

      When you have an email in a profile and use it in a registration form, it's a duplicate with the mandatory email field of the registration (for the first participant at least).

      The actual way of doing it is to keep the mandatory email field separated, then display the profile, but without the email field. Two issues:
      1) It's ugly as the email isn't aligned with the rest of the fields
      2) It forces to have the email as the first field, even so you might want to start with first+last and email in 3rd (as per the config in the profile)

      I'd therefore argue that the duplicate email should be resolved the other way around, and instead of taking off the email from the profile, it should be taken off the first line (and inserted where the profile expects to have the email, probably with a different name/id than the email in the profile)

      What do you think ?

      Need guidance: I haven't a clue in how many places the profile is used and that dedupe trick is applied. I think my changes works no matter what, but have only used civievent in that case, so not sure if it might have an impact on donation or other locations

      Timing: If I can put my hand on one of my custom versions of civi that does that an only that, might be able to push on 3.2, otherwise, will be 3.3. ie no rush on my side, just checking for now if it makes sense and if I haven't missed a big issue

        Attachments

          Activity

            People

            • Assignee:
              kurund Kurund Jalmi
              Reporter:
              xavier xavier dutoit
            • Votes:
              1 Vote for this issue
              Watchers:
              2 Start watching this issue

              Dates

              • Created:
                Updated:
                Resolved: