Make WordPress Core

Opened 11 years ago

Closed 11 years ago

#19743 closed defect (bug) (fixed)

Trim code from Twenty Eleven theme-options

Reported by: emhr's profile emhr Owned by: markjaquith's profile markjaquith
Milestone: 3.4 Priority: high
Severity: normal Version: 3.3.1
Component: Bundled Theme Keywords: has-patch commit
Focuses: Cc:


Suggest removal of the conditional statement for add_option() in twentyeleven_theme_options_init()

The function twentyeleven_get_theme_options() uses get_option() and sets the $default parameter.

The conditional statement will never evaluate twentyeleven_get_theme_options() as false. This is probably an artifact from a previous release. As far as I can tell, removing it has no effect to the current revision except for trimming code.

Attachments (3)

twenty-eleven-options-patch.diff (803 bytes) - added by emhr 11 years ago.
19743.diff (1.3 KB) - added by nacin 11 years ago.
19743.2.diff (1.0 KB) - added by Mamaduka 11 years ago.

Download all attachments as: .zip

Change History (11)

#1 @dd32
11 years ago

For a new install this would mean an additional SQL each page load until they do something to cause the options to be set (for example, setting a theme option).

11 years ago

#2 @nacin
11 years ago

Hmm, yes, it is an artifact. The goal was to avoid an extra query, as dd32 writes. 19743.diff is what was intended.

#3 @emhr
11 years ago

Where is the extra query generated?

#4 @nacin
11 years ago

If the option doesn't exist in the database, it won't get auotloaded, which means WordPress needs to hit the options table to check for that individual row.

#5 @emhr
11 years ago


11 years ago

#6 @Mamaduka
11 years ago

I think there's no need to save options into DB, before user save theme from options panel. Twenty Twelve is going use this solution and Lance mentioned, that Nacin had same idea for Twenty Eleven. The diff above removes it.

#7 @nacin
11 years ago

  • Cc Otto42 added
  • Keywords commit added
  • Milestone changed from Awaiting Review to 3.4
  • Priority changed from normal to high
  • Severity changed from minor to normal

#8 @markjaquith
11 years ago

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

In [20875]:

Remove some dead add_option() code in Twenty Eleven. props Mamaduka, emhr. fixes #19743

Note: See TracTickets for help on using tickets.