Bug 366025 - Skrooge perpetually asks me to save updated tab state
Summary: Skrooge perpetually asks me to save updated tab state
Status: RESOLVED FIXED
Alias: None
Product: skrooge
Classification: Applications
Component: general (show other bugs)
Version: unspecified
Platform: unspecified Linux
: NOR minor
Target Milestone: ---
Assignee: Stephane MANKOWSKI
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2016-07-24 00:40 UTC by dberg918
Modified: 2016-08-22 18:40 UTC (History)
1 user (show)

See Also:
Latest Commit:
Version Fixed In:


Attachments
Anonymized test file (3.04 MB, application/octet-stream)
2016-07-26 00:18 UTC, dberg918
Details
Proof video (2.36 MB, video/webm)
2016-07-26 22:04 UTC, dberg918
Details

Note You need to log in before you can comment on or make changes to this bug.
Description dberg918 2016-07-24 00:40:54 UTC
I have run into this bug a number of times and managed to fix it in some way or another, but I'm having problems again so I thought I'd file an issue.

When I click on a Page right after opening skrooge, it asks me if I want to update the Page I'm leaving with its current state. If I check the box "Don't ask me again," I don't see that pop-up anymore for that session. However, if I reopen skrooge, the pop-up shows up again. It seems as if the config settings aren't being saved between sessions.

I have a skrooge file that is approximately 3 years old, so it has seen a number of different skrooge versions. Perhaps this has something to do with it?
Comment 1 Stephane MANKOWSKI 2016-07-25 16:41:03 UTC
Hi,

Could you launch this from a command line?
find ~/ -name skroogerc

You should find only one file in .config.
If you have an other one, remove it.

Tell me if this works well after that.
Comment 2 dberg918 2016-07-26 00:18:11 UTC
Created attachment 100305 [details]
Anonymized test file

Anonymized SKG file that may have broken page states saved in it.
Comment 3 dberg918 2016-07-26 00:20:22 UTC
Thanks for your reply.

Actually, I have tried removing ~/.config/skroogerc a number of times in the past. It works the first time I open Skrooge after removal--the popup windows go away--but once it writes a new config file and I close the application and reopen it, the problem comes back.

There are a few options about saving page states in Skrooge's settings, but they don't seem to work. One is "Update modified pages on close" and the other is "Update modified bookmarks on close." I have both set to "Never." I don't think those are related to my issue though, because those options sound like they affect behavior when closing the whole application, not just a page or a bookmark within the application itself.

What I find strange is that I haven't even changed the state of any of these pages, but Skrooge thinks they've changed and asks me to save the "modified" state. Based on my observation, it seems that page states are actually saved into the SKG file and not the skroogerc file, which leads me to believe there's something wrong with my SKG file. I have attached an anonymized version with which you may be able to reproduce the issue.

This is occurring on a newly compiled 2.4.0 package in Fedora 24. On Arch Linux, which I have installed on another computer, I don't get the popups, but I can see that Skrooge thinks the page states are all modified on startup.
Comment 4 Stephane MANKOWSKI 2016-07-26 06:13:41 UTC
(In reply to dberg918 from comment #3)
> Thanks for your reply.
> 
> Actually, I have tried removing ~/.config/skroogerc a number of times in the
> past. It works the first time I open Skrooge after removal--the popup
> windows go away--but once it writes a new config file and I close the
> application and reopen it, the problem comes back.
I asked you to remove the other skroogerc file not this one.
In fact, for compatibility reason, the old skroogerc file (for KDE4) is copied in ~/.config/skroogerc (KF5).
So, if you still have the old version, when skrooge is started, the old version overwrite the new one.

So, please, remove the other skroogerc file as explained in my previous post.
> 
> There are a few options about saving page states in Skrooge's settings, but
> they don't seem to work. One is "Update modified pages on close" and the
> other is "Update modified bookmarks on close." I have both set to "Never." I
> don't think those are related to my issue though, because those options
> sound like they affect behavior when closing the whole application, not just
> a page or a bookmark within the application itself.
No, these options are exactly done to do what you want.
But, if another skroogerc file exists, the this file will overwrite the official one and these options will come back to the previous value.
> 
> What I find strange is that I haven't even changed the state of any of these
> pages, but Skrooge thinks they've changed and asks me to save the "modified"
> state. Based on my observation, it seems that page states are actually saved
> into the SKG file and not the skroogerc file, which leads me to believe
> there's something wrong with my SKG file. I have attached an anonymized
> version with which you may be able to reproduce the issue.
OK, I will check with your file this evening.
> 
> This is occurring on a newly compiled 2.4.0 package in Fedora 24. On Arch
> Linux, which I have installed on another computer, I don't get the popups,
> but I can see that Skrooge thinks the page states are all modified on
> startup.
Comment 5 dberg918 2016-07-26 07:15:17 UTC
(In reply to Stephane MANKOWSKI from comment #4)
> I asked you to remove the other skroogerc file not this one.
> In fact, for compatibility reason, the old skroogerc file (for KDE4) is
> copied in ~/.config/skroogerc (KF5).
> So, if you still have the old version, when skrooge is started, the old
> version overwrite the new one.
> 
> So, please, remove the other skroogerc file as explained in my previous post.
Ah, my apologies. Yes, I used to have the .kde4/../skroogerc file, but I deleted that folder after I had migrated away from KDE4 apps.
>
> No, these options are exactly done to do what you want.
> But, if another skroogerc file exists, the this file will overwrite the
> official one and these options will come back to the previous value.
Oh, okay. This must be an issue with my SKG file then. I've been using this particular file since before the transition to KF5/Qt5, so it has seen a lot of changes!
Comment 6 Stephane MANKOWSKI 2016-07-26 21:19:21 UTC
Hi,

I am not able to reproduce the issue with your file.
In settings, options are well on "Never", so there is no panel asking to save the state.

Could you launch the following command and confirm that you have only one file found?
find ~/ -name skroogerc
Comment 7 dberg918 2016-07-26 22:04:55 UTC
Created attachment 100323 [details]
Proof video

Video showing the bug in action
Comment 8 dberg918 2016-07-26 22:08:27 UTC
Interesting. Can you confirm that all pages are showing a modified state even after just opening the file? That shouldn't happen, should it?

I have attached a video showing the results of the command you mentioned, as well as the popups. You can also see that each tab has the "save" icon on it, even after it has just been opened.
Comment 9 Stephane MANKOWSKI 2016-08-08 20:25:26 UTC
(In reply to dberg918 from comment #8)
> Interesting. Can you confirm that all pages are showing a modified state
> even after just opening the file? 
I confirm.
> That shouldn't happen, should it?
> 
> I have attached a video showing the results of the command you mentioned, as
> well as the popups. You can also see that each tab has the "save" icon on
> it, even after it has just been opened.
The behavior of the video is not normal.
I don't understand why you have this.

Could you confirm that you have the following lines in your skroogerc file?
[Notification Messages]
updateBookmarkOnClose=false
updateContextOnClose=false
Comment 10 Stephane MANKOWSKI 2016-08-08 20:26:57 UTC
Could you also check the box in confirmation panel to avoid to be asked again and again?
Comment 11 dberg918 2016-08-09 01:06:20 UTC
(In reply to Stephane MANKOWSKI from comment #9)
> Could you confirm that you have the following lines in your skroogerc file?
> [Notification Messages]
> updateBookmarkOnClose=false
> updateContextOnClose=false
I don't have these lines in my skroogerc file, but I do have these:
[Main Panel]
update_modified_bookmarks=2
update_modified_contexts=2

Even after checking the boxes in the confirmation panel for the Dashboard bookmark and the other Page states, those lines you mentioned don't show up.

I tried adding them manually to the bottom of my skroogerc file and after opening the application, they moved up to a different spot in the file, but they had no effect on the behavior of the popups. Checking the boxes doesn't change the behavior either. It works during one session, but then after reopening the application, the popups come back.
Comment 12 Stephane MANKOWSKI 2016-08-17 06:48:30 UTC
Hi,

For your information, another user has the same kind of issue.
I am analyzing with him because hes is french (like me) and on ubuntu (I am on kubuntu):
https://forum.ubuntu-fr.org/viewtopic.php?pid=21569431#p21569431

But, I am still not able to reproduce this issue.
Comment 13 Stephane MANKOWSKI 2016-08-22 18:40:31 UTC
Git commit a55de18a80f02c1631850975ca7da758dccb7dc3 by Stephane Mankowski.
Committed on 22/08/2016 at 18:40.
Pushed by smankowski into branch 'master'.
Skrooge perpetually asks me to save updated tab state
Corrected by a previous commit

M  +1    -0    CHANGELOG

http://commits.kde.org/skrooge/a55de18a80f02c1631850975ca7da758dccb7dc3