CerealGuy

  • Posts: 343
Re: Class-DB changes?
« Reply #30, on October 1st, 2016, 02:15 PM »
Quote from Freñiçh on October 1st, 2016, 12:06 PM
Well i'm done with it,deleted the last revs and get back to a working forum before the new revs.....even media gallery give's me all kind of errors and troubles.

I'm sorry but I can not afford to have a malfunctioning forum.
Argument order was also wrong.

https://github.com/Wedge/wedge/pull/39/commits/58da2c6abf56a9db1d0147c69506449e0637978a

But still, it's some changes which aren't really tested yet. So i'm sure there are some more things to fix, but all changes @Nao made are definetly worth it. It just needs some time and commits :lol:.

I'm now getting `The database value you're trying to insert does not exist: current_topic` :eheh:

Nao

  • Dadman with a boy
  • Posts: 16,079
Re: Class-DB changes?
« Reply #31, on October 1st, 2016, 07:40 PM »
Merged your pr, thanks.
Can you debug that new thing? ;)

I promise I'll get to it next week. This has just bern the shittiest week ever... :(

CerealGuy

  • Posts: 343
Re: Class-DB changes?
« Reply #32, on October 2nd, 2016, 11:36 AM »
I'm on it but it needs some time. The sql stuff is not throwing an error but echoing it. So no line which is doing bad stuff. But I guess something with the assoc stuff is doing that.

Nao

  • Dadman with a boy
  • Posts: 16,079
Re: Class-DB changes?
« Reply #33, on October 2nd, 2016, 03:53 PM »
Did you enable everything in the admin area, debugging section?
Posted: October 2nd, 2016, 01:01 PM

Testing...
Nope, no errors posting for me.
Anywhere else..? I need to know where to reproduce. ;)
Posted: October 2nd, 2016, 03:21 PM

I've double-checked the error_backtrace function, and it doesn't seem to be incompatible with ::get_* functions, and I've even forced a fake error (setting an int to empty string before I pass it), and the error shows up in the error log, complete with file & line number... :-/

CerealGuy

  • Posts: 343

Nao

  • Dadman with a boy
  • Posts: 16,079
Re: Class-DB changes?
« Reply #35, on October 2nd, 2016, 11:12 PM »
I'd really really like to know on WHAT page/action you're getting the error...

CerealGuy

  • Posts: 343
Re: Class-DB changes?
« Reply #36, on October 3rd, 2016, 10:15 AM »
Got it on every ?topic request.

EDIT: Can't reproduce it with the last commit.

Nao

  • Dadman with a boy
  • Posts: 16,079
Re: Class-DB changes?
« Reply #37, on October 3rd, 2016, 08:25 PM »
Possibly a caching issue then.

Btw Wedge.org is synced to the official repo, for the first time in years... I decided that if I never committed its changes, then maybe they weren't worth it. (I'm keeping them on the side though, just in case...)

CerealGuy

  • Posts: 343
Re: Class-DB changes?
« Reply #38, on October 5th, 2016, 01:36 PM »
Yeah definetly possible. Didn't look into the wedge update mechanism, but does it force clean the cache? :hmm:

Nao

  • Dadman with a boy
  • Posts: 16,079
Re: Class-DB changes?
« Reply #39, on October 7th, 2016, 08:58 PM »
Yes, unless your file dates weren't modified.

Jurien

  • All i want is a couple days off
  • Posts: 132
Re: Class-DB changes?
« Reply #40, on October 8th, 2016, 10:57 AM »
Quote from Nao on October 1st, 2016, 07:40 PM
I promise I'll get to it next week. This has just bern the shittiest week ever... :(
You have recovered yourself,thank god for that :)....well done.I am back on the track just üploaded again all files of the latest Wedge source code repository,and guess what all went smooth.

Except for this guess it had to do something with my Dutch language (translation).

Check Detailed version.
Errors Manage Maintenance.php on line 894,896 and 897 & Admin.template.php

📎 Admin.template.php.png - 34.05 kB, 950x376, viewed 212 times.

📎 Manage Maintenance.php.png - 56.66 kB, 896x401, viewed 198 times.


Nao

  • Dadman with a boy
  • Posts: 16,079
Re: Class-DB changes?
« Reply #41, on October 8th, 2016, 01:49 PM »
It doesn't say what error it is..?
And I don't think there should be an error due to missing language strings. Wedge always defaults to English strings in that case.

Jurien

  • All i want is a couple days off
  • Posts: 132

Nao

  • Dadman with a boy
  • Posts: 16,079
Re: Class-DB changes?
« Reply #43, on October 8th, 2016, 06:45 PM »
Probably something that can be fixed by fiddling with language settings in the admin area and then resetting them. ^^

Wow, that quick reply focus hack I added works wonders on mobile :)