Save the settings in the user management page

[rullzer]

Fixes #12727

Quick fix to save the state of the settings checkboxes. This makes sure that once they are set ownCloud remebers the settings.

Note that it currently is global. So all admins see the same. But like I said quick fix.

CC: @jancborchardt @PVince81 @MorrisJobke @LukasReschke

[MorrisJobke]

Tested and works. 👍

I guess for 8.2 we are fine to fix this in the global way.

[MorrisJobke]

@tessus You created the initial ticket ;) Do you mind to test this?

@kawohl You also reported this. ;)

[PVince81]

@rullzer I guess you didn't find any existing JS API to save to "oc_preferences" instead of "oc_appconfig" ?

[PVince81]

...JS API or backend endpoints

[rullzer]

@PVince81 ah that could also be done of course... let me check.

[rullzer]

@PVince81 https://github.com/owncloud/core/blob/master/core/js/config.js#L58

[PVince81]

Right, then 👍

[rullzer]

@PVince81 I'll create proper tickets for 9.0. For now I think this solution is sufficient but then we should introduce OC.Preference to the js stuff and use that for 9.0. Since adding that now does not seem like a good idea.

[rullzer]

Ticket in #19361

[tessus]

@MorrisJobke I'm running version 8.1.3, but I copied the 3 changed files from branch fix_12727 on top, but it didn't work. I expect that there are more dependencies than just these 3 files which are present only in the current master.

Thus the test failed for me. I could install oc from current master on a spare machine, but I guess several people have already done this.

[MorrisJobke]

Thus the test failed for me. I could install oc from current master on a spare machine, but I guess several people have already done this.

Yes. Thanks anyway :)

[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.