Add occ command to reset for all users

[tomneedham]

For admins to show new features / clients / manuals to their users, then can theme the firstrunwizard again and trigger it nicely

• Test on Config class method

[DeepDiver1975]

occ already has a command to set user preferences .... maybe we add an option --all-users over there?

[tomneedham]

Possibly - but that feels like handing a loaded gun to admins - at least this is contained to a flag that is expected to be on all users, and has limited impact.

[PVince81]

I wonder if we need a base class in core for all commands that apply to all users... The number of commands with "one user", "multiple users" or "multiple group" or "all" as arguments is increasing and we keep on reimplementing.

[tomneedham]

Rebased

[PVince81]

please take over. goal is just to finish, no need to add further options for now

[PVince81]

@sharidas ^

[codecov]

Codecov Report

Merging #83 into master will increase coverage by 0.59%.
The diff coverage is 30.43%.

@@             Coverage Diff              @@
##             master      #83      +/-   ##
============================================
+ Coverage     27.47%   28.07%   +0.59%     
- Complexity       10       16       +6     
============================================
  Files             8        9       +1     
  Lines            91      114      +23     
============================================
+ Hits             25       32       +7     
- Misses           66       82      +16

Impacted Files	Coverage Δ	Complexity Δ
lib/Command/ResetAll.php	0% <0%> (ø)	4 <4> (?)
lib/config.php	100% <100%> (ø)	9 <2> (+2)	⬆️

---

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update c17a10f...e4e4527. Read the comment docs.

[codecov]

Codecov Report

Merging #83 into master will increase coverage by 0.59%.
The diff coverage is 30.43%.

@@             Coverage Diff              @@
##             master      #83      +/-   ##
============================================
+ Coverage     27.47%   28.07%   +0.59%     
- Complexity       10       16       +6     
============================================
  Files             8        9       +1     
  Lines            91      114      +23     
============================================
+ Hits             25       32       +7     
- Misses           66       82      +16

Impacted Files	Coverage Δ	Complexity Δ
lib/Command/ResetAll.php	0% <0%> (ø)	4 <4> (?)
lib/config.php	100% <100%> (ø)	9 <2> (+2)	⬆️

---

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update c17a10f...e4e4527. Read the comment docs.

[sharidas]

Ouch. I wonder if this is tested. Sorry no offense here. Or may be its missed to commit :). But this app has missing namespace in the info.xml. And as a result the command occ fails. I have updated the PR with the same. Below is the result I have verified in the console:

 sujith@sujith-ownCloud  ~/test/owncloud3   master ●  ./occ firstrunwizard:reset-all
Cannot load Xdebug - it was already loaded
Resetting firstrunwizard for all users
    1 [============================]
Done
 sujith@sujith-ownCloud  ~/test/owncloud3   master ● 

[sharidas]

@PVince81 The pull request is ready for review. The CI is cleared.

[tomneedham]

But this app has missing namespace in the info.xml. And as a result the command occ fails.

Nice spot. My guess is a osx vs linux filesystem casing issue I've seen before with the autoloader. So probably worked for me locally but on a linux box it dies. Good catch!

[PVince81]

👍

[PVince81]

as firstrunwizard is bundled, we might want to add this addition in the core changelog or release notes only ? @pmaier1

[pmaier1]

core changelog or release notes

both, plx

[phil-davis]

Backport stable10 #93