Playlist Loader: Drop more bad playlist entries

kode54 · kode54 · commit ff50a733cd97 · 2026-02-02T03:34:22.000-08:00
Do this on startup, with more stringent checks, because apparently some
people have bad playlist data still? Really weird.

Signed-off-by: Christopher Snowhill &lt;kode54@gmail.com&gt;
diff --git a/Application/AppController.m b/Application/AppController.m
@@ -514,8 +514,9 @@ - (void)applicationWillTerminate:(NSNotification *)aNotification {
 
 	NSManagedObjectContext *moc = playlistController.persistentContainer.viewContext;
 
+	// How the heck are people getting this in their playlists
 	for(PlaylistEntry *pe in playlistController.arrangedObjects) {
-		if(pe.deLeted) {
+		if(pe.deLeted || !pe.urlString || [pe.urlString isEqualToString:@""] || !pe.url) {
 			[moc deleteObject:pe];
 			continue;
 		}

Original file line number	Diff line number	Diff line change
`@@ -514,8 +514,9 @@ - (void)applicationWillTerminate:(NSNotification *)aNotification {`
`514`	`514`
`515`	`515`	`NSManagedObjectContext *moc = playlistController.persistentContainer.viewContext;`
`516`	`516`
	`517`	`+ // How the heck are people getting this in their playlists`
`517`	`518`	`for(PlaylistEntry *pe in playlistController.arrangedObjects) {`
`518`		`- if(pe.deLeted) {`
	`519`	`+ if(pe.deLeted \|\| !pe.urlString \|\| [pe.urlString isEqualToString:@""] \|\| !pe.url) {`
`519`	`520`	`[moc deleteObject:pe];`
`520`	`521`	`continue;`
`521`	`522`	`}`