[FIX] Rank - make sorting setting PyQt6 compatible

[PrimozGodec]

Issue

Probably fixes #6229

Description of changes

Store the sorting setting as an integer and transform it to Qt.SortOrder when passed to model/wiev

Includes

• Code changes
• Tests
• Documentation

[PrimozGodec]

@markotoplak I cannot reproduce the issue you reported, but I fixed it to be PyQt6 compatible now. Can you try if the issue is solved for you?

[codecov]

Codecov Report

Merging #6301 (db73b53) into master (feb6341) will decrease coverage by 0.01%.
The diff coverage is 100.00%.

Additional details and impacted files

@@            Coverage Diff             @@
##           master    #6301      +/-   ##
==========================================
- Coverage   87.48%   87.47%   -0.01%     
==========================================
  Files         316      316              
  Lines       68043    68085      +42     
==========================================
+ Hits        59526    59556      +30     
- Misses       8517     8529      +12     

[markotoplak]

This PR does not fix the problem. I still get

/home/marko/venv310/lib/python3.10/site-packages/orangewidget/settings.py:500: UserWarning: Could not read defaults for widget <class 'Orange.widgets.data.owrank.OWRank'>
The following error occurred:

No module named 'sip'
  warnings.warn("Could not read defaults for widget {0}\n"

after opening Orange with PyQt6 onto settings saved in Orange and PyQt5. But it did fix the opposite problem (saving settings with PyQt6 and opening with PyQt5).

Then I checked what the created defaults file is like. I see sorting<94>K^@<8c>^Csip<94><8c>^N_unpickle_enum<94><93><94><8c>^LPyQt5.QtCore<94><8c> SortOrder in it....

I tried making this int even more "inty" by doing return int(enum) if isinstance(enum, int) else int(enum.value) (one additional int statement), but no, no change.

[markotoplak]

@PrimozGodec, thanks! I added a commit that just bumps the settings_version. Perhaps not needed, but I like settings structure changes noted somewhere.

[PrimozGodec]

@PrimozGodec, thanks! I added a commit that just bumps the settings_version. Perhaps not needed, but I like settings structure changes noted somewhere.

I agree with this change.