Make WordPress Core

Opened 16 years ago

Closed 16 years ago

Last modified 15 years ago

#1953 closed defect (bug) (fixed)

clean installation not ok as of mysq

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


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:
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.


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 16 years ago.
Set new DB field $comment_parent when comment posted.

Download all attachments as: .zip

Change History (8)

16 years ago

Set new DB field $comment_parent when comment posted.

#1 @error
16 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
16 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
16 years ago

  • Priority changed from normal to low

#4 @ryan
16 years ago

  • Owner changed from anonymous to ryan

#6 @ryan
16 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)
15 years ago

  • Milestone 2.0 deleted

Milestone 2.0 deleted

Note: See TracTickets for help on using tickets.