[1.3 QA] Graph item presets and saved views seem to be ignored.

[sgvfr]

Describe the bug

Preset Graphs items in the user profile, or using the "save" button seems to be ignored.

To Reproduce

Steps to reproduce the behavior:

1. Go to 'Console/Configuration/Users//User Settings'
2. Change Preview Graphs Per Page to anything other than 10
3. Go to 'Console/Configuration/Users//General'
4. Enable 'Maintain Custom Graph and User Settings'

logout,
log back in - View device with more than 10 graphs and only 10 are displayed in tree mode

Go to graph_view.php?action=preview
Select 100 and save.
logout, log back in and return to the preview page.. the saved profile is not used.

Have I simply missed a configuration option somewhere?

[TheWitness]

Well, this may be a real issue and likely in 1.2.x too. However, we won't be fixing it there. I'll log it as a non QA bug for now and dig in later.

If you can not save custom settings, it should take the default. If you enable custom settings, it should take either the default or the last thing you saved before it was disabled, or the cached value from the session.

The question becomes, do we wipe the custom settings when an Admin takes away your right to save them. I'm pretty sure that's what we have to do. What do you think?

[sgvfr]

well, I guess it depends on what priority you want the custom settings to be, admin is first of course, then is it groups then users?

If a user is a member of a group, and the option to save custom settings is disabled in the group, but enabled for the user, who wins? and the other way around?

My thought would be default settings are always inherited, group settings override defaults, but user settings override groups so some group members of a group have a little more flexibility . but if the admin removes those rights, then defaults always inherit?

[TheWitness]

Right now, it's strictly a permissive system and there is no precedence. If the user gets it, then user wins by it's nature. I guess that could change. I know that at one point, a user had asked that it be more restrictive and that the admin should be able to define the order. My guess is that they wanted some groups to go last if you were a member. I know many customers who will remove all local permissions and leave it entirely to the group. But right now, there is no way to do that en-mass other than with database calls.

[github-actions]

This issue has been automatically marked as stale because it has not had recent activity. It will be closed if no further activity occurs. Thank you for your contributions.

[github-actions]

This issue has been automatically marked as stale because it has not had recent activity. It will be closed if no further activity occurs. Thank you for your contributions.

[TheWitness]

Moved to an enhancement to keep it from closing.