-
Notifications
You must be signed in to change notification settings - Fork 211
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 - string in number_format() in sqlgen item_stats #411
Comments
My temporary solution works: |
I've built your workaround into the item_stats.func.php file, but I still get this error. Another solution available? |
no, just add the two |
I don't really understand that now. I don't really understand what you're writing now. |
So, the error message you received pointed you at the exact line in the exact file and named the function on that line where the error occurs. If you lack the basic programming knowledge to act on the provided info and are unwilling to acquire it, then this project isn't for you. There WILL be bugs in the future and i WILL be unable to fix them in a timely fashion. You will need to be able to help yourself! |
Unfortunately my knowledge is somewhat limited and I installed the workaround, but unfortunately without success. You are the developer of this project and have better knowledge about it than I do. As a user, I can only report bugs that would then have to be fixed by the project developer. It would be the same as if Windows had problems and you reported the bug and Microsoft told users to solve the problem themselves if workarounds didn't work for everyone. I don't want to sound rude, but it's not my job to fix the bugs, it's the developer of the code who understands the code better than someone who downloads the program and tries to install it according to the instructions. Not everyone knows anything about programming. |
Well no. You (supposedly) paid for Windows and are entitled to some amount of support. I'm not even claiming everything to be in working order. Hell, i'm not even really adervtising the existens of this project. This issue tracker doen't exist because i >have< to help you, but because >i myself want< my project to be in working order .. however long that may take.
You are not rude, but it also isn't my job >job< to maintain this project. It's my silly little brain exercise. An outlet for maintaining my proficiency with php. I do try to keep it in working order, but well, here we are.
Sure, but you just entered a field where it is all but required. (Also i could have sworn TC once had a text blob explaining itself as an educational project first. Or maybe that was MaNGOS.) In any case, you have two options.
|
The Windows thing was a stupid example. The same can also be done on Linux distributions, a free operating system. And people use this bug tracker to report bugs that they can't fix themselves. Now I'm approaching 50 and it's difficult for me to learn anything new. In my younger years and with a lot of time, that might have been possible. And I inserted the patch/workaround into the file where the code snippet appears. I'd like to attach this file here so you can see that I installed it, but the error still exists. I attached the file in question as a text file because you can't upload php files and you can see I applied the patch from the user, but if the error continues to appear there's nothing I can do about it. |
@Thor1423 there are two lines in that function that needs to be "fixed".
The next line is just 10 lines below.
to
As @Sarjuuk mentioned this is just a temporary fix and it's not added because the whole part of this setup is about to be rewritten as a whole. |
@peycho Thank you very much, that's it. I copied and pasted the code, but something was still missing. Thank you, that worked. |
Where is the error and how can I fix it? I've restarted the setup several times now, but it always hangs at this point.
The text was updated successfully, but these errors were encountered: