RealmResultsController icon indicating copy to clipboard operation
RealmResultsController copied to clipboard

Fixed issue where item wasn't moved to new section.

Open jallen opened this issue 9 years ago • 2 comments

:tophat: What? Why?

If an item is moved to a new section and the section doesn't exist the item is Inserted instead of moved causing the section in the tableView to not be displayed.

:family: Dependencies?

:clipboard: Developer checklist

  • [ ] Documentation
  • [x] Unit Tests
  • [ ] Update README (if necessary)

:ghost: GIF

If the section didn't exist before the item was moved the updated item would be inserted instead of moved causing the new section to not be displayed properly in the tableView.

jallen avatar Jun 20 '16 18:06 jallen

Thanks for the PR @jallen 😄 Take a look at the comment I put, and sorry for the delay, I've been really busy the past days

polqf avatar Jun 27 '16 22:06 polqf

I see where my test went wrong.

I'm now just filtering the initial set of objects. As far as I can tell this doesn't interfere with the request.

Also, maybe an initResolvedWithKeyPath method would be cleaner?

Let me know what you think.

jallen avatar Jun 28 '16 01:06 jallen