This topic was marked solved by Nao, on February 20th, 2015, 11:44 AM
updateSettingsFile in install.php

CerealGuy

  • Posts: 343

Nao

  • Dadman with a boy
  • Posts: 16,013
Re: updateSettingsFile in install.php
« Reply #1, on January 6th, 2015, 11:55 AM »
While your solution is certainly more elegant than what I ultimately chose to do (and I really, really wanted to just do that include!), ultimately this is the installer we're talking about, the entry point into Wedge, and I just don't have the time to commit to testing whether using the 'default' version will work in the installer. There has to be a reason why SMF (or whatever old version of Wedge) didn't just do include('Subs-Admin.php') to load the updateSettingsFile function and instead chose to re-declare it, so I'm not going to bother for now...

Feel free to test your own patch thoroughly, i.e. go through the painful steps of installation several times, and convince me to use it... The less code, the better. :^^;:

Drunken Clam

  • Drool, drool, drool....!
  • Posts: 149

Nao

  • Dadman with a boy
  • Posts: 16,013
Re: updateSettingsFile in install.php
« Reply #3, on February 20th, 2015, 11:44 AM »
I'm guessing this is solved by then..?
I'll mark it as solved.

Also, I'll be doing a test install on the wedge.org server today, so I can see what's up with the installer. And test the media gallery properly...