lionheart / openradar-mirror

A mirror of radars pulled from http://openradar.me/.
244 stars 17 forks source link

35164051: SKStoreReviewController prompt can be displayed multiple times #18739

Open openradar-mirror opened 7 years ago

openradar-mirror commented 7 years ago

Description

Summary: SKStoreReviewController doesn’t prevent multiple requests for review from producing multiple presentations of the alert dialog.

In the (contrived) example attached, an application opts to present the review prompt after a process has completed, such as posting a photo or refreshing a timeline. Another process completes momentarily that also results in the application requesting a review. If the first prompt is still on screen, the second prompt will be displayed over it.

Steps to Reproduce:

  1. Request a review with SKStoreReviewController.requestReview
  2. Shortly after, request a review again
  3. Observe multiple dialogs are displayed

Expected Results: The system should track if it is already presenting a review prompt and not display multiple instances of the prompt.

Actual Results: Multiple instances of the review prompt are instantiated when multiple requests for review are sent to StoreKit.

Version: 11.0

Notes: Based on reading the SKStoreReviewController.requestReview() documentation, this issue may be limited to ad-hoc and developer-signed builds.

https://developer.apple.com/documentation/storekit/skstorereviewcontroller/2851536-requestreview

See the attached project for an example that demonstrates this behavior in the iOS 11 simulator. This behavior is also observed in Adhoc builds. It has not been tested with App Store builds.

- Product Version: 11.0 Created: 2017-10-25T00:53:38.789240 Originated: 2017-10-24T19:53:00 Open Radar Link: http://www.openradar.me/35164051

openradar-mirror commented 7 years ago

Modified: 2017-11-03T16:17:43.071060

openradar-mirror commented 7 years ago

Modified: 2017-11-03T16:17:43.071060

openradar-mirror commented 7 years ago

Modified: 2017-11-03T16:17:43.071060

openradar-mirror commented 7 years ago

Modified: 2017-11-03T16:17:43.071060