search

godotengine / godot-asset-library

PHP frontend for Godot Engine's asset library
https://godotengine.org/asset-library
MIT License
284 stars 85 forks source link

Possible fix to asset approval failure #317

Closed Gramps closed 4 months ago

Gramps commented 4 months ago

This asset approval bug has been around for a while. Yuri made some minor database changes that fixed it for certain assets but there are still assets trapped behind the generic Error: An error occured while executing DB queries error. Some have been sitting for a few months.

I asked Emi to try to approve one then send me a snippet of the log file where the error exists since this has been so pervasive. Based on that, I'm guessing this may have something to do with it: Slim Application Error: Type: RuntimeException Code: 7 Message: Inf and NaN cannot be JSON encoded. So this pull-request may fix that or may not and, at the last, maybe leads someone with better access to an actual fix.

coppolaemilio commented 4 months ago

Let's give it a go

Gramps commented 4 months ago

Alas, this did not work but hopefully testing this pull sheds some light in the error logs. Going to trace the chain of functions to grasp what is happening.