Working on new Installer for v1.6.0 ...

Printable View

Show 100 post(s) from this thread on one page

10 May 2013, 09:06 PM
wilt

Re: ZC 1.6 Installer

Quote:

Originally Posted by torvista

As per the previous poster, I tried this on a local install xampp 181, php 5.4.7 but not using the default configurations for php.ini nor apache httpd.conf due to previous problem solving attempts. Also using virtual hosts rather than alias.

I subscribe to the design theory: "Don't make me think" (or rather "...assume I cannot think").
It looks great and very important for first impressions.

Some nitpicking comments:

1) I note tabbing between the input fields requires an extra click every time.

2) Database setup screen: why not check for the existence of the mysql server when entering this page and flag it immediately?

3) Creating database:
I got multiple repeat error messages:
Warning: sha1_file(DIR_FS_CATALOGDIR_WS_INCLUDESconfigure.php): failed to open stream: No such file or directory in D:\My Documents\GitHub\zc-v1-series\zc_install\includes\functions\password_funcs.php on line 118

The read.md says you have to rename the configures, why? They get created and the store works without doing that.

4) I think the labels stating that you must make a note of your password and the new name of the admin folder should be more in your face.

5) There are some minor validation errors (using Total Validator) on each page.

6) Shop: alt-nav looks like the page is broken, no css is loaded.

What about detecting its a localhost install and creating local/configure.php?

Steve

Thanks for the detailed feedback Steve.

I'll address the tab order asap :)

Quote:

2) Database setup screen: why not check for the existence of the mysql server when entering this page and flag it immediately?

Not sure what you mean here. The system checking code should have already tried to see if there is a live mysql server, and if not informed you on the initial screen.

Quote:

3) Creating database:

Noted: Will check code.

Quote:

The read.md says you have to rename the configures, why? They get created and the store works without doing that.

Apologies the read.md is a little out of date, and based on old installer/v1.51

Quote:

4) I think the labels stating that you must make a note of your password and the new name of the admin folder should be more in your face.

Fair enough :)

Quote:

5) There are some minor validation errors (using Total Validator) on each page.

Double fair enough :)

Quote:

6) Shop: alt-nav looks like the page is broken, no css is loaded.

Assume you mean the admin menu alt-nav. You haven't signed an NDA so I can't tell you that there will be changes coming to the admin menu infrastructure.

Quote:

What about detecting its a localhost install and creating local/configure.php?

Nice idea. will ponder. :)
10 May 2013, 09:23 PM
Design75

Re: ZC 1.6 Installer

I just tried installing again, same result. This time I checked the apache logs, and found the following

Code:

[Fri May 10 22:12:17.468750 2013] [:error] [pid 5072:tid 500] [client 127.0.0.1:1050] PHP Warning: fopen(F:\\xampp\\htdocs\\zen4all.nl160beta/logs/progress.json): failed to open stream: Invalid argument in F:\\xampp\\htdocs\\zen4all.nl160beta\\zc_install\\includes\\classes\\class.zcDatabaseInstaller.php on line 330, referer: http://localhost/zen4all.nl160beta/zc_install/index.php?main_page=database [Fri May 10 22:12:17.468750 2013] [:error] [pid 5072:tid 500] [client 127.0.0.1:1050] PHP Warning: fwrite() expects parameter 1 to be resource, boolean given in F:\\xampp\\htdocs\\zen4all.nl160beta\\zc_install\\includes\\classes\\class.zcDatabaseInstaller.php on line 332, referer: http://localhost/zen4all.nl160beta/zc_install/index.php?main_page=database [Fri May 10 22:12:17.468750 2013] [:error] [pid 5072:tid 500] [client 127.0.0.1:1050] PHP Warning: fclose() expects parameter 1 to be resource, boolean given in F:\\xampp\\htdocs\\zen4all.nl160beta\\zc_install\\includes\\classes\\class.zcDatabaseInstaller.php on line 333, referer: http://localhost/zen4all.nl160beta/zc_install/index.php?main_page=database
10 May 2013, 10:10 PM
lhungil

Re: ZC 1.6 Installer

Quote:

Originally Posted by torvista

...
The read.md says you have to rename the configures, why? They get created and the store works without doing that.
...

Have not played with the installer yet, So apologies if this already happens during an upgrade....

I'd like to see the first step handling the "configure.php" files rename the originals if present, pull in the variables as needed while installing, write brand new "configure.php" files, and optionally delete the old configure.php when done.

This would ensure the old files are present as a "backup" from before upgrading... Make sure people have the most recent (and a clean) version of the "configure.php" (including any changes such as the automatic "admin folder location" change in 1.5)... And make it easier for people to do a manual "diff" to see what has changed (if anything) between versions.
11 May 2013, 01:19 AM
torvista

Re: ZC 1.6 Installer

Quote:

The system checking code should have already tried to see if there is a live mysql server, and if not informed you on the initial screen.

With mysl server off and my pc disconnected from the internet no page detects the absence of the mysql server...it only gets flagged after entering the db details on Database setup page and pressing Continue.

Its a small point but I am thinking brand new user here and a bit of psychology, I know I would want the initial inspection page to tell me that all is not well, but would be irritated on being informed only after filling in form fields...which no doubt would time out before I had fixed the problem.

Languages
I see the installer text has been moved to language defines...I created another to see if it would automagically offer the two choices but no!
Maybe it should detect the presence of other language files and offer the choices automatically. Pretty important for world domination I would say.
Since its only one file it should be easy to get a multi-language installer completed by the community once its finalised.

Quote:

When the databases are created and being filled with the demo data the installation program stops. The progress bar stay at almost filled, end never proceeds any further.

I had a similar problem with Apsona running locally on php 5.4.7. when importing a csv. It was due to php strict warnings (that initially did not appear) that I could not supress.

I have now tried the installer using the default xampp 1.8.1 php.ini and apache httpd.conf and it works for me using virtual hosts and/or if located in the default xampp \htdocs directory.

/local/whatever
I feel this aspect of zen cart is not well-known and a discussion for another thread which I'll kick-start in due course...
11 May 2013, 03:39 PM
Commentics

1 Attachment(s)

Re: ZC 1.6 Installer

Similar issue and error messages as 'Design75' in posts #5 and #12.

The 'Loading Demo Data' modal window permanently froze as it was near the end.

Local install of Wamp 2.2 running PHP 5.4.3, MySQL 5.5.24 and Apache 2.4.2 on Windows 7 64 bit with Firefox 20.0.1.

Only change made was to remove the password requirement in post #4.

Apache error log attached (as it exceeded the maximum character limit).

Attachment 12484
11 May 2013, 08:39 PM
wilt

Re: ZC 1.6 Installer

Quote:

Originally Posted by Commentics

Similar issue and error messages as 'Design75' in posts #5 and #12.

The 'Loading Demo Data' modal window permanently froze as it was near the end.

This is a weird error as the file in question must have been written to before if the main sql progress bar completed.

When writing the Ajax/Json progress logging initially, I would see some problems myself, but thought I had ironed them out.

It's more than likely a race condition where the attempt to write to the file is blocked by an ongoing read (due to the asynchronous nature of Ajax)

Will put some code in to test the validity of the file resource, which should hopefully mitigate the race condition.
11 May 2013, 10:33 PM
lat9

1 Attachment(s)

Re: ZC 1.6 Installer

My initial comments (the installer interface looks good!):

System Setup Screen
- Need to determine which 'version' of English is to be used: licence vs license

Database Setup
- I made the change noted in post#4 regarding the database password; I've never set one on my localhost/XAMPP testing
- What happened to SQL cache method: None?
- Database Charset dropdown, is there Latin9 (should be latin1?)

Admin Setup
- Messages regarding initial password (This is your initial temporary password for your Admin Superuser Account. Please ensure you make a note of it.) and initial directory (Your store Admin directory has been automatically renamed. Please ensure you make a note of this directory.) appear below the associated fields; I was confused initially as to which field was associated with the message and think it would flow better if the messages were either above or beside the associated field. I tend to read top->bottom, left->right (that might be different in different cultures).

Setup Finished
- The message (You need to remove the /zc_install folder so that someone can't re-install your shop again and wipe out your database! Warnings will appear until the folder has been removed ) is overly scary.
-The message "warnings will appear" is not quite correct because I can't log into the admin without the rename. A "warning" shouldn't be as insistent; the message should read "A message will appear and you will not be able to log into your admin until the folder has been removed." (or something of the ilk).

Admin main page
- Click the V on General Statistics. TEXT_CANCEL instead of Cancel

Admin Configuration Menu
- Dropdown shows Cowoa Checkout ... shouldn't it be COWOA (all caps)?
- Once in Cowoa Checkout, the page title reads "Guest Checkout" ... shouldn't it be the same as the dropdown title?
- I thought that the zen_trace function was supposed to be included in v1.6.0, but couldn't find it in the Configuration->Logging menu.

Storefront, COWOA checkout
- Some funky formatting going on with the progress bar:
Attachment 12488
12 May 2013, 05:12 PM
lat9

Re: ZC 1.6 Installer
Just a couple more questions:
1. Looking in /zc_install/sql/updates, it appears that there will be an upgrade path from 1.3.9, 1.5.0 or 1.5.1 to v1.6.0 ... but not any earlier release. I just wanted to note this for confirmation and to provide (yet another) heads-up to those store-owners still running a v1.3.8 or earlier store.
2. There's no mysql_latin1.sql file in /zc_install/sql/install; I'm assuming that that's just an oversight ...
12 May 2013, 07:15 PM
wilt

Re: ZC 1.6 Installer
Quote:
Originally Posted by lat9

Just a couple more questions:

Looking in /zc_install/sql/updates, it appears that there will be an upgrade path from 1.3.9, 1.5.0 or 1.5.1 to v1.6.0 ... but not any earlier release. I just wanted to note this for confirmation and to provide (yet another) heads-up to those store-owners still running a v1.3.8 or earlier store.
There's no mysql_latin1.sql file in /zc_install/sql/install; I'm assuming that that's just an oversight ...
There probably will be an upgrade path for older versions, although we have not yet decided how far back we will go with that path. It almost certainly won't go back to 1.2 as the old installer does.

Yes, the latin1 file was overlooked :blush:
12 May 2013, 07:24 PM
wilt

Re: ZC 1.6 Installer

Quote:

Originally Posted by lat9

My initial comments (the installer interface looks good!):

System Setup Screen
- Need to determine which 'version' of English is to be used: licence vs license

K, that's the prob with having core team spread between UK/USA/Canada. :blink:

Quote:

Database Setup
- I made the change noted in post#4 regarding the database password; I've never set one on my localhost/XAMPP testing
- What happened to SQL cache method: None?
- Database Charset dropdown, is there Latin9 (should be latin1?)

Ive removed the required option now on the DB password. My bad on missing the None option. Similarly with the latin1/9

Quote:

Admin Setup
- Messages regarding initial password (This is your initial temporary password for your Admin Superuser Account. Please ensure you make a note of it.) and initial directory (Your store Admin directory has been automatically renamed. Please ensure you make a note of this directory.) appear below the associated fields; I was confused initially as to which field was associated with the message and think it would flow better if the messages were either above or beside the associated field. I tend to read top->bottom, left->right (that might be different in different cultures).

Agreed, and othere have pointed this out as well. Will try and rejig that page to make the prompts better.

Quote:

Setup Finished
- The message (You need to remove the /zc_install folder so that someone can't re-install your shop again and wipe out your database! Warnings will appear until the folder has been removed ) is overly scary.
-The message "warnings will appear" is not quite correct because I can't log into the admin without the rename. A "warning" shouldn't be as insistent; the message should read "A message will appear and you will not be able to log into your admin until the folder has been removed." (or something of the ilk).

Noted :smile:

Quote:

Admin main page
- Click the V on General Statistics. TEXT_CANCEL instead of Cancel

Admin Configuration Menu
- Dropdown shows Cowoa Checkout ... shouldn't it be COWOA (all caps)?
- Once in Cowoa Checkout, the page title reads "Guest Checkout" ... shouldn't it be the same as the dropdown title?
- I thought that the zen_trace function was supposed to be included in v1.6.0, but couldn't find it in the Configuration->Logging menu.

Storefront, COWOA checkout
- Some funky formatting going on with the progress bar:
Attachment 12488

COWOA and other checkout stuff will be getting a refresh shortly once I finish and merge the new checkout flow infrastructure code.

Show 100 post(s) from this thread on one page