I welcome any questions, comments and feedback.Praise: - Nice error messages. - Clean code
Thank you!
Unfortunately, I can't get any further because CGI::Session::MySQL is looking for an autoload file that does not exist in the CGI::Session distribution---set_table_n.al. Any ideas? I've appended the full error message at the end of this reply.
[From end of reply]
Can't locate auto/CGI/Session/MySQL/set_table_n.al in @INC (@INC contains: /home/william/local/flexmart-0.1/flexmart/lib /home/william/local/flexmart-0.1/fmdemo/lib/cgiapp /etc/perl /usr/local/lib/perl/5.8.4 /usr/local/share/perl/5.8.4 /usr/lib/perl5 /usr/share/perl5 /usr/lib/perl/5.8 /usr/share/perl/5.8 /usr/local/lib/site_perl /usr/local/lib/perl/5.8.3 /usr/local/share/perl/5.8.3 /usr/local/lib/perl/5.8.2 /usr/local/share/perl/5.8.2 . /usr/local/apache/ /usr/local/apache/lib/perl) at /home/william/local/flexmart-0.1/flexmart/lib/FlexMart/Session.pm line 35
I suspect you did not install CGI::Session::MySQL package separately? Do you have a MySQL.pm file? Is it in your path? I think perl is trying to autoload because CGI::Session::MySQL.pm is not being found in your path.
Criticisms: - Tedious, manual install.
Well, its not ideal, and the config file is not as well documented as it could be. However, the instructions do say where to put stuff, and its mostly a matter of picking 2 "top level directories" and untarring the 2 directory trees into them (one for system libs and the other for host (or rather, virtual host) local files).
- I don't use MySQL so had to install it and figure out how to configure it in order to continue; support for alternative databases such as PostgreSQL or SQLLite (which would require no external database engine to be configured) would be great
Yes. SQLLite would be perfect. The config files and class structure allow for this implementation in the future. In fact, I may make this a higher priority to encourage support for both engines by others as well.
- There are several module dependenices which I did not have installed (DBD::MySQL, CGI::Session::MySQL, Error to name a few). A Build.PL would be helpful to let me know what I need to install on my local system. Including some of the pure Perl modules that have appropriate licenses in a PAR file would be even better.
Yes. Understood.
- You do not have any automated tests to confirm the installation or the logic of your code. This concerns me and would prevent me from going further with this codebase. I'd be willing to help you get the Apache::Test framework working and writing some basic tests to get you started.
I am not familiar with this framework or the benefits it would provide. I am sure that I would enjoy gaining a better understanding of what you had in mind. Examples or a further description would be a great start.
- I don't understand why you have split the distribution into two folders. I suppose it's for environments where the user does not have access to the global httpd.conf file.
No. One config is a system config, the other a host (or perhaps better said, a virtual host) config. This allows a single system config with default and system settings, and tweaking for virtual hosts in the host config. What's more, the files can contain interpolateable variables that can reference the other config - ie, interpolation can go both ways between the files!
However, having two sets
of configuration files is confusing. From the looks of things, it appears that the files in fmdemo subclass/override those in the flexmart directory. If this is the case, why have a global config file at all?
You can have templates in the sys config that reference vhost config values for all or part of a sys value. Its a much more powerful setup for systems running FlexMart on multiple vhosts.
Thanks, William
Thanks for the feedback William! Please let me know if you figure out the CGI::Session::MySQL issue!
Bill
----------
Uh oh ... there seems to have been an error. We apologize for any inconvenience.The following error has occurred: The erroneous input: Not available. Suggested help:The previous request generated a fatal error. The error has been logged and a system administrator has been notified. We apologize for any inconvenience.---------- -- Knowmad Services Inc. http://www.knowmad.com --------------------------------------------------------------------- Web Archive: http://www.mail-archive.com/suppressed/ http://marc.theaimsgroup.com/?l=cgiapp&r=1&w=2 To unsubscribe, e-mail: suppressed For additional commands, e-mail: suppressed
---------------------------------------------------------------------
Web Archive: http://www.mail-archive.com/suppressed/
http://marc.theaimsgroup.com/?l=cgiapp&r=1&w=2
To unsubscribe, e-mail: suppressed
For additional commands, e-mail: suppressed
Mail converted by mhonarc 2.6.15
This archive provided courtesy of JSW4.NET, Internet Hosting Services for Small Business.