Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Error During Conversion #42

Open
mslmn opened this issue Nov 14, 2023 · 3 comments
Open

Error During Conversion #42

mslmn opened this issue Nov 14, 2023 · 3 comments

Comments

@mslmn
Copy link

mslmn commented Nov 14, 2023

I get this error during conversion, but the conversion itself seems to work fine.

2023-11-14 17:03:49 auto-m4b  | Starting Conversion
2023-11-14 17:05:20 auto-m4b  | an error occured, that has not been caught:
2023-11-14 17:05:20 auto-m4b  | Array
2023-11-14 17:05:20 auto-m4b  | (
2023-11-14 17:05:20 auto-m4b  |     [type] => 8192
2023-11-14 17:05:20 auto-m4b  |     [message] => Return type of M4bTool\Audio\Tag::offsetUnset($offset) should either be compatible with ArrayAccess::offsetUnset(mixed $offset): void, or the #[\ReturnTypeWillChange] attribute should be used to temporarily suppress the notice
2023-11-14 17:05:20 auto-m4b  |     [file] => phar:///usr/local/bin/m4b-tool/src/library/M4bTool/Audio/Tag.php
2023-11-14 17:05:20 auto-m4b  |     [line] => 158
2023-11-14 17:05:20 auto-m4b  | )
2023-11-14 17:05:20 auto-m4b  | Finished Converting
2023-11-14 17:05:20 auto-m4b  | Putting the m4b into a folder
2023-11-14 17:05:20 auto-m4b  | Deleting duplicate mp3 audiobook folder
2023-11-14 17:05:20 auto-m4b  | Skipping making a backup
2023-11-14 17:05:20 auto-m4b  | Making sure all books are in their own folder
2023-11-14 17:05:20 auto-m4b  | Moving folders with 2 or more audiofiles to /temp/merge/ 
2023-11-14 17:05:20 auto-m4b  | Nested subfolders are BAD moving to /temp/fix
2023-11-14 17:05:20 auto-m4b  | Moving single file mp3's to /temp/merge/ 
2023-11-14 17:05:20 auto-m4b  | Moving all the single m4b books to /temp/untagged/ 
2023-11-14 17:05:20 auto-m4b  | No folders detected, next run 1m min...
@mslmn mslmn changed the title Error During Conversino Error During Conversion Nov 14, 2023
@gitrys
Copy link

gitrys commented Nov 26, 2023

I've got the same problem. File plays fine. My audiobook has 800 plus files, i understand that this tool should create the same amount of chapters, but there are significantly less chapters in the merged file.

The audiobook.chapters.txt contains all chapters (=same amount of mp3 files previously). But the .m4b file has only 95 chapters (with one loooong final chapter to the end).
ps.: this only is a problem in vlc media player, in mpc-h64 there are all 863 chapters... I checked in mediainfo, there are also the 863 chapters, but the file contains a "Timed Text", there are only 512 chapters, dont know why`s that...
ldf.txt

I basically don't know if i can work with this file, or there is a "secret corruption" that im not aware off.

Best regards

@Dracon09
Copy link

Dracon09 commented Dec 29, 2023

Look like this was fixed in the original tool. This just needs to be updated
sandreas/m4b-tool#200

@stevenffrench
Copy link

I'm running into this as well.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

4 participants