Table Option changes not persistent

[hermann-san]

Describe the bug

changed table options are no persistent where the should be persistent.

Context

• Hardware: VPS bought online
• YunoHost version: 4.3.6.3 (stable)
• I have access to my server: Through SSH | through the webadmin | direct access via keyboard / screen | ...
• Are you in a special context or did you perform some particular tweaking on your YunoHost instance?: no
• Using, or trying to install package version/branch: Grccy 3.3.1

Steps to reproduce

1. go to "manage master data" - "products"
2. Table Option - view columns, checkmark "Default shop" + click OK. Column "default shop" is now visible.
3. close the Grocy browser tab.
4. after 3 hours check the table again. Default shop might be gone, where it should be still there.

Expected behavior

changes made in the table options should be persistent. They should not revert back to default (according to the Grocy developer)
Grocy developer says that this issue is not reproduceable in a regular Grocy installation and the issue may be related to Yunohost.
grocy/grocy#1366

Logs

N/A

[memo-567]

Unfortunately, I could not reproduce the issue.

I tested it with different browsers, closed the browsers, restarted OS and waited three hours.
The settings stayed the way I made them.

[hermann-san]

well, according to the message post time you could not have waited more then 2h 15 min. (at max.) ;-)
The important bit is that you must not refresh the page in between (e.g. after 60 min.) That will prolong the table option reset time.
It seems that the reset timer will start again after each refresh.

Either way, it's very bizarre. I've used a completely new installation with a single entry only. Especially in a Yunohost environment it should be easy to reproduce.
Thanks for trying anyways though.

[memo-567]

well, according to the message post time you could not have waited more then 2h 15 min. (at max.) ;-)

Well, since I also read the issues on Grocy itself - I can :)

The important bit is that you must not refresh the page in between

However, I have not tested this way, but I will and will then report.

[ericgaspar]

Is this an upstream issue?

[memo-567]

After testing it exactly like this, I can confirm this issue.

While this practice is uncommon and rare, it should not result in this error. However, I can't imagine that this is related to YunoHost.

@berrnd

[hermann-san]

@memo-567 from my point of view it's something helpful in my daily Grocy work, to be able to change a table view and expecting it to be still here the next day, without having to refresh the page constantly.
Thanks for testing this again.

[hermann-san]

I did test this issue grocy/grocy#1366 with Grocy on non-Yunohost environment and I was able to reproduce the error. So for me the issue doesn't appear to be related to Yunohost.

[berrnd]

Resolved by grocy/grocy#2494, thanks for sharing your time help tracking this down in the last 2 years.