Detect Upgrade Script Needs to be Run

[craigbrass]

As I was saying in another thread, it would be nice if when you login to the admin area it checks to see if you have uploaded files but haven't run the database upgrade script and if this is the case, redirect you to the upgrade script.

The reason for this is I imagine a lot of people upload the files and then either forget or do not know they need to do anything more.

[Jamie Edwards]

Good idea

[Digital Mayhem]

Thing is the upgrade script is within the "setup" directory which if the "setup" directory is there you will get a red text that says you must delete the setup directory and it wont let you login. So it pretty much already does it.. granted it doesnt tell you that you haven't run the upgrade script yet or not so someone could not run the script and delete the setup directory..

[craigbrass]

That is true Steve but I think something that does as I said would be better. I imagine a fair few people submit support tickets about problems because they haven't run the upgrade script.

[Jamie Edwards]

This has already been implemented:

if (version_compare(SWIFTVERSION, $_SWIFT["settingscache"]["core"]["version"]) == -1 )
...
 echo "<center><b><font color='red' face='verdana, arial, helvetica' size='2'>[WARNING]: Database Version Tag (". $_SWIFT["settingscache"]["core"]["version"] .") and File (". SWIFTVERSION .") Mismatch!</font></b><BR /><font color='black' face='verdana, arial, helvetica' size='1'>Please update your files to the latest revision, This error happens when your database structure is different than the PHP files you are using.</font></center>\n<BR />";
...

However, this has now been tightened in the latest CVS as it is apparent that for some reason it was not triggering the warning.

[craigbrass]

Ah great. Yea, must admit I never saw anything like then when I tested just before posting this topic.

[Jamie Edwards]

Committed 9th August 2007.