Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Index out of bounds exception in ReorderableBrickViewController.swift:107 test method #194

Open
pdtgct opened this issue Oct 5, 2017 · 1 comment

Comments

@pdtgct
Copy link

pdtgct commented Oct 5, 2017

On branch master, tag v2.4.1, current as of commit 8d815f019e3a9b90cdfcbcc7933010854661f94b

Steps to Reproduce

  • clone master
  • open the .xcworkspace
  • build and run the example app target, BrickApp-iOS
  • from the list of samples, select SIMPLE
  • in the SIMPLE list of samples, scroll down and select Reorderable Section
  • in Reorderable Section, click the + icon in the top right

Expected Result

The Xcode debugger console should print the section titles and bricks in each section.

Actual Result

Section: 0
// text omitted
Section: 1
Label 1 Index 0: 0
Label 1 Index 1: 2
fatal error: Index out of range
2017-10-04 20:14:19.723618-0400 BrickApp[34526:35108613] fatal error: Index out of range

Exception produced is at ReorderableBrickViewController.swift:107 but would also occur at line 111

@jay18001
Copy link
Contributor

jay18001 commented Oct 5, 2017

That looks like a dummy button that was left over from when we implemented reorderable bricks. I'll put up an PR tomorrow to remove it. Good find.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants