ReadYouApp / ReadYouApp/ReadYou

Line and column numbers are swapped in error messages

Open
#1,291 1 comment 0 reactions 0 assignees View on GitHub

Nobody has claimed this yet.

bug untriaged
Dominant language
Kotlin
Stars
7.5k
Forks
330
PR merge metrics
No merged PRs in 30d

Description

Checklist
  • I've already searched in existing issues and haven't found a similar problem reported.
  • I'm using the latest version of the app
  • This error caused the app to crash
  • This error is reproducible
Which category best fits this error?

User Interface

App version

0.16.2

Reproduction steps
  1. Paste a URL to an invalid RSS feed
  2. The app shows a parsing error, but the line and column numbers are swapped, eg. error is on line 3 column 138, but it is shown as being on line 138, column 3
Image

[!NOTE]
The feed at https://tpaau.dev/feed.xml will be likely modified before someone reviews this, so I'm attaching the contents of the feed at the time of writing.

feed.xml

Crash report
n/a

Contributor guide

No contributing guide indexed for this repository

First steps

  1. Read the whole issue, then the project's contributing guide.
  2. Comment on the issue to say you are picking it up — it saves two people doing the same work.
  3. Fork the repository and make your change on a branch.
  4. Open a pull request that references the issue number.

Research direction

Start by reproducing the parsing error in the Android app with the attached feed.xml and compare the reported line and column values with the displayed message. Trace the parsing-error presentation from the invalid RSS feed to the UI; done means the line and column numbers appear in the correct order.

Written by the indexing model from the issue text.

Assessment

Tech stack
kotlin
Domain
mobile
Issue type
Bug
Difficulty
3/5
Estimated time
1-2 days
Activity status
Quiet
Clarity
Mostly clear
Newbie friendliness
58/100

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.