[GH-ISSUE #1309] Type error during sync #755

Closed
opened 2026-02-26 02:34:10 +03:00 by kerem · 9 comments
Owner

Originally created by @Derkades on GitHub (Apr 23, 2021).
Original GitHub issue: https://github.com/koel/koel/issues/1309

To reproduce

docker-compose exec koel php artisan koel:sync
Syncing media from /music

  188/1871 [▓▓░░░░░░░░░░░░░░░░░░░░░░░░░░]  10%
In getid3.lib.php line 437:
                                                                  
  decbin(): Argument #1 ($num) must be of type int, string given  

This is the first time I am trying this software so it could be caused by an invalid file in my library.

Environment

  • Koel version 5.1.0, docker image hyzual/koel
  • PHP version 8.0.3
  • Node version no idea
Originally created by @Derkades on GitHub (Apr 23, 2021). Original GitHub issue: https://github.com/koel/koel/issues/1309 **To reproduce** ``` docker-compose exec koel php artisan koel:sync Syncing media from /music 188/1871 [▓▓░░░░░░░░░░░░░░░░░░░░░░░░░░] 10% In getid3.lib.php line 437: decbin(): Argument #1 ($num) must be of type int, string given ``` This is the first time I am trying this software so it could be caused by an invalid file in my library. **Environment** - Koel version 5.1.0, docker image hyzual/koel - PHP version 8.0.3 - Node version no idea
kerem closed this issue 2026-02-26 02:34:10 +03:00
Author
Owner

@phanan commented on GitHub (Apr 26, 2021):

This looks like a getID3 issue to me, can you perhaps post it there?

For a workaround, it looks like the PHP installation is in strict mode, hence string-int coercion doesn't work.

<!-- gh-comment-id:826826479 --> @phanan commented on GitHub (Apr 26, 2021): This looks like a [getID3](https://github.com/JamesHeinrich/getID3) issue to me, can you perhaps post it there? For a workaround, it looks like the PHP installation is in strict mode, hence string-int coercion doesn't work.
Author
Owner

@bubu8941 commented on GitHub (Apr 28, 2021):

Hello, I encounter the same problem in docker type installation.

Syncing media from /music

  2798/26765 [▓▓░░░░░░░░░░░░░░░░░░░░░░░░░░]  10%libpng warning: iCCP: known incorrect sRGB profile
  5304/26765 [▓▓▓▓▓░░░░░░░░░░░░░░░░░░░░░░░]  19%libpng warning: iCCP: known incorrect sRGB profile
  5862/26765 [▓▓▓▓▓▓░░░░░░░░░░░░░░░░░░░░░░]  21%
In getid3.lib.php line 437:
                                                                  
  decbin(): Argument #1 ($num) must be of type int, string given  
                                                                  

@phanan : the bug you mentionned is here isn't it ?

So, do I have to pull another release of hyzual/koel docker image to make it work ?

And must we consider this Docker image as buggy ?

<!-- gh-comment-id:828721458 --> @bubu8941 commented on GitHub (Apr 28, 2021): Hello, I encounter the same problem in docker type installation. ``` Syncing media from /music 2798/26765 [▓▓░░░░░░░░░░░░░░░░░░░░░░░░░░] 10%libpng warning: iCCP: known incorrect sRGB profile 5304/26765 [▓▓▓▓▓░░░░░░░░░░░░░░░░░░░░░░░] 19%libpng warning: iCCP: known incorrect sRGB profile 5862/26765 [▓▓▓▓▓▓░░░░░░░░░░░░░░░░░░░░░░] 21% In getid3.lib.php line 437: decbin(): Argument #1 ($num) must be of type int, string given ``` @phanan : the bug you mentionned is [here](https://github.com/JamesHeinrich/getID3/issues/299) isn't it ? So, do I have to pull another release of hyzual/koel docker image to make it work ? And must we consider this Docker image as buggy ?
Author
Owner

@phanan commented on GitHub (Apr 28, 2021):

The Docker image isn’t buggy, nor is strictly Koel in this regard. As you
linked it yourself, the bug appears to be in getID3 and unfortunately
there’s very little we can do about it. You might consider downgrading your
PHP version if that’s an option.

On Wed 28. Apr 2021 at 21:30 bubu8941 @.***> wrote:

Hello, I encounter the same problem in docker type installation.

Syncing media from /music

2798/26765 [▓▓░░░░░░░░░░░░░░░░░░░░░░░░░░] 10%libpng warning: iCCP: known incorrect sRGB profile

5304/26765 [▓▓▓▓▓░░░░░░░░░░░░░░░░░░░░░░░] 19%libpng warning: iCCP: known incorrect sRGB profile

5862/26765 [▓▓▓▓▓▓░░░░░░░░░░░░░░░░░░░░░░] 21%

In getid3.lib.php line 437:

decbin(): Argument #1 ($num) must be of type int, string given

@phanan https://github.com/phanan : the bug you mentionned is here
https://github.com/JamesHeinrich/getID3/issues/299 isn't it ?

So, do I have to pull another release of hyzual/koel docker image to make
it work ?

And must we consider this Docker image as buggy ?


You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/koel/koel/issues/1309#issuecomment-828721458, or
unsubscribe
https://github.com/notifications/unsubscribe-auth/AB5O3UU2OY5D2OQMVWSIPPTTLBO3PANCNFSM43OU2WPQ
.

<!-- gh-comment-id:828742624 --> @phanan commented on GitHub (Apr 28, 2021): The Docker image isn’t buggy, nor is strictly Koel in this regard. As you linked it yourself, the bug appears to be in getID3 and unfortunately there’s very little we can do about it. You might consider downgrading your PHP version if that’s an option. On Wed 28. Apr 2021 at 21:30 bubu8941 ***@***.***> wrote: > Hello, I encounter the same problem in docker type installation. > > Syncing media from /music > > > > 2798/26765 [▓▓░░░░░░░░░░░░░░░░░░░░░░░░░░] 10%libpng warning: iCCP: known incorrect sRGB profile > > 5304/26765 [▓▓▓▓▓░░░░░░░░░░░░░░░░░░░░░░░] 19%libpng warning: iCCP: known incorrect sRGB profile > > 5862/26765 [▓▓▓▓▓▓░░░░░░░░░░░░░░░░░░░░░░] 21% > > In getid3.lib.php line 437: > > > > decbin(): Argument #1 ($num) must be of type int, string given > > > > > @phanan <https://github.com/phanan> : the bug you mentionned is here > <https://github.com/JamesHeinrich/getID3/issues/299> isn't it ? > > So, do I have to pull another release of hyzual/koel docker image to make > it work ? > > And must we consider this Docker image as buggy ? > > — > You are receiving this because you were mentioned. > Reply to this email directly, view it on GitHub > <https://github.com/koel/koel/issues/1309#issuecomment-828721458>, or > unsubscribe > <https://github.com/notifications/unsubscribe-auth/AB5O3UU2OY5D2OQMVWSIPPTTLBO3PANCNFSM43OU2WPQ> > . >
Author
Owner

@bubu8941 commented on GitHub (Apr 28, 2021):

Hmm, it's the PHP provided by the official Koel docker image as mentioned here.
So, must I consider to open a issue in Docker/koel project ?

<!-- gh-comment-id:828748266 --> @bubu8941 commented on GitHub (Apr 28, 2021): Hmm, it's the PHP provided by the official Koel docker image as mentioned [here](https://docs.koel.dev/#using-docker). So, must I consider to open a issue in Docker/koel project ?
Author
Owner

@Derkades commented on GitHub (Apr 28, 2021):

It might be a good idea to have both php74 and php8 images if there are still php8 related bugs

Simply changing 8.0.3 to 7.4 in the dockerfile seems to work, maybe add a build arg to build for both versions?

<!-- gh-comment-id:828759489 --> @Derkades commented on GitHub (Apr 28, 2021): It might be a good idea to have both php74 and php8 images if there are still php8 related bugs Simply changing 8.0.3 to 7.4 in the dockerfile seems to work, maybe add a build arg to build for both versions?
Author
Owner

@bubu8941 commented on GitHub (Apr 29, 2021):

@Derkades , thx, I confirm the workaround. Using php7.4 works.

<!-- gh-comment-id:828867604 --> @bubu8941 commented on GitHub (Apr 29, 2021): @Derkades , thx, I confirm the workaround. Using php7.4 works.
Author
Owner

@Hyzual commented on GitHub (May 1, 2021):

Hi, I've reverted php version to 7.4 in hyzual/koel, could you please pull hyzual/koel:latest and let me know if it solved the scanning problems ?

<!-- gh-comment-id:830692499 --> @Hyzual commented on GitHub (May 1, 2021): Hi, I've reverted php version to 7.4 in `hyzual/koel`, could you please pull [hyzual/koel:latest](https://hub.docker.com/layers/hyzual/koel/latest/images/sha256-1f0cf5a174cb141f18f296566de46331792210a7beb278578e7198db267a4b61?context=explore) and let me know if it solved the scanning problems ?
Author
Owner

@bubu8941 commented on GitHub (May 4, 2021):

Hello,

I confirm it works , thx !!

PS : sorry for the delay.
Jonathan Régnier

Le sam. 1 mai 2021 à 13:50, Joris MASSON @.***> a
écrit :

Hi, I've reverted php version to 7.4 in hyzual/koel, could you please
pull hyzual/koel:latest
https://hub.docker.com/layers/hyzual/koel/latest/images/sha256-1f0cf5a174cb141f18f296566de46331792210a7beb278578e7198db267a4b61?context=explore
and let me know if it solved the scanning problems ?


You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/koel/koel/issues/1309#issuecomment-830692499, or
unsubscribe
https://github.com/notifications/unsubscribe-auth/AT3F5DCQFJ3QOOLD46FVWIDTLRSQPANCNFSM43OU2WPQ.

<!-- gh-comment-id:832170898 --> @bubu8941 commented on GitHub (May 4, 2021): Hello, I confirm it works , thx !! PS : sorry for the delay. Jonathan Régnier Le sam. 1 mai 2021 à 13:50, Joris MASSON ***@***.***> a écrit : > Hi, I've reverted php version to 7.4 in hyzual/koel, could you please > pull hyzual/koel:latest > <https://hub.docker.com/layers/hyzual/koel/latest/images/sha256-1f0cf5a174cb141f18f296566de46331792210a7beb278578e7198db267a4b61?context=explore> > and let me know if it solved the scanning problems ? > > — > You are receiving this because you commented. > Reply to this email directly, view it on GitHub > <https://github.com/koel/koel/issues/1309#issuecomment-830692499>, or > unsubscribe > <https://github.com/notifications/unsubscribe-auth/AT3F5DCQFJ3QOOLD46FVWIDTLRSQPANCNFSM43OU2WPQ>. >
Author
Owner

@phanan commented on GitHub (May 7, 2021):

Glad it worked out!

<!-- gh-comment-id:834757128 --> @phanan commented on GitHub (May 7, 2021): Glad it worked out!
Sign in to join this conversation.
No milestone
No project
No assignees
1 participant
Notifications
Due date
The due date is invalid or out of range. Please use the format "yyyy-mm-dd".

No due date set.

Dependencies

No dependencies set.

Reference
starred/koel-koel#755
No description provided.