[BUG]: Bouncing sorting of articles when feed is fetched by shortcut

[sakkamade]

Brief description of the issue.

If the feed is being fetched by shortcut (seemingly only if shortcut comprise Ctrl) the order of articles in list is changing.

How to reproduce the bug?

1. Fetch the feed couple of times by shortcut (default is Ctrl+U)
2. See articles jumping about

What was the expected result?

The sorting to stay unchanged.

What actually happened?

It just skipping...

Other information

• OS: Arch Linux, KDE
• RSS Guard
  Version: 4.0.0 (built on Linux/x86_64)
  Revision: 68be51b
  Build date: 27/07/2021 03:02
  Qt: 5.15.2 (compiled against 5.15.2)

[martinrotter]

i was able to reproduce, no i am not, wtf :D

[sakkamade]

i was able to reproduce, no i am not, wtf :D

Well, I am not sure myself; mostly it happens, but occasionally it is not.

EDIT: It's frightening even to think how I am abusing the feed servers fetching them 30+ times per minute to find some solid steps to reproduce it.

[martinrotter]

"30+ times" - that's no load. I am quite sure that they have the feed file cached server-side. Every sane webserver does.

[sakkamade]

From what I see, the list's order must be manually changed before the fetch, e.g. title -> contents, or contents -> date, etc.
Upon pressing Ctrl+U the order seem to return to the previous set. Although I am not so sure. Most certainly.

"30+ times" - that's no load.

Yeah, I am joking ❤️

EDIT: (slightly elaborated)

[martinrotter]

@sakkamade Did more testing. Interestingly, if you hit your CTRL+ combo fast (quickly release the keys) the bug does not happen. If you, however, keep CTRL pressed for longer time (still pressed for a while after feed is updated) then the bug will trigger.

[martinrotter]

@sakkamade OK, found the issue. Holding down CTRL while "sorting" articles triggers multi-column sorting feature. In this case this is obviously unwanted. I will fix.

[martinrotter]

OK, it should now be fixed. Test and let me know.

[sakkamade]

Appears to be fixed.

Thank you.