long theme name length causes menu bug

[jaredatch]

Was working on a theme where the theme name was "TTI Center for International Intelligent Transportation Research" (long, I know) when I stumbled upon a strange issue: the menu options under Theme Locations would not save.

I was able to duplicate the problem with Twenty Ten. If the theme is renamed to something longer, such as Twenty Ten Nine Eight Seven Six Five Four Three Two, the menu locations will not save or come up.

[nacin]

It would need to be longer than about 60 characters.

Theme mods (which include headers, backgrounds, and theme locations) are stored by theme name rather than by slug. I really want to change that and all of the other times we pass theme name around in the themes functions.

What we could do is start storing it as the slug, then check the name second (and shift it to the slug after). That should handle any compat issues with people using get_theme_mod.

[jaredatch]

Yeah it's not really a big deal, it just took me a whole day of smashing my head against the wall to figure out why my menus wouldn't save. I just wish I would of known there was a name limit :P

Yesterday I dug through wp.org and found 3 threads - all unsolved - with people describing this exact issue, even though most likely rare.

I realize having a name that long (like described above) is pretty stupid but in my case it just happened to be the client name.

I like nacin's suggestion, +1

[nacin]

First step in removing theme names from being passed around and stored.

[ryan]

To avoid options table write storms and mass cache invalidation on busy multisite sites, option updates need to be limited to admin page loads.

[nacin]

Only perform the swap when is_admin.

[nacin]

(In [15736]) Shift theme mods to be reliant on the theme slug instead of name. fixes #15048.

[Denis-de-Bernardy]

Don't forget to announce this one in the change log. It might break a couple themes and plugins.

[nacin]

(In [15739]) Some doc updates. see #15048.