Page MenuHomekolab.org

TypeError: an integer is required / T6989 Bugs logged for pykolab
ClosedPublic

Authored by ghane on Sep 30 2022, 1:10 PM.

Details

Summary
res.append(c)

Just remove chr()
c is single character of a string, but not an integer value of a character.
I could see via grep, decode(s) is used with folders in modules used for imap

for c in s:

See also https://git.kolab.org/diffusion/P/browse/pykolab-0.7/pykolab/imap_utf7.py line 52 , 74

Diff Detail

Repository
rP pykolab
Lint
Automatic diff as part of commit; lint not applicable.
Unit
Automatic diff as part of commit; unit tests not applicable.

Event Timeline

ghane requested review of this revision.Sep 30 2022, 1:10 PM
ghane created this revision.
sicherha accepted this revision.Sep 30 2022, 2:07 PM
This revision is now accepted and ready to land.Sep 30 2022, 2:07 PM
This revision was automatically updated to reflect the committed changes.
ghane updated this revision to Diff 11060.Oct 7 2022, 11:11 AM

I ran into an integer error on other tests,
so I corrected this with this patch.

sicherha reopened this revision.Oct 16 2022, 9:46 AM
This revision is now accepted and ready to land.Oct 16 2022, 9:46 AM
sicherha accepted this revision.Oct 16 2022, 9:46 AM
This revision was automatically updated to reflect the committed changes.