Page 1 of 2

Testing 0.10b

Posted: Thu Apr 14, 2005 4:51 pm
by piratos
Installing  0.10b local Windows

First Step Ok
Continue
Than this:
Fatal error: Class 'ModuleOperations' not found in C:\xampp\htdocs\cms\lib\classes\class.module.inc.php on line 29

Cannot test this version.

Re: Testing 0.10b

Posted: Thu Apr 14, 2005 4:58 pm
by Ted
Do you have eAccelerator running?  There is something about it that's causing a problem that I'll look into tonight or this weekend.

Re: Testing 0.10b

Posted: Thu Apr 14, 2005 5:04 pm
by piratos
Yes i have .

Next Error

Mysqli needs fom adodb the file  \datadict\datadict-generic.inc.php, that file is not in the download file

To use Mysqli the install.php must change:

from

$result = $db->Connect($_POST['host'].":".$_POST['port'],$_POST['username'],$_POST['password'],$_POST['database']);

to

$result = $db->Connect($_POST['host'],$_POST['username'],$_POST['password'],$_POST['database']);

Re: Testing 0.10b

Posted: Thu Apr 14, 2005 5:04 pm
by Ted
Does that type of connect string work for mysql and pgsql as well?  (I assume they do)

Re: Testing 0.10b

Posted: Thu Apr 14, 2005 5:10 pm
by Ted
Nevermind, I answered my own question on that one.  Thanks for the bug fixes!

Re: Testing 0.10b

Posted: Thu Apr 14, 2005 5:13 pm
by piratos
Now without eaccelerator, but with changes in install.php and mysqli.

Install ok but wrong output:



[attachment deleted by admin]

Re: Testing 0.10b

Posted: Thu Apr 14, 2005 5:15 pm
by Ted
I guess I better install php5 and see what happens.  Maybe that's an issue.

Re: Testing 0.10b

Posted: Thu Apr 14, 2005 5:58 pm
by Ted
Found the problem with eAccelerator.  Moving the ModuleOperations class above the Module class in lib/classes/class.module.inc.php seems to fix it.  And, yes, it does make things a lot faster.  I'll release all these fixes in a beta2 sometime tomorrow.

Re: Testing 0.10b

Posted: Thu Apr 14, 2005 6:12 pm
by piratos
The other error is that your initial.sql is not compatibel with mysqli .

Re: Testing 0.10b

Posted: Thu Apr 14, 2005 6:29 pm
by Ted
What was the exact deal with that?  Was it a lowercase/uppercase issue?

Re: Testing 0.10b

Posted: Fri Apr 15, 2005 8:44 am
by piratos
This i have reported here:

http://forum.cmsmadesimple.org/index.ph ... ml#msg3780

You see , this problem is an old problem. 0.10b would be better if you read more in this forum.

Re: Testing 0.10b

Posted: Fri Apr 15, 2005 9:44 am
by Ted
...and it would be much easier to find bugs if they were reported in the bug tracking system instead of randomly on the forum.

Anyway, I fixed this last night.

Re: Testing 0.10b

Posted: Fri Apr 15, 2005 10:13 am
by piratos
Another error with mysqli:

In install.php sometimes tables are declared as follow:

$dbdict = NewDataDictionary($db);
$flds = "
content_id I KEY,
type C(25),
prop_name C(255) KEY,
param1 C(255),
param2 C(255),
param3 C(255),
content X,
create_date T,
modified_date T
";

The Value X comes as VARCHAR(255) must be LONGTEXT.

And here is the complete initial.sql:




[attachment deleted by admin]

Re: Testing 0.10b PHOPMYADMIN Warnings

Posted: Fri Apr 15, 2005 10:16 am
by piratos
Too many indexex on one field causes warnings so here where you are using content_alias twice:


Indizes: Dokumentation
Name Typ Kardinalität Aktion Feld
PRIMARY PRIMARY 5  Bearbeiten Löschen content_id
content_alias INDEX keine  Bearbeiten Löschen content_alias
active
content_alias_2 INDEX keine  Bearbeiten Löschen content_alias
Warning Es sollte nicht mehr als ein Index des Typs INDEX für die Spalte `content_alias` gesetzt sein

Re: Testing 0.10b

Posted: Fri Apr 15, 2005 11:34 am
by Ted
That's what I fixed.  The generic datadict makes them varchar(255).  Instead, I modifed the mysqli driver to use the mysql datadict and now it creates the tables correctly.

That's strange about the indexes.  You should be able to index it a bunch of different ways based on various queries.