Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Refractor profile email change behaviour #47

Open
alexander-schranz opened this issue Dec 20, 2016 · 1 comment
Open

Refractor profile email change behaviour #47

alexander-schranz opened this issue Dec 20, 2016 · 1 comment

Comments

@alexander-schranz
Copy link
Member

alexander-schranz commented Dec 20, 2016

Q A
Bug? no
New Feature? no
Sulu Version ~

Actual Behavior

The profile email address is currently read from the Contact Main Email. This make some problem because the content manager always need to change the contact and the user email adress.

Expected Behavior

The expect behaviour is the user email is read from the user itself on the profile and is written with the email token to an own entity.

Possible Solutions

Add an email field to EmailConfirmationToken entity.
Read (when exists) and write to this entity instead of the User entity.

@alexander-schranz alexander-schranz added this to the 1.0.0 milestone Mar 6, 2017
@alexander-schranz alexander-schranz removed this from the 1.0.0 milestone Dec 4, 2017
alexander-schranz pushed a commit to alexander-schranz/SuluCommunityBundle that referenced this issue Dec 4, 2017
@alexander-schranz
Copy link
Member Author

Best solution would that the email address is stored in the EmailConfirmationToken entity instead of ContactMainEmail Entity. In the ProfileType form we would do something like this:

        $builder->add(
            'email',
            EmailType::class,
            [
                'label' => 'E-Mail-Adresse',
                'mapped' => false,
            ]
        );

so the email of the user does not get directly mapped to the User entity. From the form you its possible to read then the not mapped value with $form->get('email')->getData() and if it is another value as $user->getEmail(); it can be written in the EmailConfirmationToken and send a confirmation email to the user.
Also it could be configureable if the user can change it's email without confirming it.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

1 participant