Implemented default case to extract second path component as video identifier

SvenTiigi · SvenTiigi · commit fdebe4947d30 · 2023-10-26T17:44:24.000+02:00
#75
diff --git a/Sources/Models/YouTubePlayer+Source.swift b/Sources/Models/YouTubePlayer+Source.swift
@@ -90,15 +90,6 @@ public extension YouTubePlayer.Source {
                         startSeconds: startSeconds
                     )
                 }
-            case "embed":
-                // Check if a video identifier is available
-                if let videoId = url?.pathComponents[safe: 2] {
-                    // Return video source
-                    return .video(
-                        id: videoId,
-                        startSeconds: startSeconds
-                    )
-                }
             case "c", "user":
                 // Check if a channel name is available
                 if let channelName = url?.pathComponents[safe: 2] {
@@ -107,7 +98,7 @@ public extension YouTubePlayer.Source {
                         name: channelName
                     )
                 }
-            case "v":
+            default:
                 // Check if a video identifier is available
                 if let videoId = url?.pathComponents[safe: 2] {
                     // Return video source
@@ -116,8 +107,6 @@ public extension YouTubePlayer.Source {
                         startSeconds: startSeconds
                     )
                 }
-            default:
-                break
             }
         }
         // Otherwise return nil

Original file line number	Diff line number	Diff line change
`@@ -90,15 +90,6 @@ public extension YouTubePlayer.Source {`
`90`	`90`	`startSeconds: startSeconds`
`91`	`91`	`)`
`92`	`92`	`}`
`93`		`- case "embed":`
`94`		`- // Check if a video identifier is available`
`95`		`- if let videoId = url?.pathComponents[safe: 2] {`
`96`		`- // Return video source`
`97`		`- return .video(`
`98`		`- id: videoId,`
`99`		`- startSeconds: startSeconds`
`100`		`- )`
`101`		`- }`
`102`	`93`	`case "c", "user":`
`103`	`94`	`// Check if a channel name is available`
`104`	`95`	`if let channelName = url?.pathComponents[safe: 2] {`
`@@ -107,7 +98,7 @@ public extension YouTubePlayer.Source {`
`107`	`98`	`name: channelName`
`108`	`99`	`)`
`109`	`100`	`}`
`110`		`- case "v":`
	`101`	`+ default:`
`111`	`102`	`// Check if a video identifier is available`
`112`	`103`	`if let videoId = url?.pathComponents[safe: 2] {`
`113`	`104`	`// Return video source`
`@@ -116,8 +107,6 @@ public extension YouTubePlayer.Source {`
`116`	`107`	`startSeconds: startSeconds`
`117`	`108`	`)`
`118`	`109`	`}`
`119`		`- default:`
`120`		`- break`
`121`	`110`	`}`
`122`	`111`	`}`
`123`	`112`	`// Otherwise return nil`