mirror of
https://github.com/koel/koel.git
synced 2026-04-26 01:06:00 +03:00
[GH-ISSUE #1651] Can't create Smart playlist #903
Labels
No labels
Authentication
Dependencies
Documentation
Feature Request
Flac
Help Wanted
Installation/Setup
Integration
Mobile
PR Welcome
Pending Release
Performance
Playlist
S3
Search
Sync
[Pri] Low
[Pri] Normal
[Status] Keep Open
[Status] Needs Author Reply
[Status] Needs Review
[Status] Stale
[Status] Will Implement
[Type] Blessed
[Type] Bug
[Type] Duplicate
[Type] Enhancement
[Type] Help Request
[Type] Question
[Type] Task
pull-request
No milestone
No project
No assignees
1 participant
Notifications
Due date
No due date set.
Dependencies
No dependencies set.
Reference
starred/koel-koel#903
Loading…
Add table
Add a link
Reference in a new issue
No description provided.
Delete branch "%!s()"
Deleting a branch is permanent. Although the deleted branch may continue to exist for a short time before it actually gets removed, it CANNOT be undone in most cases. Continue?
Originally created by @h4kuna on GitHub (Jan 10, 2023).
Original GitHub issue: https://github.com/koel/koel/issues/1651
Hello,
Describe the bug
I can't create "Smart playlist". I got response "Something went wrong. Please try again.".
To reproduce
Steps to reproduce the behavior:
Expected behavior
Save smart list
Environment
Additional context
Info from log
@phanan commented on GitHub (Feb 25, 2023):
I can't reproduce the error. Can you provide the exact input for step 4?
@h4kuna commented on GitHub (Mar 6, 2023):
Yes, here are screens.
@h4kuna commented on GitHub (Mar 6, 2023):
Could be wrong any database migration between koel versions? I started with koel 6.8.
@phanan commented on GitHub (Mar 6, 2023):
Hmm, I can't reproduce the bug with the provided input. What's your DB? Also, what happens if you run
php artisan db:migrate?@h4kuna commented on GitHub (Mar 6, 2023):
I use docker container, with user www-data
docker exec --user www-data -it koel-koel-1 bash.I haven't command
db:migrateis it ok? The artisan suggestmigrateonly.Result is
Nothing to migrate.@phanan commented on GitHub (Mar 6, 2023):
According to the log it's trying to assign a
floatID into the smart playlist rule's ID which only acceptsint?. What confuses me is, how can it be a float? If you look at the request from within the dev tool, what's the payload?@h4kuna commented on GitHub (Mar 6, 2023):
The id in payload is
1678109062462. Maybe, the good point is, I am running koel on raspbery ... 32bit. On 32bit system if you want keep value, it must to use float. I had same trouble with read file size.I will reinstall the system in half a year on 64bit.
OT: How you can see the manifest does not exist, how can I fix it? By artisan?
@phanan commented on GitHub (Mar 6, 2023):
Ah, indeed, 32-bit OS would have this problem.
For manifest, you can run
cp public/manifest.json.example public/manifest.json.@h4kuna commented on GitHub (Mar 6, 2023):
I update environment
Environment server:
Koel version v6.11.0
OS: Raspbery 32bit
Client environment:
OS: Ubuntu 64bit
Browser: Chrome 108
OS for client is not important! Now in retrospect I see my mistake.
@rxerium commented on GitHub (Apr 16, 2023):
Hey, did we confirm what the issue is here? I'm having the same issue and have tried the 64bit ubuntu 22.04 OS.
@phanan commented on GitHub (Apr 16, 2023):
The original issue seems to be in the browser sending a float value as the rule ID instead of the expected int type in a 32-bit env, so yours might be another issue. In retrospect, this value should be a UUID anyway. I will try to find some time to fix it in a next version.
@rxerium commented on GitHub (Apr 16, 2023):
thanks @phanan. Also more than happy to provide further details to help when resolving this bug :)
@phanan commented on GitHub (Apr 16, 2023):
Please follow what was suggested in this thread and see if it's the same problem and if not open another issue :)
@phanan commented on GitHub (Apr 17, 2023):
Closed in
81748676ee.