Make WordPress Core

Opened 19 years ago

Closed 19 years ago

Last modified 18 years ago

#1953 closed defect (bug) (fixed)

clean installation not ok as of mysq

Reported by: edgarm's profile EdgarM Owned by: ryan's profile ryan
Milestone: Priority: low
Severity: normal Version: 1.6
Component: Administration Keywords: installation mysql default bg|has-patch
Focuses: Cc:

Description

When installing a new instance of Wordpress 2.0 Beta is not really possible, as several tables have new columns but the columns are not filled via the installationscript.

Also the fix that was mentioned in here:
http://trac.wordpress.org/changeset/3199
will not really help, as of mysql in both 4.x and 5.x Version do not allow defaults in text, mediumtext and longtext fields.

So the bugfix should be the other way around, not to change the default than the inserts in the install.php.

regards
EdgarM

PS: The installation does not fail, but most inserts are not executed, so e.g. no post in in the db.

Attachments (1)

comment_parent.diff (896 bytes) - added by error 19 years ago.
Set new DB field $comment_parent when comment posted.

Download all attachments as: .zip

Change History (8)

@error
19 years ago

Set new DB field $comment_parent when comment posted.

#1 @error
19 years ago

  • Keywords bg|has-patch added; removed

One example of this seems to be that comment_parent is unset and MySQL 5 doesn't like this (it's NOT NULL, of course). The attached patch sets comment_parent (to 0). This will probably need to be reworked so that a plugin can override this and provide thhe threaded comments functionality, or it can be put in core, but if that isn't needed, this patch should do fine.

#2 @RustIndy
19 years ago

This worked for me, MySQL 5.0/strict.

I have to agree with Edgar though. Fixing the installer should be done by fixing the INSERTs, not by changing the schema.

#3 @matt
19 years ago

  • Priority changed from normal to low

#4 @ryan
19 years ago

  • Owner changed from anonymous to ryan

#6 @ryan
19 years ago

  • Resolution set to fixed
  • Status changed from new to closed

(In [3285]) Make sure fields without defaults are specified when inserting. fixes #1953

#7 @(none)
18 years ago

  • Milestone 2.0 deleted

Milestone 2.0 deleted

Note: See TracTickets for help on using tickets.