sharky555
/
Logchecker
1
0
Vork 0
Code Kwesties 6 Pull-aanvragen 0 Publicaties 18 Wiki Activiteit
Labels Mijlpalen
Nieuw probleem

#6 Don't delete CUETools DB Plugin info section in non-checksum EAC logs

Open
opened 6 jaren geleden by itismadness · 0 comments
itismadness commented 6 jaren geleden (Gemigreerd van github.com)

The logchecker currently deletes all sections that have the “CUETools” byline if it’s for a log that doesn’t have a checksum:
b38c25a13d/src/Logchecker.php (L338-L342)

We should not be doing this, but rather leave it as part of the analyzed log.

The logchecker currently deletes all sections that have the "CUETools" byline if it's for a log that doesn't have a checksum: https://github.com/OPSnet/Logchecker/blob/b38c25a13d37243ce170ab5c7d76436c48c1c363/src/Logchecker.php#L338-L342 We should not be doing this, but rather leave it as part of the analyzed log.
Log in om deel te nemen aan deze discussie.
Geen Branch/Tag gespecificeerd
Branches Labels
Labels
Labels toepassen
Verwijder labels
bug
Something isn't working duplicate
This issue or pull request already exists enhancement
New feature or request good first issue
Good for newcomers help wanted
Extra attention is needed invalid
This doesn't seem right question
Further information is requested wontfix
This will not be worked on
Geen label
bug
duplicate
enhancement
good first issue
help wanted
invalid
question
wontfix
Mijlpaal
Stel mijlpaal in
Verwijder mijlpaal
Geen items
Geen mijlpaal
Toegewezen aan
Gebruikers toewijzen
Verwijder toegewezen aan
Niet toegewezen
1 deelnemers
Notificaties
Vervaldatum

Geen vervaldatum ingesteld.

Afhankelijkheden

Deze kwestie heeft momenteel geen afhankelijkheden.

Schrijf Voorbeeld
Laden…
Annuleren
Opslaan
Er is nog geen inhoud.
Verwijder branch '%!s(MISSING)'

Het verwijderen van een branch is permanent. Het KAN NIET ongedaan gemaakt worden. Wil je toch doorgaan?

Nee
Ja