WordPress.org

Make WordPress Core

Opened 10 years ago

Closed 8 years ago

#1586 closed enhancement (wontfix)

Database enum('Y','N') and enum('YES','N') replaced with BOOLEAN

Reported by: J2000_ca Owned by:
Milestone: Priority: low
Severity: normal Version: 1.5.2
Component: Optimization Keywords:
Focuses: Cc:

Description

In the linkcategories, links, options there are both enum('Y','N') and enum('YES','N') and in posts ping_status might be better served as a boolean.

Change History (3)

comment:1 @markjaquith10 years ago

This could break any plugins that access these directly...

comment:2 @J2000_ca10 years ago

It will probably break all plugins that use add_option unless it is left acccepting yes and no for one version. What I would do is switch the database in 1.6 so anyone who directly has to change and make the method optionally use yes and no and then in 1.8 remove it.

comment:3 @mdawaffe8 years ago

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

BOOLEAN was introduced in MySQL 4.1. WP only requires 3.23.

We could go to TINYINT, but I don't see any reason to. With these and markjaquith's above, I'm marking as wontfix. Reopen with further thoughts if necessary.

Note: See TracTickets for help on using tickets.