AbanteCart Community
Shopping Cart Operations => Support => Topic started by: Nimitz1061 on July 02, 2012, 10:23:32 AM
-
I have not found any instructions for doing a manual upgrade of an existing site.
Given that Abantecart does track install and upgrade activities, there would seem to be more steps than simply uploading the files and running the sql file. Any pointers???
David
-
It is simply uploading file over (replacing) and running SQL. That is exactly what happening under the hood, when you do the upgrade in the admin.
Unfortunately, we do not have instructions for manual upgrade at this time. It is good point to have it and we will create it.
-
Thanks. I attempted exactly that.
Following the file upload I had to modify the SQL, as it had a table prefix which was not present in my 1.0.2 installation.
Once that was done the SQL worked fine - but the DIR_SYSTEM definition is NOT recognized, so all paths are broken.
David
-
Please check that FTP does put all files correctly and did not fail in the middle.
-
I can confirm that FTP reported successful completion of the repeat upload twice and the issue remains.
David
-
Any way we can check what you have ? Can you PM temporary FTP access?
-
I don't think that will be necessary at this time. I may have found the issue.
In 1.0.3, in lib/error.php a change was made in line 96 from:
$log = new ALog('system/logs/error.txt');
to
$log = new ALog(DIR_SYSTEM.'logs/error.txt');
lib/error.php is included into init.php at line 52.
DIR_SYSTEM is defined in init.php at line 275 or so.
So, this would seem to be a valid error, that should be expected on at least some PHP installations in the absence of some undocumented configuration settings..
David
-
What is strange, that DIR_SYSTEM is not new. It has been there in init.php since the beginning.
I wonder why you had it missing.
Glad you resolved it.
-
It is NOT missing.
A developer added a new usage of DIR_SYSTEM in the lib/error.php file during the 1.0.3 development cycle.
lib/error.php is included into init.php BEFORE DIR_SYSTEM is defined. Every new install or upgrade should now have this problem.
David
-
Yes. This is a valid issue. We will update the code.
Thank you.