[TIMOB-28067] iOS: swipe down gallery photo picker does not fire cancel callback

GitHub Issue	n/a
Type	Bug
Priority	Critical
Status	Closed
Resolution	Fixed
Resolution Date	2020-08-25T00:40:38.000+0000
Affected Version/s	n/a
Fix Version/s	Release 9.1.0
Components	iOS
Labels	Ti.SDK, ios, ios13, media, openPhotoGallery
Reporter	Caio Perdona
Assignee	Vijay Singh
Created	2020-08-04T15:03:44.000+0000
Updated	2021-01-18T09:08:19.000+0000

Description

Create a component that picks photos from gallery with Ti.Media.openPhotoGallery(). When photo picker modal shows up, dispose it, dragging it down. It does not fire the "cancel" callback from method. If you try to open gallery picker again right after completing the above, it opens error callback. There's still no working around this. Is this a known bug? I would appreciate any help. Thanks in advance!

Attachments

File	Date	Size
SampleApp.zip	2020-08-04T15:03:12.000+0000	8752258

Comments

Vijay Singh 2020-08-05

[~perdona] Thanks for reporting. I'll check it.

Vijay Singh 2020-08-05

PR - https://github.com/appcelerator/titanium_mobile/pull/11868 How To Test - 1. Run test code, mentioned below, on iOS 13.0+ iPhone/iPad simulator. 2. Click on button 'Open Photo Gallery'. 3. In iPhone, swipe down the gallery screen to close. In iPad click outside of gallery screen. 4. Observe log message. It should show ' Photo gallery selection canceled.' Test Case -

   var win = Ti.UI.createWindow({
       backgroundColor: '#fff'
   });
   
   var btn = Ti.UI.createButton({
       title: 'Open Photo Gallery'
   });
   
   btn.addEventListener('click', function() {
       Ti.Media.openPhotoGallery({
   		success: function(e) {
   		    Ti.API.info("Photo gallery selection successful.");
   		},
   		cancel: function() {
   			Ti.API.info(" Photo gallery selection canceled.");
   		},
   		error: function() {
   			Ti.API.info("Photo gallery selection error.");
   		}
   	})
   });
   
   win.add(btn);
   win.open();

Caio Perdona 2020-08-05

Thanks a lot, @Vijay Singh
Samir Mohammed 2020-08-20

FR Passed, waiting for Jenkins build and backport.
Christopher Williams 2020-08-21

merged to master and 9_1_X. [~lchoudhary][~amukherjee] Note: that if we change the version to deploy for 9.1.0 GA from the RC, then this may get picked up.
Abir Mukherjee 2020-08-21

[~cwilliams] thanks; I changed the fix version to 9.1.0.
Lokesh Choudhary 2020-08-25

Closing.

JSON Source