WordPress-iOS icon indicating copy to clipboard operation
WordPress-iOS copied to clipboard

NSRangeException: Attempted to scroll the table view to an out-of-bounds row (3) when there are only 3 rows in sect...

Open sentry[bot] opened this issue 1 year ago • 1 comments

Sentry Issue: JETPACK-IOS-193E

NSRangeException: Attempted to scroll the table view to an out-of-bounds row (3) when there are only 3 rows in section 0. Table view: <UITableView: 0x12a7a1000; frame = (0 0; 414 566); clipsToBounds = YES; gestureRecognizers = <NSArray: 0x2832a4000>; layer = <CALayer: 0x283820800>; contentOffset: {0, 0}; contentSize: {414, 858.33333333333326}; adjustedContentInset: {0, 0, 0, 0}; dataSource: <WPTableViewHandler: 0x280c53960>>
  File "ReaderCommentsViewController.m", line 887, in -[ReaderCommentsViewController navigateToCommentIDIfNeeded]
  File "ReaderCommentsViewController.m", line 847, in -[ReaderCommentsViewController refreshTableViewAndNoResultsView]
  ?, in WPContentSyncHelper.syncContentWithUserInteraction
  File "<compiler-generated>", in WPContentSyncHelper.syncContentWithUserInteraction
  File "<compiler-generated>", in thunk for closure
...
(50 additional frame(s) were not displayed)

This is currently only affecting one user in Jetpack iOS 24.2.

sentry[bot] avatar Feb 22 '24 10:02 sentry[bot]

Assigning this a Medium priority since it's a crash (high severity) that's affecting a very low number of users (low impact).

I'm not sure why this happened in 24.2, seeing that the stack trace points to code that has been left untouched for a while. We could probably prevent the crash by ensuring that the index path exists in the table view before telling it to scroll.

https://github.com/wordpress-mobile/WordPress-iOS/blob/a96f685837f8d8b62a1b3614f96d8b3a3ccb3c59/WordPress/Classes/ViewRelated/Reader/Comments/ReaderCommentsViewController.m#L887

dvdchr avatar Feb 22 '24 10:02 dvdchr

Closing as this is a duplicate of #20742

dvdchr avatar Mar 01 '24 11:03 dvdchr