Merge pull request #409 from wordpress-mobile/fix/19965-classic-block-media-crash

hassaanelgarem · web-flow · commit df58ab80d154 · 2023-05-14T01:29:40.000+03:00
Fix crash when displaying an empty view
diff --git a/Example/Podfile.lock b/Example/Podfile.lock
@@ -1,5 +1,5 @@
 PODS:
-  - WPMediaPicker (1.8.7)
+  - WPMediaPicker (1.8.8-beta.1)
 
 DEPENDENCIES:
   - WPMediaPicker (from `../`)
@@ -9,7 +9,7 @@ EXTERNAL SOURCES:
     :path: "../"
 
 SPEC CHECKSUMS:
-  WPMediaPicker: 0d45dfd7b3c5651c5236ffd48c1b0b2f60a2d5d2
+  WPMediaPicker: 3b0d7272bec6eb0d8bc267daa5606c9b259e6cbb
 
 PODFILE CHECKSUM: 31590cb12765a73c9da27d6ea5b8b127c095d71d
 
diff --git a/Pod/Classes/WPMediaPickerViewController.m b/Pod/Classes/WPMediaPickerViewController.m
@@ -696,7 +696,8 @@ - (UIViewController *)emptyViewController
     if ([self usingEmptyViewController]) {
         _emptyViewController = [self.mediaPickerDelegate emptyViewControllerForMediaPickerController:self];
     }
-    else {
+
+    if (_emptyViewController == nil) {
         _emptyViewController = self.defaultEmptyViewController;
     }
     
diff --git a/WPMediaPicker.podspec b/WPMediaPicker.podspec
@@ -2,7 +2,7 @@
 
 Pod::Spec.new do |s|
   s.name          = 'WPMediaPicker'
-  s.version       = '1.8.7'
+  s.version       = '1.8.8-beta.1'
 
   s.summary       = 'WPMediaPicker is an iOS controller that allows capture and picking of media assets.'
   s.description   = <<-DESC

Original file line number	Diff line number	Diff line change
`@@ -696,7 +696,8 @@ - (UIViewController *)emptyViewController`
`696`	`696`	`if ([self usingEmptyViewController]) {`
`697`	`697`	`_emptyViewController = [self.mediaPickerDelegate emptyViewControllerForMediaPickerController:self];`
`698`	`698`	`}`
`699`		`- else {`
	`699`	`+`
	`700`	`+ if (_emptyViewController == nil) {`
`700`	`701`	`_emptyViewController = self.defaultEmptyViewController;`
`701`	`702`	`}`
`702`	`703`