Promoveer ook picture type 255 naar de albumhoes #60

New issue

Closed

opened 2026-08-06 09:45:31 +02:00 by roelof · 1 comment

roelof commented

2026-08-06 09:45:31 +02:00

Owner

Na de albumreparatie zitten er nog 85 tracks over 9 platen zonder FrontCover. Drie van die negen — 27 tracks, allemaal REO Speedwagon — dragen de hoes wel degelijk in het bestand, maar onder ID3-picture-type 255:

01 - Let Me Ride.mp3
   type=255 (255)  mime=image/jpeg  desc='Portada'  bytes=136779
   type=Artist (8)  mime=image/jpeg  desc='Artist'  bytes=455056
   type=Band (10)   mime=image/jpeg  desc='Artist background'  bytes=614456

255 staat niet in de ID3-standaard; TagLib geeft die waarde terug voor een type-byte die hij niet kent. Met mimetype image/jpeg, beschrijving Portada en 137 KB is het onmiskenbaar de hoes — dezelfde situatie als een plaatje getypeerd Other, alleen met een andere niet-uitspraak van de tagger.

Mp3Service.Promotable accepteert nu alleen PictureType.Other:

return pictures
    .Where(picture => picture.Type == TagLib.PictureType.Other)
    .OrderByDescending(picture => picture.Data.Count)
    .FirstOrDefault();

Voorstel

Ook (TagLib.PictureType)255 meenemen, maar alleen met een image/-mimetype. Die voorwaarde is wat het eerlijk houdt: 255 is bij TagLib ook de uitkomst voor iets dat helemaal geen plaatje is, en zo'n frame mag nooit de hoes worden.

Wat er níét bij hoeft: de bestaande wacht dat Promotable null antwoordt zodra er al een FrontCover in zit, dekt de 65 bestanden met FrontCover+255+Artist+Band al af — die veranderen niet.

BackCover, Media en de rest blijven buiten schot, om de reden die er al staat: dat heeft de tagger wél gezegd.

Wat het oplevert

tracks	plaat
10	REO Speedwagon — Hi Infidelity Then Again Live
9	REO Speedwagon — You Can Tune a Piano, But You Can't Tune a Fish
8	REO Speedwagon — Two

27 tracks, geen enkel netwerkverzoek, één scanronde. Daarna blijven er zes platen over waarvoor MusicBrainz, fanart.tv noch Cover Art Archive een hoes heeft — die moeten met de hand.

Nog aan te passen

CLAUDE.md zegt onder Album covers and years: "Other is the only type promoted." Die regel klopt dan niet meer en moet het waarom van 255 erbij krijgen.

Na de albumreparatie zitten er nog 85 tracks over 9 platen zonder `FrontCover`. Drie van die negen — 27 tracks, allemaal REO Speedwagon — **dragen de hoes wel degelijk in het bestand**, maar onder ID3-picture-type `255`: ``` 01 - Let Me Ride.mp3 type=255 (255) mime=image/jpeg desc='Portada' bytes=136779 type=Artist (8) mime=image/jpeg desc='Artist' bytes=455056 type=Band (10) mime=image/jpeg desc='Artist background' bytes=614456 ``` 255 staat niet in de ID3-standaard; TagLib geeft die waarde terug voor een type-byte die hij niet kent. Met mimetype `image/jpeg`, beschrijving `Portada` en 137 KB is het onmiskenbaar de hoes — dezelfde situatie als een plaatje getypeerd `Other`, alleen met een andere niet-uitspraak van de tagger. `Mp3Service.Promotable` accepteert nu alleen `PictureType.Other`: ```csharp return pictures .Where(picture => picture.Type == TagLib.PictureType.Other) .OrderByDescending(picture => picture.Data.Count) .FirstOrDefault(); ``` ## Voorstel Ook `(TagLib.PictureType)255` meenemen, **maar alleen met een `image/`-mimetype**. Die voorwaarde is wat het eerlijk houdt: 255 is bij TagLib ook de uitkomst voor iets dat helemaal geen plaatje is, en zo'n frame mag nooit de hoes worden. Wat er níét bij hoeft: de bestaande wacht dat `Promotable` null antwoordt zodra er al een `FrontCover` in zit, dekt de 65 bestanden met `FrontCover+255+Artist+Band` al af — die veranderen niet. `BackCover`, `Media` en de rest blijven buiten schot, om de reden die er al staat: dat heeft de tagger wél gezegd. ## Wat het oplevert | tracks | plaat | |---:|---| | 10 | REO Speedwagon — Hi Infidelity Then Again Live | | 9 | REO Speedwagon — You Can Tune a Piano, But You Can't Tune a Fish | | 8 | REO Speedwagon — Two | 27 tracks, geen enkel netwerkverzoek, één scanronde. Daarna blijven er zes platen over waarvoor MusicBrainz, fanart.tv noch Cover Art Archive een hoes heeft — die moeten met de hand. ## Nog aan te passen `CLAUDE.md` zegt onder **Album covers and years**: "**`Other` is the only type promoted.**" Die regel klopt dan niet meer en moet het waarom van 255 erbij krijgen.

roelof referenced this issue from a commit

2026-08-06 09:55:05 +02:00

Een hoes zonder geldig type is ook een hoes

roelof commented

2026-08-06 09:56:26 +02:00

Author

Owner

Opgepakt op branch hoes-type-255 — vergelijken met main.

Commit: 89a6a65 — Een hoes zonder geldig type is ook een hoes. Sluit dit issue bij de merge.

Opgepakt op branch [`hoes-type-255`](https://forgejo.ridderman.nl/KavalirOS/RockHeaven/src/branch/hoes-type-255) — [vergelijken met main](https://forgejo.ridderman.nl/KavalirOS/RockHeaven/compare/main...hoes-type-255). Commit: [`89a6a65`](https://forgejo.ridderman.nl/KavalirOS/RockHeaven/commit/89a6a65) — *Een hoes zonder geldig type is ook een hoes*. Sluit dit issue bij de merge.

roelof referenced this issue from a pull request that will close it,

2026-08-06 09:56:57 +02:00

Een hoes zonder geldig type is ook een hoes #61

roelof added reference hoes-type-255

2026-08-06 09:58:05 +02:00

roelof closed this issue

2026-08-06 09:58:24 +02:00