https://issues.apache.org/ooo/show_bug.cgi?id=118845
--- Comment #3 from Ariel Constenla-Haile <[email protected]> --- The theme was removed with impress210: #i105310# set tooltip for items and some refactoring author [email protected] Tue Feb 08 12:29:15 2011 +0100 (21 months ago) changeset 276753 902597eed541 parent 276752 2f0f385e8b25 child 276754 fbdede3ae6d9 http://hg.services.openoffice.org/OOO340/rev/902597eed541 http://hg.services.openoffice.org/OOO340/diff/902597eed541/svx/inc/svx/gallery.hxx http://hg.services.openoffice.org/OOO340/diff/902597eed541/svx/source/gallery2/galtheme.src http://hg.services.openoffice.org/OOO340/diff/902597eed541/svx/inc/galtheme.hrc The underlying issue is that this commit removed all the code, including the resource ID and the string, but it didn't remove the theme files. Copy and paste from the dev@ mailing list: * the private URL is stored in the theme itself as its name * hidden themes are not displayed in the UI: A hidden theme is one whose name begins with "private://gallery/hidden/" see GalleryThemeEntry sal_Bool IsHidden() const { return aName.SearchAscii( "private://gallery/hidden/" ) == 0; } This gallery theme is displayed on a non-pro build because the gallery theme is not considered as hidden: though the name is really stored inside the theme file, it is replaced in the GalleryThemeEntry constructor: * the constructor gets the theme name as read from the theme file * it tries to get the localized theme name from the resource first * UniString::UniString( const ResId& rResId ) instead of creating an empty string when the respective resource is missing (default behaviour on a pro build), it returns a string with the error, see http://opengrok.adfinis-sygroup.org/source/xref/aoo-trunk/main/tools/source/string/strucvt.cxx#195 * this string has a length > 0, then it replaces the original string as read from the theme file * this way, IsHidden will return false: the theme is not hidden as its name does not start with private://gallery/hidden anymore * then the theme is displayed in the UI on a non-pro build -- You are receiving this mail because: You are on the CC list for the bug.
