pivotal-legacy / PivotalCoreKit

Shared library and test code for iOS and macOS projects
http://pivotallabs.com
Other
168 stars 85 forks source link

`completion` block is executed twice in `pck_dismissViewControllerAnimated` and should only be executed once #184

Closed xtreme-doug-boutillier closed 8 years ago

xtreme-doug-boutillier commented 8 years ago

Hi,

We're running into issues on PivotalTracker re: our ui specs and the method pck_dismissViewControllerAnimated implemented here.

This method takes a block, completion, and will execute completion twice under certain circumstances. This seems like incorrect behaviour. Is that true?

Will write a PR.