Fixed remnant bug from changed behavior in python 2 where int < None …

[JacobTerritory]

…would evaluate to False rather than a ValueError. Manifested when openning another user's user folders

[codecov]

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 32.55%. Comparing base (2ad3aea) to head (e4ef22a).
⚠️ Report is 42 commits behind head on master.

Additional details and impacted files

@@             Coverage Diff             @@
##           master     #186       +/-   ##
===========================================
+ Coverage        0   32.55%   +32.55%     
===========================================
  Files           0       68       +68     
  Lines           0     7688     +7688     
===========================================
+ Hits            0     2503     +2503     
- Misses          0     5185     +5185     

Flag	Coverage Δ
Linux	32.55% <ø> (?)
Python-3.10	32.53% <ø> (?)
Python-3.11	32.54% <ø> (?)
Python-3.13	32.54% <ø> (?)
Python-3.9	32.53% <ø> (?)
Windows	19.63% <ø> (?)
macOS	32.55% <ø> (?)

Flags with carried forward coverage won't be shown. Click here to find out more.

☔ View full report in Codecov by Harness.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:

• ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

[julien-lang]

I understand that the change of behaviour between Python 2 and 3 with Int<None and I agree this fix makes sense.

However, I am unable to find a way to reproduce this problem.

I mean, if max_publish_version is None, that means there are no file_versions with is_published. In this situation, the condition on L54 (and self.file.is_published) will never pass so we never get to this line.

Would it be possible to provide reproduction steps for this issue, please?