Fix wrong value of playerState (#386)

denkeni · web-flow · commit 174eca2c35bd · 2020-06-24T19:10:44.000-07:00
- Fix bug introduced by e9dbe9e regarding mapping of player state. The value returned from the player has a lower-bound of "-1", whereas YTPlayerState is a 0-based NSInteger enum. - Remove redundant code
diff --git a/Classes/YTPlayerView.m b/Classes/YTPlayerView.m
@@ -326,7 +326,7 @@ - (void)playerState:(_Nullable YTPlayerStateCompletionHandler)completionHandler
       completionHandler(kYTPlayerStateUnknown, error);
       return;
     }
-    YTPlayerState state = [result intValue];
+    YTPlayerState state = [YTPlayerView playerStateForString:[result stringValue]];
     completionHandler(state, nil);
   }];
 }
@@ -579,22 +579,7 @@ - (void)notifyDelegateOfYouTubeCallbackUrl:(NSURL *) url {
     }
   } else if ([action isEqual:kYTPlayerCallbackOnStateChange]) {
     if ([self.delegate respondsToSelector:@selector(playerView:didChangeToState:)]) {
-      YTPlayerState state = kYTPlayerStateUnknown;
-
-      if ([data isEqual:kYTPlayerStateEndedCode]) {
-        state = kYTPlayerStateEnded;
-      } else if ([data isEqual:kYTPlayerStatePlayingCode]) {
-        state = kYTPlayerStatePlaying;
-      } else if ([data isEqual:kYTPlayerStatePausedCode]) {
-        state = kYTPlayerStatePaused;
-      } else if ([data isEqual:kYTPlayerStateBufferingCode]) {
-        state = kYTPlayerStateBuffering;
-      } else if ([data isEqual:kYTPlayerStateCuedCode]) {
-        state = kYTPlayerStateCued;
-      } else if ([data isEqual:kYTPlayerStateUnstartedCode]) {
-        state = kYTPlayerStateUnstarted;
-      }
-
+      YTPlayerState state = [YTPlayerView playerStateForString:data];
       [self.delegate playerView:self didChangeToState:state];
     }
   } else if ([action isEqual:kYTPlayerCallbackOnPlaybackQualityChange]) {
