Make WordPress Core

Opened 19 years ago

Closed 18 years ago

Last modified 6 months ago

#1587 closed defect (bug) (invalid)

linkcategories table sort_order varchar(64) should be an enum

Reported by: j2000_ca's profile J2000_ca Owned by:
Milestone: Priority: low
Severity: minor Version: 1.5.2
Component: Optimization Keywords: bg|has-patch|needs review optimization
Focuses: Cc:

Description

In the linkcategories table the column sort_order is a varchar(64). Really though the sort_order can only be name or id or url or rating or updated or rand. So it would be better served as an enum with those choices.

Attachments (1)

ticket-1587-patch.diff (1.9 KB) - added by MikeLittle 19 years ago.
Patch to implement this change

Download all attachments as: .zip

Change History (6)

#1 @markjaquith
19 years ago

  • Keywords bg|needs-patch optimization added

Sounds reasonable

@MikeLittle
19 years ago

Patch to implement this change

#2 @MikeLittle
19 years ago

I've added a patch to implement this change. Note that I have updated the db version number to match the svn revision current at the time (3022). I appreciate that it may need to be changed when the patch is submitted.

#3 @MikeLittle
19 years ago

  • Keywords bg|has-patch|needs review added; bg|needs-patch removed

#4 @ketsugi
18 years ago

Is this still necessary now that the linkcategories table is deprecated?

#5 @ryan
18 years ago

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

linkcategories is going away. So are enums for that matter.

Note: See TracTickets for help on using tickets.