search

deftk / OpenWebWeaver-Android

Unofficial WebWeaver® app for Android
GNU General Public License v3.0
10 stars 1 forks source link

Mail Index keeps blank #9

Open wi-wissen opened 2 years ago

wi-wissen commented 2 years ago

Great app! In Saxony, the platform is called LernSax and I was able to log in with it as well. When I open my mails, this remains empty and a shortend error appears

Failed parse JSON int

The mail seems to be from Apple and is thus coded in HTML, if that helps.

deftk commented 2 years ago

Thanks for your feedback! Just to get this clear: Does this happen when you try to see the list of your emails or if you explicitly click on that email from Apple? Are you able to see other emails (maybe from other folders; should be possible to change the folder at the app bar at the top)?

wi-wissen commented 2 years ago

This happens when I try to see the list of my emails. If I switch vom INBOX to SENT I see all emails in this folder.

deftk commented 2 years ago

I guess the error appears inside a greyish text box at the bottom. What is the full text? Just "Failed parse JSON int" or is more text at the beginning/end? It would be very helpful to have the full error message (german/english).

Right now I am guessing this error itself is not from the app but rather reported from the server (however this could be wrong). At least I was not able to find the error string inside the APK file. But from my knowledge this doesn't make sense: The app doesn't send any integers (or rather data that is supposed to be an integer) to the server while looking up the email list, especially not using different methods for different folders.

Anyway, looking forward to your response

wi-wissen commented 2 years ago

This is weird. Attached is the error. But the mail can't be found on lernsax.de, IMAP or the iOS app.

grafik

deftk commented 2 years ago

Okay, now it's getting interesting. The error message indicates an error inside the app, more specifically while parsing the received list of emails. Each object inside this list has some attributes like subject, is_unread, is_answered and so on. I am not able to reproduce this behavior (wwschool.de), but if I artificially set any attribute to null, the error occurs. Now I have to guess which attribute could be null. I implemented a fix for every attribute that comes into question. Hopefully the error will be gone when the app gets updated (may take a few days but the new version 0.4.4 is out now on GitHub).

deftk commented 2 years ago

The new version 0.4.4 is out now on F-Droid. Does the error still appear with the new version?

wi-wissen commented 2 years ago

yes

deftk commented 1 year ago

I think I was finally able to reproduce the issue and implemented a fix for it (998325f236b32d9d9e781e8d19458ea29658d40b). Apparently - I don't know why anybody would ever do this - the date field of an email could also be 'false' instead of a specific number. This however also means that some emails won't show up at the correct position in the list, because emails are sorted by date and if no date is provided, the email will sadly just appear at the bottom. I just pushed this change to the dev branch, because there are some more pending changes that will be included in the next release. If you already want to test it, you would have to compile the app yourself or still wait a bit longer for the next release.

wi-wissen commented 1 year ago

Thank you for the information! Then I would wait for the next release. I don't develop Android apps and don't actively use the app at the moment. But I'll be happy to check it out again.