Add email option to user:add command

[tomneedham]

Description

In the user:add command you can now specify the email for a user.

I would consider this a regression in functionality, assigned 10.0.3 to reintroduce this feature.

Related Issue

https://central.owncloud.org/t/setting-email-with-occ-doesnt-show-in-gui/7987/3

Motivation and Context

Now that emails are stored in the accounts table there is no way to set the email of a user via occ. Since it is metadata stored alongside users in the accounts table I thought it made sense to add this to the user:add command.

How Has This Been Tested?

Local command line.

Types of changes

• Bug fix (non-breaking change which fixes an issue)
• New feature (non-breaking change which adds functionality)
• Breaking change (fix or feature that would cause existing functionality to change)

Checklist:

• My code follows the code style of this project.
• My change requires a change to the documentation.
• [] I have updated the documentation accordingly. Setting email using occ command is incorrect owncloud-archive/documentation#3152 (comment)
• I have read the CONTRIBUTING document.
• I have added tests to cover my changes.
• All new and existing tests passed.

[PVince81]

Fine by me 👍

@tomneedham was this removed on purpose ? Any link to the diff/PR that removed this option ?

Would like to hear from @DeepDiver1975 if it was on purpose.

[tomneedham]

My guess was that since we used to have email in the preferences table, we never added this to the user add command after the accounts table was added. Unless I missed this getting removed...

[tomneedham]

(You used to be able to run ./occ user:setting email --value=$email)

[tomneedham]

jenkins died from mysql/docker issues....

[PVince81]

@tomneedham rebase it to death until it works

[tomneedham]

I'm going to script this at some point:

• curl github PR api and get CI status
• curl jenkins console | grep 'mysql timeout'
• git rebase origin/master && git push -f origin $branch

-.-

[PVince81]

if you do, make sure to give the script a violent name like "kick-jenkins", "hit-jenkins-in-the-face" or so

[tomneedham]

kicked

[PVince81]

rebased 🔨

[PVince81]

rebased 🔨

@tomneedham can you raise the backport PR to stable10 ?

[PVince81]

stable10: #28306

[lock]

This thread has been automatically locked since there has not been any recent activity after it was closed. Please open a new issue for related bugs.